Title: [111566] trunk/Source/WebKit2
Revision
111566
Author
[email protected]
Date
2012-03-21 10:23:30 -0700 (Wed, 21 Mar 2012)

Log Message

[Qt] Don't clip the contents rendering to the contents size.
https://bugs.webkit.org/show_bug.cgi?id=81770

Reviewed by Kenneth Rohde Christiansen.

The original bug was caused by the contents size updates
having to go through the UI process before being applied
on the TiledBackingStore of the non composited contents layer.
With this bug being fixed, the clipping isn't necessary anymore.

* UIProcess/qt/LayerBackingStore.cpp:
(WebKit::LayerBackingStore::paintToTextureMapper):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (111565 => 111566)


--- trunk/Source/WebKit2/ChangeLog	2012-03-21 17:22:04 UTC (rev 111565)
+++ trunk/Source/WebKit2/ChangeLog	2012-03-21 17:23:30 UTC (rev 111566)
@@ -1,5 +1,20 @@
 2012-03-21  Jocelyn Turcotte  <[email protected]>
 
+        [Qt] Don't clip the contents rendering to the contents size.
+        https://bugs.webkit.org/show_bug.cgi?id=81770
+
+        Reviewed by Kenneth Rohde Christiansen.
+
+        The original bug was caused by the contents size updates
+        having to go through the UI process before being applied
+        on the TiledBackingStore of the non composited contents layer.
+        With this bug being fixed, the clipping isn't necessary anymore.
+
+        * UIProcess/qt/LayerBackingStore.cpp:
+        (WebKit::LayerBackingStore::paintToTextureMapper):
+
+2012-03-21  Jocelyn Turcotte  <[email protected]>
+
         [Qt] Apply tile removals at the same time as update buffer swaps.
         https://bugs.webkit.org/show_bug.cgi?id=81768
 

Modified: trunk/Source/WebKit2/UIProcess/qt/LayerBackingStore.cpp (111565 => 111566)


--- trunk/Source/WebKit2/UIProcess/qt/LayerBackingStore.cpp	2012-03-21 17:22:04 UTC (rev 111565)
+++ trunk/Source/WebKit2/UIProcess/qt/LayerBackingStore.cpp	2012-03-21 17:23:30 UTC (rev 111566)
@@ -119,19 +119,10 @@
             continue;
 
         tilesToPaint.prepend(&tile);
-        coveredRect.unite(tile.rect());
     }
 
-    bool shouldClip = !targetRect.contains(coveredRect);
-
-    if (shouldClip)
-        textureMapper->beginClip(transform, targetRect);
-
     for (size_t i = 0; i < tilesToPaint.size(); ++i)
         tilesToPaint[i]->paint(textureMapper, transform, opacity, mask);
-
-    if (shouldClip)
-        textureMapper->endClip();
 }
 
 void LayerBackingStore::commitTileOperations(TextureMapper* textureMapper)
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to