Title: [176384] trunk/Source/WebCore
Revision
176384
Author
[email protected]
Date
2014-11-20 00:23:19 -0800 (Thu, 20 Nov 2014)

Log Message

REGRESSION (r172854): Web Viewer in FileMaker does not render a Base64 encoded animated-GIF
https://bugs.webkit.org/show_bug.cgi?id=138807
<rdar://problem/18829540>

Reviewed by Simon Fraser.

Animation gets paused because WebKit thinks the GIF is outside of the view.

* page/FrameView.cpp:
(WebCore::FrameView::windowClipRect):

We need to convert to window coordinates in paintsEntireContents mode too so these functions are consistent.
This matters with some WK1 API clients.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (176383 => 176384)


--- trunk/Source/WebCore/ChangeLog	2014-11-20 07:42:19 UTC (rev 176383)
+++ trunk/Source/WebCore/ChangeLog	2014-11-20 08:23:19 UTC (rev 176384)
@@ -1,3 +1,19 @@
+2014-11-20  Antti Koivisto  <[email protected]>
+
+        REGRESSION (r172854): Web Viewer in FileMaker does not render a Base64 encoded animated-GIF
+        https://bugs.webkit.org/show_bug.cgi?id=138807
+        <rdar://problem/18829540>
+
+        Reviewed by Simon Fraser.
+
+        Animation gets paused because WebKit thinks the GIF is outside of the view.
+
+        * page/FrameView.cpp:
+        (WebCore::FrameView::windowClipRect):
+
+        We need to convert to window coordinates in paintsEntireContents mode too so these functions are consistent.
+        This matters with some WK1 API clients.
+
 2014-11-19  Chris Dumez  <[email protected]>
 
         Move 'clip' CSS property to the new StyleBuilder

Modified: trunk/Source/WebCore/page/FrameView.cpp (176383 => 176384)


--- trunk/Source/WebCore/page/FrameView.cpp	2014-11-20 07:42:19 UTC (rev 176383)
+++ trunk/Source/WebCore/page/FrameView.cpp	2014-11-20 08:23:19 UTC (rev 176384)
@@ -3201,7 +3201,7 @@
     ASSERT(frame().view() == this);
 
     if (paintsEntireContents())
-        return IntRect(IntPoint(), totalContentsSize());
+        return contentsToWindow(IntRect(IntPoint(), totalContentsSize()));
 
     // Set our clip rect to be our contents.
     IntRect clipRect;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to