[BlackBerry] Manual zoom propagates to next page load
authorzhajiang@rim.com <zhajiang@rim.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 15 Jan 2013 19:00:28 +0000 (19:00 +0000)
committerzhajiang@rim.com <zhajiang@rim.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 15 Jan 2013 19:00:28 +0000 (19:00 +0000)
https://bugs.webkit.org/show_bug.cgi?id=106914

Patch by Jacky Jiang <zhajiang@rim.com>.
Reviewed by Yong Li.
Internally reviewed by Eli Fidler and Jakob Petsovits.

PR: 280481
During load committed, the web page scale was reset. However, UI thread
still kept the old scale in UserInterfaceViewportAccessor so that it
blitted the contents at the old scale although WebKit thread could
update the UI thread scale later when finished the load.
We should update the scale of the UI thread right after we reset it in
WebKit thread to keep them in sync.

* Api/WebPage.cpp:
(BlackBerry::WebKit::WebPagePrivate::resetScales):

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

Source/WebKit/blackberry/Api/WebPage.cpp
Source/WebKit/blackberry/ChangeLog

index 49ef2a8..dad09d8 100644 (file)
@@ -1779,6 +1779,7 @@ void WebPagePrivate::resetScales()
     m_initialScale = m_webSettings->initialScale() > 0 ? m_webSettings->initialScale() : -1.0;
     m_minimumScale = -1.0;
     m_maximumScale = -1.0;
+    m_client->scaleChanged();
 
     // We have to let WebCore know about updated framerect now that we've
     // reset our scales. See: RIM Bug #401.
index 9d7cdae..a805a87 100644 (file)
@@ -1,3 +1,22 @@
+2013-01-15  Jacky Jiang  <zhajiang@rim.com>
+
+        [BlackBerry] Manual zoom propagates to next page load
+        https://bugs.webkit.org/show_bug.cgi?id=106914
+
+        Reviewed by Yong Li.
+        Internally reviewed by Eli Fidler and Jakob Petsovits.
+
+        PR: 280481
+        During load committed, the web page scale was reset. However, UI thread
+        still kept the old scale in UserInterfaceViewportAccessor so that it
+        blitted the contents at the old scale although WebKit thread could
+        update the UI thread scale later when finished the load.
+        We should update the scale of the UI thread right after we reset it in
+        WebKit thread to keep them in sync.
+
+        * Api/WebPage.cpp:
+        (BlackBerry::WebKit::WebPagePrivate::resetScales):
+
 2013-01-14  Otto Derek Cheung  <otcheung@rim.com>
 
         [BlackBerry] FCC appears when switching search engine providers