Title: [234771] trunk/Source/WebCore
Revision
234771
Author
[email protected]
Date
2018-08-10 14:53:06 -0700 (Fri, 10 Aug 2018)

Log Message

Cleanup: Remove unnecessary code to resume animations from CachedFrameBase::restore()
https://bugs.webkit.org/show_bug.cgi?id=188459

Reviewed by Chris Dumez.

It is unnessary for CachedFrameBase::restore() to explicitly resume animations on the
document as it calls Document::resume(), which already does this.

No functionality changed. So, no new tests.

* history/CachedFrame.cpp:
(WebCore::CachedFrameBase::restore):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (234770 => 234771)


--- trunk/Source/WebCore/ChangeLog	2018-08-10 21:48:41 UTC (rev 234770)
+++ trunk/Source/WebCore/ChangeLog	2018-08-10 21:53:06 UTC (rev 234771)
@@ -1,3 +1,18 @@
+2018-08-10  Daniel Bates  <[email protected]>
+
+        Cleanup: Remove unnecessary code to resume animations from CachedFrameBase::restore()
+        https://bugs.webkit.org/show_bug.cgi?id=188459
+
+        Reviewed by Chris Dumez.
+
+        It is unnessary for CachedFrameBase::restore() to explicitly resume animations on the
+        document as it calls Document::resume(), which already does this.
+
+        No functionality changed. So, no new tests.
+
+        * history/CachedFrame.cpp:
+        (WebCore::CachedFrameBase::restore):
+
 2018-08-10  Antti Koivisto  <[email protected]>
 
         Use OptionSet for various RenderLayer flags

Modified: trunk/Source/WebCore/history/CachedFrame.cpp (234770 => 234771)


--- trunk/Source/WebCore/history/CachedFrame.cpp	2018-08-10 21:48:41 UTC (rev 234770)
+++ trunk/Source/WebCore/history/CachedFrame.cpp	2018-08-10 21:53:06 UTC (rev 234771)
@@ -98,11 +98,6 @@
     if (m_document->svgExtensions())
         m_document->accessSVGExtensions().unpauseAnimations();
 
-    if (RuntimeEnabledFeatures::sharedFeatures().webAnimationsCSSIntegrationEnabled())
-        m_document->timeline().resumeAnimations();
-    else
-        frame.animation().resumeAnimationsForDocument(m_document.get());
-
     m_document->resume(ReasonForSuspension::PageCache);
 
     // It is necessary to update any platform script objects after restoring the
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to