Title: [208068] trunk/Source/WebCore
Revision
208068
Author
[email protected]
Date
2016-10-28 14:02:53 -0700 (Fri, 28 Oct 2016)

Log 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):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (208067 => 208068)


--- trunk/Source/WebCore/ChangeLog	2016-10-28 20:54:21 UTC (rev 208067)
+++ trunk/Source/WebCore/ChangeLog	2016-10-28 21:02:53 UTC (rev 208068)
@@ -1,3 +1,15 @@
+2016-10-28  Simon Fraser  <[email protected]>
+
+        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  <[email protected]>
 
         [WebIDL] Update parser and code generators to only access type information through the type property

Modified: trunk/Source/WebCore/rendering/RenderLayerCompositor.cpp (208067 => 208068)


--- trunk/Source/WebCore/rendering/RenderLayerCompositor.cpp	2016-10-28 20:54:21 UTC (rev 208067)
+++ trunk/Source/WebCore/rendering/RenderLayerCompositor.cpp	2016-10-28 21:02:53 UTC (rev 208068)
@@ -1196,12 +1196,6 @@
         // 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);
     }
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to