REGRESSION (r164702): Double tap doesn't stay under the new element once the animatio...
authortimothy_horton@apple.com <timothy_horton@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 27 Apr 2014 22:47:54 +0000 (22:47 +0000)
committertimothy_horton@apple.com <timothy_horton@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 27 Apr 2014 22:47:54 +0000 (22:47 +0000)
https://bugs.webkit.org/show_bug.cgi?id=132239
<rdar://problem/16192842>

Reviewed by Sam Weinig.

* WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::scalePage):
The early-return added to WebPage::scalePage breaks callers who depend
on being able to call scalePage() with the same scale but a different
origin and having that change take effect.

Page::setPageScaleFactor already has the requisite logic, so move
the early return down after that call, and guard only notification
of page scale changes.

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

Source/WebKit2/ChangeLog
Source/WebKit2/WebProcess/WebPage/WebPage.cpp

index 3d69821..40145eb 100644 (file)
@@ -1,3 +1,21 @@
+2014-04-27  Tim Horton  <timothy_horton@apple.com>
+
+        REGRESSION (r164702): Double tap doesn't stay under the new element once the animation finishes
+        https://bugs.webkit.org/show_bug.cgi?id=132239
+        <rdar://problem/16192842>
+
+        Reviewed by Sam Weinig.
+
+        * WebProcess/WebPage/WebPage.cpp:
+        (WebKit::WebPage::scalePage):
+        The early-return added to WebPage::scalePage breaks callers who depend
+        on being able to call scalePage() with the same scale but a different
+        origin and having that change take effect.
+
+        Page::setPageScaleFactor already has the requisite logic, so move
+        the early return down after that call, and guard only notification
+        of page scale changes.
+
 2014-04-25  Andy Estes  <aestes@apple.com>
 
         [iOS] Stop creating a WKWebResourceQuickLookDelegate for every WebResourceLoader
index 3c5f7aa..de42198 100644 (file)
@@ -1291,9 +1291,6 @@ void WebPage::windowScreenDidChange(uint64_t displayID)
 
 void WebPage::scalePage(double scale, const IntPoint& origin)
 {
-    if (scale == pageScaleFactor())
-        return;
-
 #if PLATFORM(IOS)
     if (!m_inDynamicSizeUpdate)
         m_dynamicSizeUpdateHistory.clear();
@@ -1305,8 +1302,14 @@ void WebPage::scalePage(double scale, const IntPoint& origin)
         return;
     }
 
+    float oldPageScaleFactor = pageScaleFactor();
+
     m_page->setPageScaleFactor(scale, origin);
 
+    // We can't early return before setPageScaleFactor because the origin might be different.
+    if (scale == oldPageScaleFactor)
+        return;
+
     for (auto* pluginView : m_pluginViews)
         pluginView->pageScaleFactorDidChange();