Title: [111560] trunk/Source/WebCore
Revision
111560
Author
[email protected]
Date
2012-03-21 10:16:46 -0700 (Wed, 21 Mar 2012)

Log Message

TiledBackingStore: When checking if the visible area is covered, only convert to scaled coordinates once.
https://bugs.webkit.org/show_bug.cgi?id=81519

Reviewed by Kenneth Rohde Christiansen.

Both visibleRect and coverageRatio call mapFromContents on the input rect,
this woulc cause the coverageRatio to never reach 1.0 when the contents is
scaled up.

* platform/graphics/TiledBackingStore.cpp:
(WebCore::TiledBackingStore::visibleAreaIsCovered):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (111559 => 111560)


--- trunk/Source/WebCore/ChangeLog	2012-03-21 17:13:55 UTC (rev 111559)
+++ trunk/Source/WebCore/ChangeLog	2012-03-21 17:16:46 UTC (rev 111560)
@@ -1,3 +1,17 @@
+2012-03-19  Jocelyn Turcotte  <[email protected]>
+
+        TiledBackingStore: When checking if the visible area is covered, only convert to scaled coordinates once.
+        https://bugs.webkit.org/show_bug.cgi?id=81519
+
+        Reviewed by Kenneth Rohde Christiansen.
+
+        Both visibleRect and coverageRatio call mapFromContents on the input rect,
+        this woulc cause the coverageRatio to never reach 1.0 when the contents is
+        scaled up.
+
+        * platform/graphics/TiledBackingStore.cpp:
+        (WebCore::TiledBackingStore::visibleAreaIsCovered):
+
 2012-03-21  Mao Yujie  <[email protected]>
 
         MediaStream API: Need to support MediaStream constructor for JSC

Modified: trunk/Source/WebCore/platform/graphics/TiledBackingStore.cpp (111559 => 111560)


--- trunk/Source/WebCore/platform/graphics/TiledBackingStore.cpp	2012-03-21 17:13:55 UTC (rev 111559)
+++ trunk/Source/WebCore/platform/graphics/TiledBackingStore.cpp	2012-03-21 17:16:46 UTC (rev 111560)
@@ -227,7 +227,7 @@
 
 bool TiledBackingStore::visibleAreaIsCovered() const
 {
-    return coverageRatio(visibleRect()) == 1.0f;
+    return coverageRatio(m_client->tiledBackingStoreVisibleRect()) == 1.0f;
 }
 
 void TiledBackingStore::createTiles()
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to