Closed Bug 1343240 Opened 3 years ago Closed 2 years ago

Assertion failure: !HaveChildren() || (mDependentBits & 0x01000000) (SetIsAnimationRule must only set the IS_ANIMATION_RULE bit before the rule node has children)

Categories

(Core :: DOM: Animation, defect, P3)

defect

Tracking

()

RESOLVED WONTFIX
Tracking Status
firefox-esr52 --- unaffected
firefox54 --- wontfix
firefox56 --- wontfix
firefox57 --- fixed
firefox58 --- fixed

People

(Reporter: truber, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: assertion, testcase, Whiteboard: [fixed by stylo])

Attachments

(1 file)

Attached file testcase.html
The attached testcase causes an assertion failure in mozilla-central rev 5d1ebf7782df.

Assertion failure: !HaveChildren() || (mDependentBits & 0x01000000) (SetIsAnimationRule must only set the IS_ANIMATION_RULE bit before the rule node has children), at /home/worker/workspace/build/src/layout/style/nsRuleNode.h:840
#01: nsStyleSet::RuleNodeWithReplacement at layout/style/nsStyleSet.cpp:1599
#02: nsStyleSet::ResolveStyleWithReplacement at mfbt/RefPtr.h:283
#03: mozilla::ElementRestyler::RestyleSelf at layout/base/GeckoRestyleManager.cpp:2495
#04: mozilla::ElementRestyler::Restyle at layout/base/GeckoRestyleManager.cpp:1809
#05: mozilla::ElementRestyler::ComputeStyleChangeFor at layout/base/GeckoRestyleManager.cpp:3073
#06: mozilla::GeckoRestyleManager::ComputeAndProcessStyleChange at layout/base/GeckoRestyleManager.cpp:3486
#07: mozilla::GeckoRestyleManager::RestyleElement at layout/base/GeckoRestyleManager.cpp:153
#08: mozilla::RestyleTracker::ProcessOneRestyle at layout/base/RestyleTracker.cpp:96
#09: mozilla::RestyleTracker::DoProcessRestyles at layout/base/RestyleTracker.cpp:264
#10: mozilla::GeckoRestyleManager::ProcessPendingRestyles at layout/base/GeckoRestyleManager.cpp:507
#11: mozilla::GeckoRestyleManager::RebuildAllStyleData at layout/base/GeckoRestyleManager.cpp:375
#12: CharSetChangingRunnable::Run at layout/base/nsPresContext.cpp:133
#13: nsThread::ProcessNextEvent at mfbt/Maybe.h:445
Flags: in-testsuite?
Priority: -- → P3
I can't reproduce this on trunk (or on builds from around the time it got filed for that matter). Can you still reproduce, Jesse?
Flags: needinfo?(jschwartzentruber)
I guess that's because stylo is enabled by default, this assertion happens only if stylo is disabled, IIUC.
Heh, I mid-aired on you about to say the same thing :)

I was able to reproduce when running the testcase locally after all, but as you said, only with Stylo disabled. Given the high likelihood of that shipping in 57 and the unlikelihood of anybody fixing this bug for the non-Stylo case, should we just close this out?
(In reply to Ryan VanderMeulen [:RyanVM] from comment #3)
> Heh, I mid-aired on you about to say the same thing :)
> 
> I was able to reproduce when running the testcase locally after all, but as
> you said, only with Stylo disabled. Given the high likelihood of that
> shipping in 57 and the unlikelihood of anybody fixing this bug for the
> non-Stylo case, should we just close this out?

Yeah, we (I) have no time to fix this in 57, we can also close this once stylo ships on Android too (if this bug still persists until that time. :-) ).
Thanks for figuring this out. Disabling stylo would have been my guess too.
Flags: needinfo?(jschwartzentruber)
Has Regression Range: --- → no
Whiteboard: [fixed by stylo]
We no longer have the old style system.
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.