Changeset 218311 in webkit

Timestamp:

Jun 14, 2017 7:36:36 PM (7 years ago)

Author:

Alan Bujtas

Message:

animations-paused-in-background-page.html and animated-svg-image-removed-from-document-paused.html fail after r218284
​https://bugs.webkit.org/show_bug.cgi?id=173393

Reviewed by Simon Fraser.

Testing cares really about whether the animation has initiated.

• platform/graphics/Image.h:

(WebCore::Image::animationPending):

• testing/Internals.cpp:

(WebCore::Internals::isImageAnimating):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/Image.h (modified) (1 diff)
• testing/Internals.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2017-06-14  Zalan Bujtas  <zalan@apple.com>
+
+        animations-paused-in-background-page.html and animated-svg-image-removed-from-document-paused.html fail after r218284
+        https://bugs.webkit.org/show_bug.cgi?id=173393
+
+        Reviewed by Simon Fraser.
+
+        Testing cares really about whether the animation has initiated.  
+
+        * platform/graphics/Image.h:
+        (WebCore::Image::animationPending):
+        * testing/Internals.cpp:
+        (WebCore::Internals::isImageAnimating):
+
 2017-06-14  Dean Jackson  <dino@apple.com>
 

trunk/Source/WebCore/platform/graphics/Image.h

@@ -136 +136 @@
     virtual void imageFrameAvailableAtIndex(size_t) { }
     virtual bool isAnimating() const { return false; }
+    bool animationPending() const { return m_animationStartTimer.isActive(); }
     
     // Typically the CachedImage that owns us.

trunk/Source/WebCore/testing/Internals.cpp

@@ -802 +802 @@
         return false;
 
-    return image->isAnimating();
+    return image->isAnimating() || image->animationPending();
 }