Bug 1303235 - Part 3: Enable test_restyle.html and remove the early return in CanIgnoreIfNotVisible.
MozReview-Commit-ID: LMKSVW2sh5N
--- a/dom/animation/KeyframeEffectReadOnly.cpp
+++ b/dom/animation/KeyframeEffectReadOnly.cpp
@@ -1809,22 +1809,16 @@ KeyframeEffectReadOnly::SetAnimation(Ani
bool
KeyframeEffectReadOnly::CanIgnoreIfNotVisible() const
{
if (!AnimationUtils::IsOffscreenThrottlingEnabled()) {
return false;
}
- // FIXME (bug 1303235): We don't calculate mCumulativeChangeHint for
- // the Servo backend yet
- if (mDocument->IsStyledByServo()) {
- return false;
- }
-
// FIXME: For further sophisticated optimization we need to check
// change hint on the segment corresponding to computedTiming.progress.
return NS_IsHintSubset(
mCumulativeChangeHint, nsChangeHint_Hints_CanIgnoreIfNotVisible);
}
void
KeyframeEffectReadOnly::MaybeUpdateFrameForCompositor()
--- a/dom/animation/test/chrome/test_restyles.html
+++ b/dom/animation/test/chrome/test_restyles.html
@@ -63,19 +63,17 @@ function ensureElementRemoval(aElement)
SimpleTest.expectAssertions(0, 1); // bug 1332970
SimpleTest.waitForExplicitFinish();
var omtaEnabled = isOMTAEnabled();
var isAndroid = !!navigator.userAgent.includes("Android");
var isServo = isStyledByServo();
var offscreenThrottlingEnabled =
- SpecialPowers.getBoolPref('dom.animations.offscreen-throttling') &&
- !isServo; // Bug 1303235 : For stylo we don't yet accumulate change
- // hints which are used for offscreen throttling.
+ SpecialPowers.getBoolPref('dom.animations.offscreen-throttling');
function add_task_if_omta_enabled(test) {
if (!omtaEnabled) {
info(test.name + " is skipped because OMTA is disabled");
return;
}
add_task(test);
}