Restore scale even if scroll is not restored

Source/core/page/Page.cpp

Index: Source/core/page/Page.cpp
diff --git a/Source/core/page/Page.cpp b/Source/core/page/Page.cpp
index 7e853b78301b4311241797a0ac3a484bc4881356..4bd0a6802466e444b29da60d01b09e339281c1ea 100644
--- a/Source/core/page/Page.cpp
+++ b/Source/core/page/Page.cpp
@@ -304,24 +304,13 @@ void Page::setDefersLoading(bool defers)
     }
 }
 
-void Page::setPageScaleFactor(float scale, const IntPoint& origin)
+void Page::setPageScaleFactor(float scale)
 {
     if (!mainFrame()->isLocalFrame())

[Nate Chapin, 2015/08/06 21:02:00]

This early exit looks like it's no longer necessary?

[majidvp, 2015/08/06 21:25:36]

Acknowledged.

         return;
 
-    FrameView* view = deprecatedLocalMainFrame()->view();
     VisualViewport& viewport = frameHost().visualViewport();
-
-    if (scale != viewport.scale()) {
-        viewport.setScale(scale);
-
-        chromeClient().pageScaleFactorChanged();
-
-        deprecatedLocalMainFrame()->loader().saveScrollState();
-    }
-
-    if (view && view->scrollPosition() != origin)
-        view->setScrollPosition(origin, ProgrammaticScroll);
+    viewport.setScale(scale);

[Nate Chapin, 2015/08/06 21:02:00]

I think this function is now only called by Internals.cpp. Can we remove it and
call setScale() directly there?

[majidvp, 2015/08/06 21:25:36]

That is true. The reasons I like to keep it are:
1- symmetry with getter pageScaleFactor()
2- hide visual viewport which is a page impl details from Internal.cpp

[bokan, 2015/08/07 15:08:01]

I agree with Nate, these methods should be removed. Though not necessary for
this patch.

 }
 
 float Page::pageScaleFactor() const