[webkit-reviews] review granted: [Bug 14282] Cannot change the height of the docked Web Inspector : [Attachment 22767] Proposed patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Aug 13 10:34:48 PDT 2008

Kevin McCullough <kmccullough at apple.com> has granted Timothy Hatcher
<timothy at hatcher.name>'s request for review:
Bug 14282: Cannot change the height of the docked Web Inspector

Attachment 22767: Proposed patch

------- Additional Comments from Kevin McCullough <kmccullough at apple.com>
on line 51 do you need to specify the name of the input parameter?

On line 1367 I don't think you need to specify that setAttachedWindowHeight is
in the WebCore namespace

Line 122.  Don't need "height" again

Line 585: I don't understand the logic in toolbarDragStart.

Also please file a bug to implement setAttachedWindowHeight in

More information about the webkit-reviews mailing list