Skip to content
Permalink
Browse files

style: Set the important rule change flag when a newly important rule…

… is inserted.

Without this change an assertion checking IsInStyleRefresh() in
EffectCompositor::PostRestyleForAnimation will be hit when we call
FindAnimationsForCompositor from RestyleManager::DoProcessPendingRestyles
that will be introduced in a subsequent commit in this series.

I wrote a crash test which causes an assertion in KeyframeEffect::CanThrottle()
without the subsequent commit, but we need more work in display item stuff to
make the assertion won't happen (bug 1508466).

Differential Revision: https://phabricator.services.mozilla.com/D12368
  • Loading branch information
Hiroyuki Ikezoe authored and emilio committed Dec 2, 2018
1 parent 2ebad8d commit 2639916d7d53204281bd456abc44140de21a7740
Showing with 1 addition and 0 deletions.
  1. +1 −0 components/style/rule_tree/mod.rs
@@ -453,6 +453,7 @@ impl RuleTree {
StyleSource::from_declarations(pdb.clone_arc()),
level,
);
*important_rules_changed = true;
}
} else {
if pdb.read_with(level.guard(guards)).any_normal() {

0 comments on commit 2639916

Please sign in to comment.
You can’t perform that action at this time.