Title: [256869] branches/safari-609.1.20.3-branch/Source/WebCore
Revision
256869
Author
[email protected]
Date
2020-02-18 15:52:56 -0800 (Tue, 18 Feb 2020)

Log Message

Revert r256696. rdar://problem/59478734

Modified Paths

Diff

Modified: branches/safari-609.1.20.3-branch/Source/WebCore/ChangeLog (256868 => 256869)


--- branches/safari-609.1.20.3-branch/Source/WebCore/ChangeLog	2020-02-18 23:52:52 UTC (rev 256868)
+++ branches/safari-609.1.20.3-branch/Source/WebCore/ChangeLog	2020-02-18 23:52:56 UTC (rev 256869)
@@ -4,39 +4,6 @@
 
 2020-02-14  Russell Epstein  <[email protected]>
 
-        Cherry-pick r256636. rdar://problem/59478734
-
-    [Web Animations] Missing call to DocumentTimeline::resumeAnimations() in Frame::resumeActiveDOMObjectsAndAnimations()
-    https://bugs.webkit.org/show_bug.cgi?id=207784
-    <rdar://problem/59251858>
-    
-    Patch by Antoine Quint <[email protected]> on 2020-02-14
-    Reviewed by Dean Jackson.
-    
-    After auditing the code, there was one call to CSSAnimationController::resumeAnimationsForDocument() that missed a matching DocumentTimeline::resumeAnimations()
-    call should the Web Animations flag be on.
-    
-    * page/Frame.cpp:
-    (WebCore::Frame::resumeActiveDOMObjectsAndAnimations):
-    
-    git-svn-id: https://svn.webkit.org/repository/webkit/trunk@256636 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
-    2020-02-14  Antoine Quint  <[email protected]>
-
-            [Web Animations] Missing call to DocumentTimeline::resumeAnimations() in Frame::resumeActiveDOMObjectsAndAnimations()
-            https://bugs.webkit.org/show_bug.cgi?id=207784
-            <rdar://problem/59251858>
-
-            Reviewed by Dean Jackson.
-
-            After auditing the code, there was one call to CSSAnimationController::resumeAnimationsForDocument() that missed a matching DocumentTimeline::resumeAnimations()
-            call should the Web Animations flag be on.
-
-            * page/Frame.cpp:
-            (WebCore::Frame::resumeActiveDOMObjectsAndAnimations):
-
-2020-02-14  Russell Epstein  <[email protected]>
-
         Cherry-pick r256623. rdar://problem/59478938
 
     Ensure animations that lose their effect don't schedule an animation update

Modified: branches/safari-609.1.20.3-branch/Source/WebCore/page/Frame.cpp (256868 => 256869)


--- branches/safari-609.1.20.3-branch/Source/WebCore/page/Frame.cpp	2020-02-18 23:52:52 UTC (rev 256868)
+++ branches/safari-609.1.20.3-branch/Source/WebCore/page/Frame.cpp	2020-02-18 23:52:56 UTC (rev 256869)
@@ -962,12 +962,7 @@
     m_doc->resumeScheduledTasks(ReasonForSuspension::PageWillBeSuspended);
 
     // Frame::clearTimers() suspended animations and pending relayouts.
-
-    if (RuntimeEnabledFeatures::sharedFeatures().webAnimationsCSSIntegrationEnabled()) {
-        if (auto* timeline = m_doc->existingTimeline())
-            timeline->resumeAnimations();
-    } else
-        animation().resumeAnimationsForDocument(m_doc.get());
+    animation().resumeAnimationsForDocument(m_doc.get());
     if (m_view)
         m_view->layoutContext().scheduleLayout();
 }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to