Closed Bug 1456166 Opened 6 years ago Closed 6 years ago

###!!! ASSERTION: bad aListVisibleBounds: 'r.GetBounds().IsEqualInterior(aListVisibleBounds)', file /builds/worker/workspace/build/src/layout/painting/nsDisplayList.cpp, line 2405

Categories

(Core :: Web Painting, defect)

defect
Not set
blocker

Tracking

()

RESOLVED FIXED
mozilla61
Tracking Status
firefox61 --- fixed

People

(Reporter: MatsPalmgren_bugz, Assigned: MatsPalmgren_bugz)

References

(Blocks 2 open bugs)

Details

Attachments

(1 file)

With the patches in bug 1398482 the devtools test devtools/client/inspector/rules/test/browser_rules_gridline-names-autocomplete.js is hitting this assertion.  Here's the log with a stack trace:
https://treeherder.mozilla.org/logviewer.html#?job_id=175130132&repo=try&lineNumber=4116

This prevents me from landing bug 1398482, which is needed for other stuff (e.g. subgrid).
Assignee: nobody → mats
Attachment #8970240 - Flags: review?(matt.woodrow)
Blocks: 765972, 838068
Comment on attachment 8970240 [details] [diff] [review]
Downgrade a non-essential assertion to a warning for now because it keeps failing.

Review of attachment 8970240 [details] [diff] [review]:
-----------------------------------------------------------------

Do you have any idea why we're hitting this assertion? It seems like a real concern, we should try fix the underlying cause.
Attachment #8970240 - Flags: review?(matt.woodrow) → review+
Pushed by mpalmgren@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c8375fd14860
Downgrade a non-essential assertion to a warning for now because it keeps failing.  r=mattwoodrow
I just added a CSS property named "gap", which made the auto-completion popup menu
in devtools for "g" slightly longer.  I have no clue why that would trigger this
assertion.
https://hg.mozilla.org/mozilla-central/rev/c8375fd14860
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
(In reply to Matt Woodrow (:mattwoodrow) from comment #2)
> Do you have any idea why we're hitting this assertion? It seems like a real
> concern, we should try fix the underlying cause.

FYI, it should be really easy to reproduce this assertion.
It occurred on all platforms on Try, IIRC.  It certainly
was 100% reproducible in a Linux debug build.
You need to log in before you can comment on or make changes to this bug.