Refactor ScrollableArea::setScrollPosition.

third_party/WebKit/Source/core/frame/FrameView.cpp

Index: third_party/WebKit/Source/core/frame/FrameView.cpp
diff --git a/third_party/WebKit/Source/core/frame/FrameView.cpp b/third_party/WebKit/Source/core/frame/FrameView.cpp
index 44f06f71c39688b9e294c312eb2d608559822fb4..db99b13216679a133a8cfd6184aabf405bdf4932 100644
--- a/third_party/WebKit/Source/core/frame/FrameView.cpp
+++ b/third_party/WebKit/Source/core/frame/FrameView.cpp
@@ -3547,11 +3547,9 @@ int FrameView::scrollSize(ScrollbarOrientation orientation) const {
   return scrollbar->totalSize() - scrollbar->visibleSize();
 }
 
-void FrameView::setScrollOffset(const DoublePoint& offset,
-                                ScrollType scrollType) {
-  // TODO(skobes): We shouldn't have to clamp here; instead we should update callers
-  // ScrollableArea::scrollPositionChanged to only pass clamped offsets.
-  DoublePoint newPosition = clampScrollPosition(offset);
+void FrameView::updateScrollPosition(const DoublePoint& position,
+                                     ScrollType scrollType) {
+  DoublePoint newPosition = clampScrollPosition(position);
 
   DoublePoint oldPosition = m_scrollPosition;
   DoubleSize scrollDelta = newPosition - oldPosition;