Title: [236313] trunk
- Revision
- 236313
- Author
- grao...@webkit.org
- Date
- 2018-09-21 06:14:27 -0700 (Fri, 21 Sep 2018)
Log Message
[Web Animations] Accelerated animations don't get suspended
https://bugs.webkit.org/show_bug.cgi?id=189783
<rdar://problem/44652315>
Unreviewed, correct a merge error in the previous commit.
Source/WebCore:
* animation/DocumentTimeline.cpp:
(WebCore::DocumentTimeline::performInvalidationTask):
LayoutTests:
* animations/suspend-resume-animation-events.html:
Modified Paths
Diff
Modified: trunk/LayoutTests/ChangeLog (236312 => 236313)
--- trunk/LayoutTests/ChangeLog 2018-09-21 13:09:45 UTC (rev 236312)
+++ trunk/LayoutTests/ChangeLog 2018-09-21 13:14:27 UTC (rev 236313)
@@ -2,6 +2,16 @@
[Web Animations] Accelerated animations don't get suspended
https://bugs.webkit.org/show_bug.cgi?id=189783
+ <rdar://problem/44652315>
+
+ Unreviewed, correct a merge error in the previous commit.
+
+ * animations/suspend-resume-animation-events.html:
+
+2018-09-21 Antoine Quint <grao...@apple.com>
+
+ [Web Animations] Accelerated animations don't get suspended
+ https://bugs.webkit.org/show_bug.cgi?id=189783
<rdar://problem/43033568>
Reviewed by Dean Jackson.
Modified: trunk/LayoutTests/animations/suspend-resume-animation-events.html (236312 => 236313)
--- trunk/LayoutTests/animations/suspend-resume-animation-events.html 2018-09-21 13:09:45 UTC (rev 236312)
+++ trunk/LayoutTests/animations/suspend-resume-animation-events.html 2018-09-21 13:14:27 UTC (rev 236313)
@@ -1,4 +1,4 @@
-<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd"><!-- webkit-test-runner [ experimental:WebAnimationsCSSIntegrationEnabled=true ] -->
+<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
<html lang="en">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
Modified: trunk/Source/WebCore/ChangeLog (236312 => 236313)
--- trunk/Source/WebCore/ChangeLog 2018-09-21 13:09:45 UTC (rev 236312)
+++ trunk/Source/WebCore/ChangeLog 2018-09-21 13:14:27 UTC (rev 236313)
@@ -2,6 +2,17 @@
[Web Animations] Accelerated animations don't get suspended
https://bugs.webkit.org/show_bug.cgi?id=189783
+ <rdar://problem/44652315>
+
+ Unreviewed, correct a merge error in the previous commit.
+
+ * animation/DocumentTimeline.cpp:
+ (WebCore::DocumentTimeline::performInvalidationTask):
+
+2018-09-21 Antoine Quint <grao...@apple.com>
+
+ [Web Animations] Accelerated animations don't get suspended
+ https://bugs.webkit.org/show_bug.cgi?id=189783
<rdar://problem/43033568>
Reviewed by Dean Jackson.
Modified: trunk/Source/WebCore/animation/DocumentTimeline.cpp (236312 => 236313)
--- trunk/Source/WebCore/animation/DocumentTimeline.cpp 2018-09-21 13:09:45 UTC (rev 236312)
+++ trunk/Source/WebCore/animation/DocumentTimeline.cpp 2018-09-21 13:14:27 UTC (rev 236313)
@@ -216,11 +216,9 @@
void DocumentTimeline::performInvalidationTask()
{
// Now that the timing model has changed we can see if there are DOM events to dispatch for declarative animations.
- if (!m_isSuspended) {
- for (auto& animation : animations()) {
- if (is<DeclarativeAnimation>(animation))
- downcast<DeclarativeAnimation>(*animation).invalidateDOMEvents();
- }
+ for (auto& animation : animations()) {
+ if (is<DeclarativeAnimation>(animation))
+ downcast<DeclarativeAnimation>(*animation).invalidateDOMEvents();
}
applyPendingAcceleratedAnimations();
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes