Removed redundant rect methods on FrameView

Source/core/layout/LayoutView.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
@@ skipping 272 matching lines @@
 {
     ASSERT_UNUSED(wasFixed, !wasFixed || *wasFixed == static_cast<bool>(mode & IsFixed));
 
     if (!paintInvalidationContainer && mode & UseTransforms && shouldUseTransformFromContainer(0)) {
         TransformationMatrix t;
         getTransformFromContainer(0, LayoutSize(), t);
         transformState.applyTransform(t);
     }
 
     if ((mode & IsFixed) && m_frameView) {
-        transformState.move(m_frameView->scrollOffsetForViewportConstrainedObjects());
+        transformState.move(toIntSize(m_frameView->scrollPosition()));
         if (hasOverflowClip())
             transformState.move(scrolledContentOffset());
         // IsFixed flag is only applicable within this LayoutView.
         mode &= ~IsFixed;
     }
 
     if (paintInvalidationContainer == this)
         return;
 
     if (mode & TraverseDocumentBoundaries) {
         if (LayoutObject* parentDocLayoutObject = frame()->ownerLayoutObject()) {
             transformState.move(-frame()->view()->scrollOffset());
             if (parentDocLayoutObject->isBox())
                 transformState.move(toLayoutBox(parentDocLayoutObject)->contentBoxOffset());
             parentDocLayoutObject->mapLocalToContainer(paintInvalidationContainer, transformState, mode, wasFixed, paintInvalidationState);
             return;
         }
     }
 }
 
 const LayoutObject* LayoutView::pushMappingToContainer(const LayoutBoxModelObject* ancestorToStopAt, LayoutGeometryMap& geometryMap) const
 {
     LayoutSize offsetForFixedPosition;
     LayoutSize offset;
     LayoutObject* container = 0;
 
     if (m_frameView) {
-        offsetForFixedPosition = LayoutSize(m_frameView->scrollOffsetForViewportConstrainedObjects());
+        offsetForFixedPosition = LayoutSize(toIntSize(m_frameView->scrollPosition()));
         if (hasOverflowClip())
             offsetForFixedPosition = LayoutSize(scrolledContentOffset());
     }
 
     if (geometryMap.mapCoordinatesFlags() & TraverseDocumentBoundaries) {
         if (LayoutPart* parentDocLayoutObject = frame()->ownerLayoutObject()) {
             offset = -LayoutSize(m_frameView->scrollOffset());
             offset += parentDocLayoutObject->contentBoxOffset();
             container = parentDocLayoutObject;
         }
@@ skipping 10 matching lines @@
     } else {
         geometryMap.push(this, offset, false, false, false, false, offsetForFixedPosition);
     }
 
     return container;
 }
 
 void LayoutView::mapAbsoluteToLocalPoint(MapCoordinatesFlags mode, TransformState& transformState) const
 {
     if (mode & IsFixed && m_frameView)
-        transformState.move(m_frameView->scrollOffsetForViewportConstrainedObjects());
+        transformState.move(toIntSize(m_frameView->scrollPosition()));
 
     if (mode & UseTransforms && shouldUseTransformFromContainer(0)) {
         TransformationMatrix t;
         getTransformFromContainer(0, LayoutSize(), t);
         transformState.applyTransform(t);
     }
 }
 
 void LayoutView::computeSelfHitTestRects(Vector<LayoutRect>& rects, const LayoutPoint&) const
 {
@@ skipping 103 matching lines @@
         obj->mapRectToPaintInvalidationBacking(paintInvalidationContainer, rect, 0);
     }
 }
 
 void LayoutView::adjustViewportConstrainedOffset(LayoutRect& rect, ViewportConstrainedPosition viewportConstraint) const
 {
     if (viewportConstraint != IsFixedPosition)
         return;
 
     if (m_frameView) {
-        rect.move(m_frameView->scrollOffsetForViewportConstrainedObjects());
+        rect.move(toIntSize(m_frameView->scrollPosition()));
         if (hasOverflowClip())
             rect.move(scrolledContentOffset());
 
         // FIXME: Paint invalidation should happen after scroll updates, so there should be no pending scroll delta.
         // However, we still have paint invalidation during layout, so we can't ASSERT for now. crbug.com/434950.
         // ASSERT(m_frameView->pendingScrollDelta().isZero());
         // If we have a pending scroll, invalidate the previous scroll position.
         if (!m_frameView->pendingScrollDelta().isZero())
             rect.move(-LayoutSize(m_frameView->pendingScrollDelta()));
     }
@@ skipping 505 matching lines @@
     return viewHeight(IncludeScrollbars) / scale;
 }
 
 void LayoutView::willBeDestroyed()
 {
     LayoutBlockFlow::willBeDestroyed();
     m_compositor.clear();
 }
 
 } // namespace blink