CHECK failure: isFirstAfterBreak(lineTopInFlowThread) || !line.paginationStrut() || !isLogicalT |
||||
Issue descriptionDetailed report: https://clusterfuzz.com/testcase?key=6183192664211456 Fuzzer: mbarbella_js_mutation_layout Job Type: linux_debug_content_shell_drt Platform Id: linux Crash Type: CHECK failure Crash Address: Crash State: isFirstAfterBreak(lineTopInFlowThread) || !line.paginationStrut() || !isLogicalT blink::MinimumSpaceShortageFinder::examineLine blink::ColumnBalancer::traverseLines Sanitizer: address (ASAN) Regressed: https://clusterfuzz.com/revisions?job=linux_debug_content_shell_drt&range=443258:443393 Reproducer Testcase: https://clusterfuzz.com/download/AMIfv95PD8Kw945diUj7oVLbZOtN1yUTG4cHhdtZ8ExROKL46YOGZu_UHySSQPugNOG4B9a50xAZbraxrJ_Csw_XbpQ4VXLEHnzZMbEU8LQGsh9UYlDMteIhyKcsSNmqD3YLgEtkUsAs_3tC9_EVPl6OtPev4WpRL4DI6TPtOEELnH9oBNeEcQN4NMCUoJ84lHssTe6yiiu10KenoIcz3dNTpDQTaRmndcPathagF2SQXKlEvemRPmR80O7sAYWfZ6cPVCbLaGfneuvE9k7L0r7y6T86KzXTY8Yy1-OhnF4ZikZRNvyqzDnvb1Z5viPEg8QXaoKPoDLd5zeS9KDDWHGYN5U17R7lyN0dXmLdLLDJ-9S964yQBZ6me7QS_wSOfZa9sIeVfAEgfgtP8_R7d2r-igSqXBg4cg?testcase_id=6183192664211456 Issue filed automatically. See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
,
Apr 7 2017
Yeah, I suspected that one too. Although that CL just removes a DCHECK, I was kind of assuming that this new bug here was just hiding behind the other now-removed DCHECK. However, reverting it doesn't make any difference. Still DCHECK-fails at the same place. Anyway, this one is mine. :)
,
Apr 7 2017
This looks like a very old bug, and something I failed to fix while working on bug 534751 . When fragmenting, we need to position everything before laying it out.
,
Apr 25 2017
,
Apr 28 2017
,
May 5 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/97b20197a53d323f7b2d92581d97817ea85ea0a4 commit 97b20197a53d323f7b2d92581d97817ea85ea0a4 Author: mstensho <mstensho@opera.com> Date: Fri May 05 17:09:08 2017 The first table row is pushed down by border-spacing. We need to make sure this happens *before* laying it out when inside a fragmentation context. Added tests fast/multicol/balance-table-with-border-spacing.html and fragmentation/table-with-border-spacing.html for this. This change also fixes breaking inside border-spacing adjacent to table rows with break-inside:avoid set. There should be no reason to prevent breaking inside border spacing, just because it's adjacent to such table rows, but it looks like this was the behavior we got, by accident. Updated printing/avoid-setting-header-offset-on-header.html accordingly and threw in an additional test fragmentation/border-spacing-break-before-unbreakable-row.html for this collateral fix. It's hopefully correct, since we now match Edge's behavior. BUG= 709387 , 534751 Review-Url: https://codereview.chromium.org/2803383002 Cr-Commit-Position: refs/heads/master@{#469690} [add] https://crrev.com/97b20197a53d323f7b2d92581d97817ea85ea0a4/third_party/WebKit/LayoutTests/fast/multicol/balance-table-with-border-spacing.html [add] https://crrev.com/97b20197a53d323f7b2d92581d97817ea85ea0a4/third_party/WebKit/LayoutTests/fragmentation/border-spacing-break-before-unbreakable-row.html [add] https://crrev.com/97b20197a53d323f7b2d92581d97817ea85ea0a4/third_party/WebKit/LayoutTests/fragmentation/table-with-border-spacing.html [modify] https://crrev.com/97b20197a53d323f7b2d92581d97817ea85ea0a4/third_party/WebKit/LayoutTests/printing/avoid-setting-header-offset-on-header-expected.html [modify] https://crrev.com/97b20197a53d323f7b2d92581d97817ea85ea0a4/third_party/WebKit/LayoutTests/printing/avoid-setting-header-offset-on-header.html [modify] https://crrev.com/97b20197a53d323f7b2d92581d97817ea85ea0a4/third_party/WebKit/Source/core/layout/LayoutTableSection.cpp [modify] https://crrev.com/97b20197a53d323f7b2d92581d97817ea85ea0a4/third_party/WebKit/Source/core/layout/LayoutTableSection.h
,
May 5 2017
,
May 6 2017
ClusterFuzz has detected this issue as fixed in range 469683:469704. Detailed report: https://clusterfuzz.com/testcase?key=6183192664211456 Fuzzer: mbarbella_js_mutation_layout Job Type: linux_debug_content_shell_drt Platform Id: linux Crash Type: CHECK failure Crash Address: Crash State: isFirstAfterBreak(lineTopInFlowThread) || !line.paginationStrut() || !isLogicalT blink::MinimumSpaceShortageFinder::examineLine blink::ColumnBalancer::traverseLines Sanitizer: address (ASAN) Regressed: https://clusterfuzz.com/revisions?job=linux_debug_content_shell_drt&range=443258:443393 Fixed: https://clusterfuzz.com/revisions?job=linux_debug_content_shell_drt&range=469683:469704 Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=6183192664211456 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. |
||||
►
Sign in to add a comment |
||||
Comment 1 by msrchandra@chromium.org
, Apr 7 2017Labels: Test-Predator-Wrong M-57
Owner: msten...@opera.com
Status: Assigned (was: Untriaged)