[webkit-reviews] review denied: [Bug 4840] Frames only resize wider, can't make them narrower : [Attachment 6833] Don't lose track of the resizing frameset

[bugzilla-request-daemon at opendarwin.org]

Darin Adler <darin at apple.com> has denied Darin Adler <darin at apple.com>'s
request for review:
Bug 4840: Frames only resize wider, can't make them narrower
http://bugzilla.opendarwin.org/show_bug.cgi?id=4840

Attachment 6833: Don't lose track of the resizing frameset
http://bugzilla.opendarwin.org/attachment.cgi?id=6833&action=edit

------- Additional Comments from Darin Adler <darin at apple.com>
We would like a capturing mechanism that works for all drags. But adding one
specifically for the frameset resizing seems fine for now.

It's not good to add stuff about resizing to the DOM, though. This should stay
in the view system (rendering and FrameView).

+    DocumentImpl *d = getDocument();
+    FrameView *w = d ? d->view() : 0;
+    if (w)
+	 w->setResizingFrameSet(e ? this : 0);

getDocument no longer ever returns 0 -- nodes now reference their document --
so we don't need a nil-check on the document. I'd also like to see the
FrameView* style used instead of FrameView * as per our style guidelines. But
this function should go.

RenderFrameSet::setResizing should handle this directly, calling the FrameView.
You don't need a boolean in the renderer or the DOM element.

Nice fix in hoverTimerFired.