Changeset 123921 in webkit

Timestamp:

Jul 27, 2012 3:04:15 PM (12 years ago)

Author:

commit-queue@webkit.org

Message:

[chromium] CCLayerTreeHost placeholder scissor should be deviceViewportSize instead of viewportSize
​https://bugs.webkit.org/show_bug.cgi?id=92469

Patch by Shawn Singh <​shawnsingh@chromium.org> on 2012-07-27
Reviewed by Adrienne Walker.

We were accidentally using viewportSize instead of
deviceViewportSize in a particular place in code. Recent
refactoring to use tighter scissor rects exposed this problem.

• platform/graphics/chromium/cc/CCLayerTreeHost.cpp:

(WebCore::CCLayerTreeHost::updateLayers):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/chromium/cc/CCLayerTreeHost.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-07-27  Shawn Singh  <shawnsingh@chromium.org>
+
+        [chromium] CCLayerTreeHost placeholder scissor should be deviceViewportSize instead of viewportSize
+        https://bugs.webkit.org/show_bug.cgi?id=92469
+
+        Reviewed by Adrienne Walker.
+
+        We were accidentally using viewportSize instead of
+        deviceViewportSize in a particular place in code. Recent
+        refactoring to use tighter scissor rects exposed this problem.
+
+        * platform/graphics/chromium/cc/CCLayerTreeHost.cpp:
+        (WebCore::CCLayerTreeHost::updateLayers):
+
 2012-07-27  Kevin Ellis  <kevers@chromium.org>
 

trunk/Source/WebCore/platform/graphics/chromium/cc/CCLayerTreeHost.cpp

@@ -474 +474 @@
         CCLayerTreeHostCommon::calculateDrawTransforms(rootLayer, deviceViewportSize(), m_deviceScaleFactor, layerRendererCapabilities().maxTextureSize, updateList);
 
-        FloatRect rootScissorRect(FloatPoint(0, 0), viewportSize());
+        FloatRect rootScissorRect(FloatPoint(0, 0), deviceViewportSize());
         CCLayerTreeHostCommon::calculateVisibleAndScissorRects(updateList, rootScissorRect);
     }