Diff
Modified: trunk/LayoutTests/ChangeLog (243916 => 243917)
--- trunk/LayoutTests/ChangeLog 2019-04-04 22:51:24 UTC (rev 243916)
+++ trunk/LayoutTests/ChangeLog 2019-04-04 23:14:50 UTC (rev 243917)
@@ -1,3 +1,16 @@
+2019-04-04 Shawn Roberts <[email protected]>
+
+ 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 <[email protected]>
Unreviewed, rolling out r243807 and r243824.
Modified: trunk/LayoutTests/legacy-animation-engine/animations/resume-after-page-cache.html (243916 => 243917)
--- trunk/LayoutTests/legacy-animation-engine/animations/resume-after-page-cache.html 2019-04-04 22:51:24 UTC (rev 243916)
+++ trunk/LayoutTests/legacy-animation-engine/animations/resume-after-page-cache.html 2019-04-04 23:14:50 UTC (rev 243917)
@@ -1,4 +1,3 @@
-<!-- webkit-test-runner [ experimental:WebAnimationsCSSIntegrationEnabled=false ] -->
<style>
@-webkit-keyframes bounce {
from {
Deleted: trunk/LayoutTests/webanimations/js-wrapper-kept-alive-expected.txt (243916 => 243917)
--- trunk/LayoutTests/webanimations/js-wrapper-kept-alive-expected.txt 2019-04-04 22:51:24 UTC (rev 243916)
+++ trunk/LayoutTests/webanimations/js-wrapper-kept-alive-expected.txt 2019-04-04 23:14:50 UTC (rev 243917)
@@ -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
-
Deleted: trunk/LayoutTests/webanimations/js-wrapper-kept-alive.html (243916 => 243917)
--- trunk/LayoutTests/webanimations/js-wrapper-kept-alive.html 2019-04-04 22:51:24 UTC (rev 243916)
+++ trunk/LayoutTests/webanimations/js-wrapper-kept-alive.html 2019-04-04 23:14:50 UTC (rev 243917)
@@ -1,33 +0,0 @@
-<!DOCTYPE html>
-<html>
-<body>
-<div id="target"></div>
-<script src=""
-<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=""
-</body>
-</html>
\ No newline at end of file
Modified: trunk/Source/WebCore/ChangeLog (243916 => 243917)
--- trunk/Source/WebCore/ChangeLog 2019-04-04 22:51:24 UTC (rev 243916)
+++ trunk/Source/WebCore/ChangeLog 2019-04-04 23:14:50 UTC (rev 243917)
@@ -1,3 +1,16 @@
+2019-04-04 Shawn Roberts <[email protected]>
+
+ 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 <[email protected]>
Pass storage quota parameters from UIProcess to NetworkProcess as part of WebsiteDataStore parameters
Modified: trunk/Source/WebCore/animation/WebAnimation.cpp (243916 => 243917)
--- trunk/Source/WebCore/animation/WebAnimation.cpp 2019-04-04 22:51:24 UTC (rev 243916)
+++ trunk/Source/WebCore/animation/WebAnimation.cpp 2019-04-04 23:14:50 UTC (rev 243917)
@@ -1162,16 +1162,10 @@
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();
Modified: trunk/Source/WebCore/animation/WebAnimation.h (243916 => 243917)
--- trunk/Source/WebCore/animation/WebAnimation.h 2019-04-04 22:51:24 UTC (rev 243916)
+++ trunk/Source/WebCore/animation/WebAnimation.h 2019-04-04 23:14:50 UTC (rev 243917)
@@ -118,8 +118,6 @@
bool isSuspended() const { return m_isSuspended; }
virtual void remove();
- bool hasPendingActivity() const final;
-
using RefCounted::ref;
using RefCounted::deref;
