[webkit-reviews] review denied: [Bug 107329] Chromium WebKit API should use updateLayout() rather than updateLayoutIgnorePendingStylesheets() : [Attachment 183551] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Jan 18 14:59:05 PST 2013


James Robinson <jamesr at chromium.org> has denied Ilya Sherman
<isherman at chromium.org>'s request for review:
Bug 107329: Chromium WebKit API should use updateLayout() rather than
updateLayoutIgnorePendingStylesheets()
https://bugs.webkit.org/show_bug.cgi?id=107329

Attachment 183551: Patch
https://bugs.webkit.org/attachment.cgi?id=183551&action=review

------- Additional Comments from James Robinson <jamesr at chromium.org>
I don't think this patch does what you think it does.  All (nearly all?) dom
accessors that trigger layout use the ..IgnorePendingStylesheets() version.

If you decide you do want a behavior change, it should be covered in
webkit_unit_tests


More information about the webkit-reviews mailing list