Unreviewed, rolling out r243868.
authorsroberts@apple.com <sroberts@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 4 Apr 2019 23:14:50 +0000 (23:14 +0000)
committersroberts@apple.com <sroberts@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 4 Apr 2019 23:14:50 +0000 (23:14 +0000)
Causing timeouts failures on several queues

Reverted changeset:

"[Web Animations] JS wrapper may be deleted while animation is
yet to dispatch its finish event"
https://bugs.webkit.org/show_bug.cgi?id=196118
https://trac.webkit.org/changeset/243868

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@243917 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/legacy-animation-engine/animations/resume-after-page-cache.html
LayoutTests/webanimations/js-wrapper-kept-alive-expected.txt [deleted file]
LayoutTests/webanimations/js-wrapper-kept-alive.html [deleted file]
Source/WebCore/ChangeLog
Source/WebCore/animation/WebAnimation.cpp
Source/WebCore/animation/WebAnimation.h

index 1e54d96..969598d 100644 (file)
@@ -1,3 +1,16 @@
+2019-04-04  Shawn Roberts  <sroberts@apple.com>
+
+        Unreviewed, rolling out r243868.
+
+        Causing timeouts failures on several queues
+
+        Reverted changeset:
+
+        "[Web Animations] JS wrapper may be deleted while animation is
+        yet to dispatch its finish event"
+        https://bugs.webkit.org/show_bug.cgi?id=196118
+        https://trac.webkit.org/changeset/243868
+
 2019-04-04  Commit Queue  <commit-queue@webkit.org>
 
         Unreviewed, rolling out r243807 and r243824.
diff --git a/LayoutTests/webanimations/js-wrapper-kept-alive-expected.txt b/LayoutTests/webanimations/js-wrapper-kept-alive-expected.txt
deleted file mode 100644 (file)
index 0bf6660..0000000
+++ /dev/null
@@ -1,10 +0,0 @@
-This test checks that registering an event listener on an animation whose JS wrapper would otherwise be garbage-collected still fires registered event listeners.
-
-On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
-
-
-PASS event.target._isMyAnimation is true
-PASS successfullyParsed is true
-
-TEST COMPLETE
-
diff --git a/LayoutTests/webanimations/js-wrapper-kept-alive.html b/LayoutTests/webanimations/js-wrapper-kept-alive.html
deleted file mode 100644 (file)
index 9351423..0000000
+++ /dev/null
@@ -1,33 +0,0 @@
-<!DOCTYPE html>
-<html>
-<body>
-<div id="target"></div>
-<script src="../resources/js-test-pre.js"></script>
-<script>
-description("This test checks that registering an event listener on an animation whose JS wrapper would otherwise be garbage-collected still fires registered event listeners.");
-
-if (window.internals)
-    jsTestIsAsync = true;
-
-// A longer animation that could not be garbage-collected under any circumstance allows us to finish the test
-// with a reasonable delay without hard-coding a timeout.
-const timeoutAnimation = document.getElementById("target").animate({ marginRight: ["0px", "100px"] }, 1000);
-timeoutAnimation.addEventListener("finish", finishJSTest);
-
-function runTest() {
-    const animation = document.getElementById("target").animate({ marginLeft: ["0px", "100px"] }, 100);
-    animation._isMyAnimation = true;
-    animation.addEventListener("finish", event => {
-        shouldBeTrue("event.target._isMyAnimation");
-        finishJSTest();
-    });
-}
-
-gc();
-runTest();
-gc();
-
-</script>
-<script src="../resources/js-test-post.js"></script>
-</body>
-</html>
\ No newline at end of file
index fcd5273..e5683b6 100644 (file)
@@ -1,3 +1,16 @@
+2019-04-04  Shawn Roberts  <sroberts@apple.com>
+
+        Unreviewed, rolling out r243868.
+
+        Causing timeouts failures on several queues
+
+        Reverted changeset:
+
+        "[Web Animations] JS wrapper may be deleted while animation is
+        yet to dispatch its finish event"
+        https://bugs.webkit.org/show_bug.cgi?id=196118
+        https://trac.webkit.org/changeset/243868
+
 2019-04-04  Youenn Fablet  <youenn@apple.com>
 
         Pass storage quota parameters from UIProcess to NetworkProcess as part of WebsiteDataStore parameters
index f6cbdcc..d24b5b7 100644 (file)
@@ -1162,16 +1162,10 @@ bool WebAnimation::canSuspendForDocumentSuspension() const
 
 void WebAnimation::stop()
 {
-    ActiveDOMObject::stop();
     m_isStopped = true;
     removeAllEventListeners();
 }
 
-bool WebAnimation::hasPendingActivity() const
-{
-    return m_timeline || m_isRelevant || ActiveDOMObject::hasPendingActivity();
-}
-
 void WebAnimation::updateRelevance()
 {
     m_isRelevant = computeRelevance();
index e4b38b4..c4cf338 100644 (file)
@@ -118,8 +118,6 @@ public:
     bool isSuspended() const { return m_isSuspended; }
     virtual void remove();
 
-    bool hasPendingActivity() const final;
-
     using RefCounted::ref;
     using RefCounted::deref;