Title: [177107] trunk/Source/WebCore
Revision
177107
Author
akl...@apple.com
Date
2014-12-10 15:19:41 -0800 (Wed, 10 Dec 2014)

Log Message

Speculative fix for assertion "frame().view() == this"
<rdar://problem/18419449>

Rubber-stamped by Anders Carlsson.

I have not reproduced this issue, but we can safely ignore animation
notifications for cached pages.

* rendering/RenderElement.cpp:
(WebCore::RenderElement::newImageAnimationFrameAvailable):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (177106 => 177107)


--- trunk/Source/WebCore/ChangeLog	2014-12-10 23:16:22 UTC (rev 177106)
+++ trunk/Source/WebCore/ChangeLog	2014-12-10 23:19:41 UTC (rev 177107)
@@ -1,3 +1,16 @@
+2014-12-10  Andreas Kling  <akl...@apple.com>
+
+        Speculative fix for assertion "frame().view() == this"
+        <rdar://problem/18419449>
+
+        Rubber-stamped by Anders Carlsson.
+
+        I have not reproduced this issue, but we can safely ignore animation
+        notifications for cached pages.
+
+        * rendering/RenderElement.cpp:
+        (WebCore::RenderElement::newImageAnimationFrameAvailable):
+
 2014-12-10  Anders Carlsson  <ander...@apple.com>
 
         Add session storage handling to StorageNamespaceProvider

Modified: trunk/Source/WebCore/rendering/RenderElement.cpp (177106 => 177107)


--- trunk/Source/WebCore/rendering/RenderElement.cpp	2014-12-10 23:16:22 UTC (rev 177106)
+++ trunk/Source/WebCore/rendering/RenderElement.cpp	2014-12-10 23:19:41 UTC (rev 177107)
@@ -1362,6 +1362,8 @@
 
 void RenderElement::newImageAnimationFrameAvailable(CachedImage& image)
 {
+    if (document().inPageCache())
+        return;
     auto& frameView = view().frameView();
     auto visibleRect = frameView.windowToContents(frameView.windowClipRect());
     if (!shouldRepaintForImageAnimation(*this, visibleRect)) {
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to