Multicolum + topLayer + HTMLDialogElement::showModal leads to a crash |
||||||
Issue descriptionDetailed report: https://cluster-fuzz.appspot.com/testcase?key=5088116829847552 Fuzzer: inferno_twister Job Type: linux_lsan_chrome_mp Platform Id: linux Crash Type: UNKNOWN Crash Address: 0x000000000000 Crash State: blink::CompositedLayerMapping::localClipRectForSquashedLayer blink::CompositedLayerMapping::updateSquashingLayerGeometry blink::CompositedLayerMapping::updateGraphicsLayerGeometry Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_lsan_chrome_mp&range=379568:379622 Minimized Testcase (2.85 Kb): https://cluster-fuzz.appspot.com/download/AMIfv97IRb5bJEuqgCcKoVse7I0cQQHFcUIvyJ-_oihrkWO4WCDpAD3gYuUHaItKTdg5KIt8QKF37F3WzIG1bdB3bjDKcNAFGhzP9I3ZibiFCIffjICY5AITfvkymZZATrzDDm0ZTN4C8q3LiYSxoH-sptF1apfF9w Filer: pucchakayala See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
,
Mar 15 2016
The proximate cause is that the div with id tCF11 is squashed into the one with id tCF7 on the first frame. On the second frame, the modal dialog is applied, which changes the layout tree. Compositing tries to update all PaintLayers to apply the new compositing, but tCF11 is not found in the PaintLayerTree for some reason. It looks like something is getting messed up with multicolumn in the presence of top layers. Morten could you look? I think you know this code best.
,
Mar 17 2016
Attaching cleaned up test case, which still seems to crash like the original test (I get an ASSERT(ancestorPaintInfo) failure in CompositedLayerMapping::localClipRectForSquashedLayer()). I tried to simplify it further, but then I got it to crash/assert at other places, so I'll stop here for now. Paginating the viewport (which puts EVERYTHING inside a flow thread child of LayoutView) may be a poisonous combination with Element::setIsInTopLayer(), but I don't know yet.
,
Mar 18 2016
Remove legacy label cr-blink
,
Mar 18 2016
,
Mar 22 2016
ClusterFuzz has detected this testcase as flaky and is unable to reproduce it in the original crash revision. Skipping fixed testing check and marking it as potentially fixed. Detailed report: https://cluster-fuzz.appspot.com/testcase?key=5088116829847552 Fuzzer: inferno_twister Job Type: linux_lsan_chrome_mp Platform Id: linux Crash Type: UNKNOWN Crash Address: 0x000000000000 Crash State: blink::CompositedLayerMapping::localClipRectForSquashedLayer blink::CompositedLayerMapping::updateSquashingLayerGeometry blink::CompositedLayerMapping::updateGraphicsLayerGeometry Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_lsan_chrome_mp&range=379568:379622 Minimized Testcase (2.85 Kb): https://cluster-fuzz.appspot.com/download/AMIfv97IRb5bJEuqgCcKoVse7I0cQQHFcUIvyJ-_oihrkWO4WCDpAD3gYuUHaItKTdg5KIt8QKF37F3WzIG1bdB3bjDKcNAFGhzP9I3ZibiFCIffjICY5AITfvkymZZATrzDDm0ZTN4C8q3LiYSxoH-sptF1apfF9w See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information. If you suspect that the result above is incorrect, try re-doing that job on the test case report page.
,
Apr 7 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5233aa994b11d17c287311089234f58dbbead280 commit 5233aa994b11d17c287311089234f58dbbead280 Author: mstensho <mstensho@opera.com> Date: Thu Apr 07 21:40:19 2016 Make top-layer elements work also when the viewport is paginated. When the viewport is paginated (by overflow:-webkit-paged-* specified on HTML or BODY), top-layer elements are redirected to a flow thread, along with everything else. So we have to go through the children of the flow thread, not the children of the layout view, when looking for them. BUG= 594306 Review URL: https://codereview.chromium.org/1850153002 Cr-Commit-Position: refs/heads/master@{#385883} [add] https://crrev.com/5233aa994b11d17c287311089234f58dbbead280/third_party/WebKit/LayoutTests/fast/pagination/modal-dialog-crash-expected.txt [add] https://crrev.com/5233aa994b11d17c287311089234f58dbbead280/third_party/WebKit/LayoutTests/fast/pagination/modal-dialog-crash.html [add] https://crrev.com/5233aa994b11d17c287311089234f58dbbead280/third_party/WebKit/LayoutTests/fast/pagination/modal-dialog-expected.html [add] https://crrev.com/5233aa994b11d17c287311089234f58dbbead280/third_party/WebKit/LayoutTests/fast/pagination/modal-dialog.html [modify] https://crrev.com/5233aa994b11d17c287311089234f58dbbead280/third_party/WebKit/Source/core/paint/PaintLayerStackingNode.cpp
,
Apr 8 2016
,
Nov 22 2016
Removing EditIssue view restrictions from ClusterFuzz filed bugs. If you believe that this issue should still be restricted, please reapply the label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by pucchakayala@chromium.org
, Mar 12 2016Labels: Te-Logged M-50
Owner: chrishtr@chromium.org
Status: Assigned (was: Available)