[webkit-reviews] review requested: [Bug 52449] Crash when logging into gmail.com with frame flattening turned on. : [Attachment 79776] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Jan 21 12:36:19 PST 2011


Yael <yael.aharon at nokia.com> has asked	for review:
Bug 52449: Crash when logging into gmail.com with frame flattening turned on.
https://bugs.webkit.org/show_bug.cgi?id=52449

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

------- Additional Comments from Yael <yael.aharon at nokia.com>
Frame flattening algorithm requires that layout always starts from the main
frame, since layout of subframes impacts the layout of their parents. There are
places in the code that call view->layout() not on the main frame. Instead of
changing all the callsites, I changed FrameView::layout() to force layout from
the main frame if frame flattening is enabled.
In addition, postLayoutTasks can trigger relayout, so make it use the timer
even more.


More information about the webkit-reviews mailing list