Bug 1622184 - Mark Animation.finished and Animation.ready as handled …

…when they are rejected; r=boris As per spec change: w3c/csswg-drafts@7bcad32 Differential Revision: https://phabricator.services.mozilla.com/D67104 --HG-- extra : moz-landing-system : lando
dothq · Mar 18, 2020 · 5903ac3 · 5903ac3
1 parent e14c3f5
commit 5903ac3
Show file tree

Hide file tree

Showing 5 changed files with 2 additions and 16 deletions.
diff --git a/dom/animation/Animation.cpp b/dom/animation/Animation.cpp
@@ -506,6 +506,7 @@ void Animation::Cancel(PostRestyleMode aPostRestyle) {
 
     if (mFinished) {
       mFinished->MaybeReject(NS_ERROR_DOM_ABORT_ERR);
+      mFinished->SetSettledPromiseIsHandled();
     }
     ResetFinishedPromise();
 
@@ -1617,6 +1618,7 @@ void Animation::ResetPendingTasks() {
 
   if (mReady) {
     mReady->MaybeReject(NS_ERROR_DOM_ABORT_ERR);
+    mReady->SetSettledPromiseIsHandled();
     mReady = nullptr;
   }
 }

diff --git a/testing/web-platform/meta/css/css-animations/CSSAnimation-ready.tentative.html.ini b/testing/web-platform/meta/css/css-animations/CSSAnimation-ready.tentative.html.ini
diff --git a/testing/web-platform/meta/web-animations/interfaces/Animation/finished.html.ini b/testing/web-platform/meta/web-animations/interfaces/Animation/finished.html.ini
diff --git a/testing/web-platform/meta/web-animations/interfaces/Animation/ready.html.ini b/testing/web-platform/meta/web-animations/interfaces/Animation/ready.html.ini
diff --git a/...ng/web-platform/meta/web-animations/timing-model/animations/pausing-an-animation.html.ini b/...ng/web-platform/meta/web-animations/timing-model/animations/pausing-an-animation.html.ini