Remove some unused code in RenderLayerCompositor::computeExtent()
authorsimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 28 Oct 2016 21:02:53 +0000 (21:02 +0000)
committersimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 28 Oct 2016 21:02:53 +0000 (21:02 +0000)
https://bugs.webkit.org/show_bug.cgi?id=163934

Reviewed by Zalan Bujtas.

viewportRect was never used.

* rendering/RenderLayerCompositor.cpp:
(WebCore::RenderLayerCompositor::computeExtent):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@208068 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/rendering/RenderLayerCompositor.cpp

index ca6f781..15c1cff 100644 (file)
@@ -1,3 +1,15 @@
+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>
 
         [WebIDL] Update parser and code generators to only access type information through the type property
index b921d3e..a290abf 100644 (file)
@@ -1196,12 +1196,6 @@ void RenderLayerCompositor::computeExtent(const OverlapMap& overlapMap, const Re
         // Because fixed elements get moved around without re-computing overlap, we have to compute an overlap
         // 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);
     }