Closed
Bug 1153695
Opened 8 years ago
Closed 7 years ago
"Assertion failure: !nif || (!mFloats.ContainsFrame(nif) && !mFrames.ContainsFrame(nif))" with -moz-column
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla45
People
(Reporter: jruderman, Assigned: MatsPalmgren_bugz)
References
Details
(Keywords: assertion, regression, testcase)
Attachments
(6 files)
666 bytes,
text/html
|
Details | |
9.62 KB,
text/plain
|
Details | |
4.65 KB,
text/html
|
Details | |
1.20 KB,
patch
|
roc
:
review+
|
Details | Diff | Splinter Review |
1.79 KB,
patch
|
Details | Diff | Splinter Review | |
750 bytes,
patch
|
Details | Diff | Splinter Review |
Assertion failure: !nif || (!mFloats.ContainsFrame(nif) && !mFrames.ContainsFrame(nif)), at layout/generic/nsBlockFrame.cpp:7419
Reporter | ||
Comment 1•8 years ago
|
||
Assignee | ||
Comment 2•8 years ago
|
||
Is this a recent regression? It seems I can reproduce it my Aurora tree but not in Beta.
Comment 3•8 years ago
|
||
Local bisection is pointing at bug 447660 as the culprit, more specifically http://hg.mozilla.org/mozilla-central/rev/56d432ad6ed6. That nicely explains why this doesn't reproduce on release builds until Fx39 but does on Nightly in the Fx37 range.
Assignee | ||
Comment 4•7 years ago
|
||
Uhm, some frames appear to be on two child lists. This is bad obviously, but fortunately it's a bit of a false alarm as it's a temporary situation and nothing bad can happen in this case.
Flags: needinfo?(mats)
Assignee | ||
Comment 5•7 years ago
|
||
The reason the frames appear on two lists is that nsAutoOOFFrameList is still in scope and it is its dtor that removes the OOF list. So we just need to wrap it in a scope to make sure that happens before the VerifyOverflowSituation() call.
Assignee: nobody → mats
Attachment #8687562 -
Flags: review?(roc)
Assignee | ||
Comment 6•7 years ago
|
||
Assignee | ||
Comment 7•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=2327517ca607
Assignee | ||
Updated•7 years ago
|
Severity: critical → normal
Attachment #8687562 -
Flags: review?(roc) → review+
Assignee | ||
Comment 8•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=39aa9258b7bb
https://hg.mozilla.org/integration/mozilla-inbound/rev/1fc0ecca3e9b https://hg.mozilla.org/integration/mozilla-inbound/rev/ceb2a5dce49b
Comment 10•7 years ago
|
||
backoutbugherder |
Backed out for test_value_cloning.html bustage https://hg.mozilla.org/integration/mozilla-inbound/rev/279aaffb417c https://hg.mozilla.org/integration/mozilla-inbound/rev/e1ef40eb7308
Flags: needinfo?(mats)
Comment 11•7 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/1ffc4b1f166c https://hg.mozilla.org/integration/mozilla-inbound/rev/029ab28c46cc
Assignee | ||
Updated•7 years ago
|
Flags: needinfo?(mats) → in-testsuite+
Comment 12•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/1ffc4b1f166c https://hg.mozilla.org/mozilla-central/rev/029ab28c46cc
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox45:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
You need to log in
before you can comment on or make changes to this bug.
Description
•