Closed Bug 1424782 Opened 5 years ago Closed 5 years ago
Intermittent GECKO(2118) | Assertion failure: false, at /builds/worker/workspace/build/src/layout/painting/ns
Display List .cpp:4953
Bug 1424782 - Ensure nsDisplayEventReceiver is only created when building the display list for hit-testing.
59 bytes, text/x-review-board-request
Filed by: kgupta [at] mozilla.com https://treeherder.mozilla.org/logviewer.html#?job_id=151190382&repo=try https://queue.taskcluster.net/v1/task/EfZShHepR0uWawbZPVknkA/runs/0/artifacts/public/logs/live_backing.log On a mochitest-plain try run I got this. nsDisplayEventReceiver::CreateWebRenderCommands is getting called, which should never happen.
This can probably be fixed by expanding the guard at  to also check aBuilder->IsForEventDelivery() like all the other places that use nsDisplayEventReceiver.  https://searchfox.org/mozilla-central/rev/281d99b3d342b71d0653b872345761372d38f5c1/layout/xul/nsSplitterFrame.cpp#374
5 years ago
Assignee: nobody → bugmail
Status: NEW → ASSIGNED
Priority: P5 → P1
Comment on attachment 8936374 [details] Bug 1424782 - Ensure nsDisplayEventReceiver is only created when building the display list for hit-testing. https://reviewboard.mozilla.org/r/207098/#review212994
Attachment #8936374 - Flags: review?(matt.woodrow) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/90aad43cbab3 Ensure nsDisplayEventReceiver is only created when building the display list for hit-testing. r=mattwoodrow
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/776901623d60 Follow-up to remove overzealous assertion I added. r=me
You need to log in before you can comment on or make changes to this bug.