[Chromium] Rename WebView::scalePage to WebView::setPageScaleFactor to match WebCore
authorfsamuel@chromium.org <fsamuel@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 7 Nov 2011 18:24:48 +0000 (18:24 +0000)
committerfsamuel@chromium.org <fsamuel@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 7 Nov 2011 18:24:48 +0000 (18:24 +0000)
https://bugs.webkit.org/show_bug.cgi?id=71485

Reviewed by Darin Fisher.

Source/WebKit/chromium:

Created new method setPageScaleFactor to replace scalePage to match
renamed method in WebCore. This change will happen in three stages:

1. A new method is created called setPageScaleFactor with the same
   functionality as scalePage.
2. All call sites of scalePage are changed to setPageScaleFactor in the Chromium
   repository.
3. scalePage is deleted from WebView/WebViewImpl

* public/WebView.h:
* src/WebViewImpl.cpp:
(WebKit::WebViewImpl::pageScaleFactor):
(WebKit::WebViewImpl::scalePage):
(WebKit::WebViewImpl::setPageScaleFactor):
* src/WebViewImpl.h:

Tools:

* DumpRenderTree/chromium/EventSender.cpp:
(EventSender::scalePageBy):
* DumpRenderTree/chromium/TestShell.cpp:
(TestShell::resetTestController):

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

Source/WebKit/chromium/ChangeLog
Source/WebKit/chromium/public/WebView.h
Source/WebKit/chromium/src/WebViewImpl.cpp
Source/WebKit/chromium/src/WebViewImpl.h
Tools/ChangeLog
Tools/DumpRenderTree/chromium/EventSender.cpp
Tools/DumpRenderTree/chromium/TestShell.cpp

index cf0c0cdbda98caf8880a18dfd203e5f5b060eb10..0c0bb6fe546ae10d70c4a046f2e8070e749c5cc2 100644 (file)
@@ -1,3 +1,26 @@
+2011-11-07  Fady Samuel  <fsamuel@chromium.org>
+
+        [Chromium] Rename WebView::scalePage to WebView::setPageScaleFactor to match WebCore
+        https://bugs.webkit.org/show_bug.cgi?id=71485
+
+        Reviewed by Darin Fisher.
+
+        Created new method setPageScaleFactor to replace scalePage to match
+        renamed method in WebCore. This change will happen in three stages:
+
+        1. A new method is created called setPageScaleFactor with the same
+           functionality as scalePage.
+        2. All call sites of scalePage are changed to setPageScaleFactor in the Chromium
+           repository.
+        3. scalePage is deleted from WebView/WebViewImpl
+
+        * public/WebView.h:
+        * src/WebViewImpl.cpp:
+        (WebKit::WebViewImpl::pageScaleFactor):
+        (WebKit::WebViewImpl::scalePage):
+        (WebKit::WebViewImpl::setPageScaleFactor):
+        * src/WebViewImpl.h:
+
 2011-11-07  Jessie Berlin  <jberlin@apple.com>
 
         Need a way to allow a scheme access to Local Storage and Databases while Private Browsing is
index 1cc4fd1f2f39568ebe8c3abaeedfb359a8a9bc23..c77f452d0f6c2d0cdf494d4538e8508c70649dc4 100644 (file)
@@ -211,10 +211,13 @@ public:
     // is scaled up, < 1.0 is scaled down.
     virtual float pageScaleFactor() const = 0;
 
+    // FIXME: Delete this once Chromium side has been updated.
+    virtual void scalePage(float scaleFactor, const WebPoint& origin) = 0;
+
     // Scales a page by a factor of scaleFactor and then sets a scroll position to (x, y).
-    // scalePage() magnifies and shrinks a page without affecting layout.
+    // setPageScaleFactor() magnifies and shrinks a page without affecting layout.
     // On the other hand, zooming affects layout of the page.
-    virtual void scalePage(float scaleFactor, WebPoint origin) = 0;
+    virtual void setPageScaleFactor(float scaleFactor, const WebPoint& origin) = 0;
 
     // The ratio of the current device's screen DPI to the target device's screen DPI.
     virtual float deviceScaleFactor() const = 0;
index 3e53ba9d534cd8975cf8aa6bc01f94574f6ab9cd..6f4d0634f2fb2b8275cb5071a38fb1542000b6e1 100644 (file)
@@ -1841,12 +1841,17 @@ double WebView::zoomFactorToZoomLevel(double factor)
 float WebViewImpl::pageScaleFactor() const
 {
     if (!page())
-        return 1.0;
+        return 1;
 
     return page()->pageScaleFactor();
 }
 
-void WebViewImpl::scalePage(float scaleFactor, WebPoint origin)
+void WebViewImpl::scalePage(float scaleFactor, const WebPoint& origin)
+{
+    setPageScaleFactor(scaleFactor, origin);
+}
+
+void WebViewImpl::setPageScaleFactor(float scaleFactor, const WebPoint& origin)
 {
     if (!page())
         return;
index de9a79f798deabb1a58bf8bd74f22651d5fde6bf..4d12b29433b2419fe69c52683204246ae61574d1 100644 (file)
@@ -158,7 +158,8 @@ public:
     virtual void zoomLimitsChanged(double minimumZoomLevel,
                                    double maximumZoomLevel);
     virtual float pageScaleFactor() const;
-    virtual void scalePage(float scaleFactor, WebPoint origin);
+    virtual void scalePage(float scaleFactor, const WebPoint& origin);
+    virtual void setPageScaleFactor(float scaleFactor, const WebPoint& origin);
     virtual float deviceScaleFactor() const;
     virtual void setDeviceScaleFactor(float);
     virtual bool isFixedLayoutModeEnabled() const;
index 3ef9d65607541fa64df2ac4bc4442562c8b5c3b5..8fe37222c3e09bc8277179aa5ae4758e7342496e 100644 (file)
@@ -1,3 +1,15 @@
+2011-11-07  Fady Samuel  <fsamuel@chromium.org>
+
+        [Chromium] Rename WebView::scalePage to WebView::setPageScaleFactor to match WebCore
+        https://bugs.webkit.org/show_bug.cgi?id=71485
+
+        Reviewed by Darin Fisher.
+
+        * DumpRenderTree/chromium/EventSender.cpp:
+        (EventSender::scalePageBy):
+        * DumpRenderTree/chromium/TestShell.cpp:
+        (TestShell::resetTestController):
+
 2011-11-07  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
 
         [Qt] Put the jsc binary in 'bin' instead of leaving it deep in the build tree
index c1b8c16921e355567fc7b657c8922a518a4a52d0..20d77a75a35a92fd785db9b86ebe86014e70713c 100644 (file)
@@ -716,7 +716,7 @@ void EventSender::scalePageBy(const CppArgumentList& arguments, CppVariant* resu
     float scaleFactor = static_cast<float>(arguments[0].toDouble());
     int x = arguments[1].toInt32();
     int y = arguments[2].toInt32();
-    webview()->scalePage(scaleFactor, WebPoint(x, y));
+    webview()->setPageScaleFactor(scaleFactor, WebPoint(x, y));
     result->setNull();
 }
 
index 2275fc505b2fcf5f2806c63d02d84345188cdfa2..de2f2c8a24db59863f62afa6a1095c02b1b60825 100644 (file)
@@ -270,7 +270,7 @@ void TestShell::resetTestController()
     m_drtDevToolsAgent->reset();
     if (m_drtDevToolsClient)
         m_drtDevToolsClient->reset();
-    webView()->scalePage(1, WebPoint(0, 0));
+    webView()->setPageScaleFactor(1, WebPoint(0, 0));
     webView()->enableFixedLayoutMode(false);
     webView()->setFixedLayoutSize(WebSize(0, 0));
     webView()->mainFrame()->clearOpener();