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

Status: RESOLVED FIXED

(Core :: Web Painting, defect)

Description

[Mats Palmgren (inactive)]

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).

Comment 1

[Mats Palmgren (inactive)]

Attachment: Downgrade a non-essential assertion to a warning for now because it keeps failing.

Comment 2

[Matt Woodrow (:mattwoodrow)]

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.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
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

Comment 4

[Mats Palmgren (inactive)]

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.

Comment 5

[Eliza Balazs [:ebalazs_]]

https://hg.mozilla.org/mozilla-central/rev/c8375fd14860

Comment 6

[Mats Palmgren (inactive)]

(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.