Changeset 208068 in webkit

Timestamp:

Oct 28, 2016, 2:02:53 PM (8 years ago)

Author:

Simon Fraser

Message:

Remove some unused code in RenderLayerCompositor::computeExtent()
​https://bugs.webkit.org/show_bug.cgi?id=163934

Reviewed by Zalan Bujtas.

viewportRect was never used.

• rendering/RenderLayerCompositor.cpp:

(WebCore::RenderLayerCompositor::computeExtent):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• rendering/RenderLayerCompositor.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2016-10-28  Simon Fraser  <simon.fraser@apple.com>
+
+        Remove some unused code in RenderLayerCompositor::computeExtent()
+        https://bugs.webkit.org/show_bug.cgi?id=163934
+
+        Reviewed by Zalan Bujtas.
+
+        viewportRect was never used.
+
+        * rendering/RenderLayerCompositor.cpp:
+        (WebCore::RenderLayerCompositor::computeExtent):
+
 2016-10-28  Sam Weinig  <sam@webkit.org>
 

trunk/Source/WebCore/rendering/RenderLayerCompositor.cpp

@@ -1197 +1197 @@
         // rect that covers all the locations that the fixed element could move to.
         // FIXME: need to handle sticky too.
-        LayoutRect viewportRect;
-        if (m_renderView.frameView().useFixedLayout())
-            viewportRect = m_renderView.unscaledDocumentRect();
-        else
-            viewportRect = m_renderView.frameView().viewportConstrainedVisibleContentRect();
-
         extent.bounds = m_renderView.frameView().fixedScrollableAreaBoundsInflatedForScrolling(extent.bounds);
     }