[webkit-reviews] review requested: [Bug 77387] Avoid Page::updateViewportArguments() if the causing frame is not the main frame : [Attachment 125366] patch v2 (moved updateViewportArguments from Page to Document)

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Feb 3 10:53:24 PST 2012


Xianzhu Wang <wangxianzhu at chromium.org> has asked  for review:
Bug 77387: Avoid Page::updateViewportArguments() if the causing frame is not
the main frame
https://bugs.webkit.org/show_bug.cgi?id=77387

Attachment 125366: patch v2 (moved updateViewportArguments from Page to
Document)
https://bugs.webkit.org/attachment.cgi?id=125366&action=review

------- Additional Comments from Xianzhu Wang <wangxianzhu at chromium.org>
Thanks for suggestions. Updated patch. Not using ASSERT because I don't want
the callers to know how viewport update and the main frame are related and to
check that.


More information about the webkit-reviews mailing list