Summary: | Should update compositing state when an out-of-view fixed position element becomes in-view | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Kevin M. Dean <kevin> | ||||||
Component: | Layout and Rendering | Assignee: | Xianzhu Wang <wangxianzhu> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | bdakin, simon.fraser, wangxianzhu, webkit-bug-importer, webkit.review.bot | ||||||
Priority: | P1 | Keywords: | InRadar, Regression | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
URL: | http://demo.phpmyadmin.net/STABLE/ | ||||||||
Attachments: |
|
Description
Kevin M. Dean
2013-01-20 20:15:04 PST
Confirmed with <http://demo.phpmyadmin.net/STABLE/> (login:root, empty password). I'm getting a blank menu bar even on initial load. Tried to regress this but demo.phpmyadmin.net is way too slow. This is the only likely cause in that range: http://trac.webkit.org/changeset/139461 Verified on Chromium ToT with "--enable-fixed-position-creates-stacking-context --enable-fixed-position-compositing". Just tested without http://trac.webkit.org/changeset/139461 and the bug still existed, so this seems not a recent regression. When a non-compositing page contains some not-composited fixed position element because of bounds out-of-view, when the element changes position and RenderLayerCompositor::updateCompositingLayers() is called, the function may return early because of "if (!m_reevaluateCompositingAfterLayout && !m_compositing)" without updating the compositing layers. Should set m_reevaluateCompositingAfterLayout when a fixed position element is not composited because of bounds out-of-view. Created attachment 184286 [details]
Patch
Comment on attachment 184286 [details]
Patch
Sorry for the wrong version of the expected file.
Created attachment 184301 [details]
Patch
Comment on attachment 184301 [details] Patch Clearing flags on attachment: 184301 Committed r140593: <http://trac.webkit.org/changeset/140593> All reviewed patches have been landed. Closing bug. |