Hook up window.scrollTo and friends.

Source/core/frame/PinchViewport.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 110 matching lines @@
     scaledSize.expand(0, m_topControlsAdjustment);
     scaledSize.scale(1 / m_scale);
     return FloatRect(m_offset, scaledSize);
 }
 
 FloatRect PinchViewport::visibleRectInDocument() const
 {
     if (!mainFrame() || !mainFrame()->view())
         return FloatRect();
 
-    FloatRect viewRect = mainFrame()->view()->visibleContentRect();
+    FloatRect viewRect = mainFrame()->view()->scrollableArea()->visibleContentRect();
     FloatRect pinchRect = visibleRect();
     pinchRect.moveBy(viewRect.location());
     return pinchRect;
 }
 
 FloatRect PinchViewport::mainViewToViewportCSSPixels(const FloatRect& rect) const
 {
     // Note, this is in CSS Pixels so we don't apply scale.
     FloatRect rectInViewport = rect;
     rectInViewport.moveBy(-location());
@@ skipping 10 matching lines @@
     // Snap the visible rect to layout units to match the input rect.
     FloatRect visible = LayoutRect(visibleRect());
 
     float centeringOffsetX = (visible.width() - rect.width()) / 2;
     float centeringOffsetY = (visible.height() - rect.height()) / 2;
 
     DoublePoint targetOffset(
         rect.x() - centeringOffsetX - visible.x(),
         rect.y() - centeringOffsetY - visible.y());
 
     view->setScrollPosition(targetOffset);

[ajuma, 2015/01/31 00:53:17]

Does this (and the other places in this file where we access or set the
FrameView's scroll position) also need to be changed to go through the
scrollableArea() accessor? Or are these left to be fixed later since they aren't
called during scrollTo?

[skobes, 2015/01/31 00:59:13]

Yes, everything that touches the FrameView's scroll position will eventually
need to be updated.  I'm trying to keep the changes small though.

 
     DoublePoint remainder = DoublePoint(targetOffset - view->scrollPositionDouble());
     move(toFloatPoint(remainder));
 }
 
 void PinchViewport::setLocation(const FloatPoint& newLocation)
 {
     setScaleAndLocation(m_scale, newLocation);
 }
 
@@ skipping 374 matching lines @@
     } else if (graphicsLayer == m_rootTransformLayer) {
         name =  "Root Transform Layer";
     } else {
         ASSERT_NOT_REACHED();
     }
 
     return name;
 }
 
 } // namespace blink