Remove FrameView::viewportConstrainedExtentRect, it is unused
authorbenjamin@webkit.org <benjamin@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 15 May 2014 00:27:17 +0000 (00:27 +0000)
committerbenjamin@webkit.org <benjamin@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 15 May 2014 00:27:17 +0000 (00:27 +0000)
https://bugs.webkit.org/show_bug.cgi?id=132927

Patch by Benjamin Poulain <bpoulain@apple.com> on 2014-05-14
Reviewed by Simon Fraser.

* page/FrameView.cpp:
(WebCore::FrameView::viewportConstrainedExtentRect): Deleted.
* page/FrameView.h:
Simon replaced this by viewportConstrainedVisibleContentRect, remove the old code.

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

Source/WebCore/ChangeLog
Source/WebCore/page/FrameView.cpp
Source/WebCore/page/FrameView.h

index 44ec1d9..35f13ac 100644 (file)
@@ -1,3 +1,15 @@
+2014-05-14  Benjamin Poulain  <bpoulain@apple.com>
+
+        Remove FrameView::viewportConstrainedExtentRect, it is unused
+        https://bugs.webkit.org/show_bug.cgi?id=132927
+
+        Reviewed by Simon Fraser.
+
+        * page/FrameView.cpp:
+        (WebCore::FrameView::viewportConstrainedExtentRect): Deleted.
+        * page/FrameView.h:
+        Simon replaced this by viewportConstrainedVisibleContentRect, remove the old code.
+
 2014-05-14  Matthew Hanson  <matthew_hanson@apple.com>
 
         Only define MAX_GRID_TRACK_REPETITIONS if CSS_GRID_LAYOUT is enabled.
index 9519bcd..9a051b0 100644 (file)
@@ -1561,18 +1561,6 @@ LayoutRect FrameView::viewportConstrainedVisibleContentRect() const
     return viewportRect;
 }
 
-LayoutRect FrameView::viewportConstrainedExtentRect() const
-{
-#if PLATFORM(IOS)
-    if (platformWidget())
-        return visibleContentRect(ScrollableArea::LegacyIOSDocumentVisibleRect);
-    
-    return renderView()->unscaledDocumentRect();
-#else
-    return viewportConstrainedVisibleContentRect();
-#endif
-}
-
 LayoutSize FrameView::scrollOffsetForFixedPosition(const LayoutRect& visibleContentRect, const LayoutSize& totalContentsSize, const LayoutPoint& scrollPosition, const LayoutPoint& scrollOrigin, float frameScaleFactor, bool fixedElementsLayoutRelativeToFrame, ScrollBehaviorForFixedElements behaviorForFixed, int headerHeight, int footerHeight)
 {
     LayoutPoint position;
index 284a1ef..9338b84 100644 (file)
@@ -246,9 +246,6 @@ public:
     // offsets from rubber-banding, and it takes zooming into account. 
     LayoutRect viewportConstrainedVisibleContentRect() const;
 
-    // A rectangle in content coordinates that is used to clip layers for fixed-position objects.
-    LayoutRect viewportConstrainedExtentRect() const;
-
     String mediaType() const;
     void setMediaType(const String&);
     void adjustMediaTypeForPrinting(bool printing);