New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 642181 link

Starred by 1 user
Status: Fixed
Owner:
█ msten...@opera.com
NOT IN USE
Closed: Feb 2017
Cc:
█ robhogan@chromium.org
█ mummare...@chromium.org
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

flowThreadOffset.mightBeSaturated() || pageLogicalTopForOffset(flowThreadOffset)

Project Member Reported by ClusterFuzz, Aug 29 2016 
Detailed report: https://cluster-fuzz.appspot.com/testcase?key=5454375290142720

Fuzzer: bj_broddelwerk
Job Type: linux_debug_chrome
Platform Id: linux

Crash Type: ASSERT
Crash Address: 
Crash State:
  flowThreadOffset.mightBeSaturated() || pageLogicalTopForOffset(flowThreadOffset)
  blink::LayoutMultiColumnSet::nextLogicalTopForUnbreakableContent
  blink::LayoutFlowThread::nextLogicalTopForUnbreakableContent
  
Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_debug_chrome&range=375946:376263

Minimized Testcase (0.26 Kb):
Download: https://cluster-fuzz.appspot.com/download/AMIfv96-ohEYKSlABY7f2uC4xF1Bx0JBpiq-b91EQQlAimwMGnITW7kJlG6HJBbnhT60w_W4RWbh74hzfEF9WQ3od_mNT4eIdqKzTgqisoz-3VajS07-TFzpN6pOTyevuCQx5BBxK6AbiNguHY3adkCA0fJhr4Idlg?testcase_id=5454375290142720
<style>
*{border-width:thick;-webkit-logical-height:57.6ex;}
.CLASS2{-webkit-column-count:auto;column-width:9ch;}
.CLASS4{-webkit-highlight:auto !important;-webkit-column-span:all;</style>
<body class="CLASS2">
<form>
<hr>
<div class="CLASS4"</textarea>
</div>
</form>
>


Issue manually filed by: mummareddy

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.

Comment 1 by mummare...@chromium.org, Aug 29 2016

Labels: Test-Layout findit-wrong Te-Logged M-53
Owner: robhogan@chromium.org
Status: Assigned (was: Untriaged)
From findit tool:

Author: robhogan
Project: chromium
Changelist: https://chromium.googlesource.com/chromium/src/+/6827228e3e093e665e2460503c258730f570e477
Time: Mon May 09 20:00:24 2016
The CL last changed line 4820 of file LayoutBox.cpp, which is stack frame 2.

Comment 2 by robho...@gmail.com, Aug 30 2016

Cc: robhogan@chromium.org
Labels: -Pri-1 Pri-2
Owner: msten...@opera.com
This is more mstensho's thing.
Project Member

Comment 3 by ClusterFuzz, Sep 23 2016 

ClusterFuzz has detected this issue as fixed in range 420372:420465.

Detailed report: https://cluster-fuzz.appspot.com/testcase?key=5454375290142720

Fuzzer: bj_broddelwerk
Job Type: linux_debug_chrome
Platform Id: linux

Crash Type: ASSERT
Crash Address: 
Crash State:
  flowThreadOffset.mightBeSaturated() || pageLogicalTopForOffset(flowThreadOffset)
  blink::LayoutMultiColumnSet::nextLogicalTopForUnbreakableContent
  blink::LayoutFlowThread::nextLogicalTopForUnbreakableContent
  
Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_debug_chrome&range=375946:376263
Fixed: https://cluster-fuzz.appspot.com/revisions?job=linux_debug_chrome&range=420372:420465

Minimized Testcase (0.26 Kb):
Download: https://cluster-fuzz.appspot.com/download/AMIfv96-ohEYKSlABY7f2uC4xF1Bx0JBpiq-b91EQQlAimwMGnITW7kJlG6HJBbnhT60w_W4RWbh74hzfEF9WQ3od_mNT4eIdqKzTgqisoz-3VajS07-TFzpN6pOTyevuCQx5BBxK6AbiNguHY3adkCA0fJhr4Idlg?testcase_id=5454375290142720
<style>
*{border-width:thick;-webkit-logical-height:57.6ex;}
.CLASS2{-webkit-column-count:auto;column-width:9ch;}
.CLASS4{-webkit-highlight:auto !important;-webkit-column-span:all;</style>
<body class="CLASS2">
<form>
<hr>
<div class="CLASS4"</textarea>
</div>
</form>
>


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.
Project Member

Comment 4 by ClusterFuzz, Sep 23 2016

Labels: ClusterFuzz-Verified
Status: Verified (was: Assigned)
ClusterFuzz testcase is verified as fixed, closing issue.

If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue.

Comment 5 by msten...@opera.com, Sep 28 2016

Status: Assigned (was: Verified)
Still failing here.

Comment 6 by msten...@opera.com, Nov 2 2016 

Something with overflow in the column row preceding a spanner causing trouble for the column row after the spanner.
tc.html
252 bytes View Download
Project Member

Comment 7 by sheriffbot@chromium.org, Nov 22 2016

Labels: -Restrict-View-EditIssue
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
Project Member

Comment 8 by bugdroid1@chromium.org, Feb 13 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2e6930336491fd5bc277dc9476873a684ebeae44

commit 2e6930336491fd5bc277dc9476873a684ebeae44
Author: mstensho <mstensho@opera.com>
Date: Mon Feb 13 18:56:19 2017

ColumnBalancer: need to examine lines that protrude into the flow thread portion.

We cannot skip a line that starts before the flow thread portion of interest,
if it ends inside the portion.

Some extra care is needed to avoid regressions here: The part of a line that
starts before the multicol container itself needs to be ignored, or we risk
overstretching the multicol container.

BUG= 642181 

Review-Url: https://codereview.chromium.org/2690863003
Cr-Commit-Position: refs/heads/master@{#450033}

[add] https://crrev.com/2e6930336491fd5bc277dc9476873a684ebeae44/third_party/WebKit/LayoutTests/fast/multicol/balance-line-underflow-1.html
[add] https://crrev.com/2e6930336491fd5bc277dc9476873a684ebeae44/third_party/WebKit/LayoutTests/fast/multicol/balance-line-underflow-2.html
[add] https://crrev.com/2e6930336491fd5bc277dc9476873a684ebeae44/third_party/WebKit/LayoutTests/fast/multicol/span/underflow-after-spanner.html
[modify] https://crrev.com/2e6930336491fd5bc277dc9476873a684ebeae44/third_party/WebKit/Source/core/layout/ColumnBalancer.cpp

Comment 9 by msten...@opera.com, Feb 13 2017

Status: Fixed (was: Assigned)
►

Sign in to add a comment