Closed Bug 1153716 Opened 9 years ago Closed 7 years ago

"ASSERTION: should not have added restyles" with <audio controls>

Categories

(Core :: Layout, defect)

x86_64
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
firefox-esr52 --- wontfix

People

(Reporter: jruderman, Unassigned)

References

Details

(Keywords: assertion, testcase)

Attachments

(2 files)

Attached file testcase
###!!! ASSERTION: should not have added restyles: 'haveNonAnimation || !mHavePendingNonAnimationRestyles', file layout/base/RestyleManager.cpp, line 1685
Attached file stack
Flags: needinfo?(cam)
INFO: First good revision: 8d2eecb7ea5a16e02862dd326ce4519082ce9901
INFO: Last bad revision: ec7fb4f14d3ec23ded7eea40ff49ebbcbec6bde1
INFO: Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=ec7fb4f14d3ec23ded7eea40ff49ebbcbec6bde1&tochange=8d2eecb7ea5a16e02862dd326ce4519082ce9901

Fixed by bug 1271765.
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: needinfo?(ryanvm)
Flags: needinfo?(cam)
Flags: in-testsuite?
Resolution: --- → FIXED
Well, no longer exercising the bug... but the actual bug is presumably still there, in the underlying Gecko code.

Whether stylo has the same bug is unclear.
Yeah, there was actually another bug I came across recently that was "caused" by that patch too (testcase used audio controls). Regardless, this bug hasn't seen any action in over two years, so I figured that this was likely as fixed as it was ever going to be. If/when we manage to trip the underlying Gecko bug again some day (assuming it's still there with Stylo), we can file a new bug for it then.
Flags: needinfo?(ryanvm)
Flags: in-testsuite?
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: