Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-06 14:10:11 +01:00
Code Issues Projects Releases Packages Wiki Activity

Don't clear animation only dirty bit during style recalc (unless the element is in a display:none subtree).

Browse source 
After bug 1356141, the setup of animation-only dirty bit should have matched
to normal dirty bit's one (Though they don't match in post traversal due to
throttled animation flush). An unset_animation_only_dirty_descendants call
removed in this patch cleared dirty bits which are needed for post traversal if
there is a second animation-only traversal and if there is no need to restyle
for the second animation-only traversal.

The reftest in this patch fails without either this fix or the fix for bug
1367975.

See [Gecko bug 1384435 comment 12](https://bugzilla.mozilla.org/show_bug.cgi?id=1384435#c12)
for more detail what's going on at that time.
This commit is contained in:
Hiroyuki Ikezoe 2017-08-08 10:01:46 +09:00
parent c9d9ed8e96
commit bdf84510f8
1 changed files with 0 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

3
components/style/traversal.rs
View file

@ -563,9 +563,6 @@ where
} else { } else {
element.has_dirty_descendants() element.has_dirty_descendants()
}; };
if flags.for_animation_only() {
unsafe { element.unset_animation_only_dirty_descendants(); }
}
// Before examining each child individually, try to prove that our children // Before examining each child individually, try to prove that our children
// don't need style processing. They need processing if any of the following // don't need style processing. They need processing if any of the following