Cleanup: Remove unnecessary code to resume animations from CachedFrameBase::restore()
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 10 Aug 2018 21:53:06 +0000 (21:53 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 10 Aug 2018 21:53:06 +0000 (21:53 +0000)
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):

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

Source/WebCore/ChangeLog
Source/WebCore/history/CachedFrame.cpp

index ab31c39..fb3b1d3 100644 (file)
@@ -1,3 +1,18 @@
+2018-08-10  Daniel Bates  <dabates@apple.com>
+
+        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  <antti@apple.com>
 
         Use OptionSet for various RenderLayer flags
index ed53f52..3b34388 100644 (file)
@@ -98,11 +98,6 @@ void CachedFrameBase::restore()
     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