[chromium] Improvements for page scale delta during commit
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 16 Nov 2011 22:37:55 +0000 (22:37 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 16 Nov 2011 22:37:55 +0000 (22:37 +0000)
https://bugs.webkit.org/show_bug.cgi?id=72471

Patch by Alexandre Elias <aelias@google.com> on 2011-11-16
Reviewed by James Robinson.

Page scale now follows the same commit flow as scroll position:
the delta is folded into m_pageScale at BFAC time, and a "sent" value
is preserved for temporary use until the commit finishes.

I also merged setPageScaleFactor and setPageScaleFactorLimits into one
function on the impl side.  The reason is that setPageFactor must
be applied after the limits are updated, but on the other hand setting
the limits first may cause an unnecessary clamp of the scale delta.
Merging the methods avoids this bind.

No new tests. (planning to add later: https://bugs.webkit.org/show_bug.cgi?id=71529)

Source/WebCore:

* platform/graphics/chromium/cc/CCLayerTreeHost.cpp:
(WebCore::CCLayerTreeHost::finishCommitOnImplThread):
(WebCore::CCLayerTreeHost::applyScrollAndScale):
* platform/graphics/chromium/cc/CCLayerTreeHostCommon.h:
* platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:
(WebCore::CCLayerTreeHostImpl::CCLayerTreeHostImpl):
(WebCore::CCLayerTreeHostImpl::setPageScaleFactorAndLimits):
(WebCore::CCLayerTreeHostImpl::adjustScrollsForPageScaleChange):
(WebCore::CCLayerTreeHostImpl::setScaleDelta):
(WebCore::CCLayerTreeHostImpl::applyScaleDeltaToScrollLayer):
(WebCore::CCLayerTreeHostImpl::scrollRootLayer):
(WebCore::CCLayerTreeHostImpl::processScrollDeltas):
* platform/graphics/chromium/cc/CCLayerTreeHostImpl.h:

Source/WebKit/chromium:

* src/WebViewImpl.cpp:
(WebKit::WebViewImpl::applyScrollAndScale):

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

12 files changed:
Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/chromium/LayerChromium.h
Source/WebCore/platform/graphics/chromium/cc/CCLayerImpl.cpp
Source/WebCore/platform/graphics/chromium/cc/CCLayerImpl.h
Source/WebCore/platform/graphics/chromium/cc/CCLayerTreeHost.cpp
Source/WebCore/platform/graphics/chromium/cc/CCLayerTreeHostCommon.cpp
Source/WebCore/platform/graphics/chromium/cc/CCLayerTreeHostCommon.h
Source/WebCore/platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp
Source/WebCore/platform/graphics/chromium/cc/CCLayerTreeHostImpl.h
Source/WebKit/chromium/ChangeLog
Source/WebKit/chromium/src/WebViewImpl.cpp
Source/WebKit/chromium/tests/CCLayerImplTest.cpp

index d28d9ca659ca0818281a8b3ea0cbdaae6f036ac5..ab8995aeb14e6c3756dbf4e667fcd0ba29f82ad7 100644 (file)
@@ -1,3 +1,36 @@
+2011-11-16  Alexandre Elias  <aelias@google.com>
+
+        [chromium] Improvements for page scale delta during commit
+        https://bugs.webkit.org/show_bug.cgi?id=72471
+
+        Reviewed by James Robinson.
+
+        Page scale now follows the same commit flow as scroll position:
+        the delta is folded into m_pageScale at BFAC time, and a "sent" value
+        is preserved for temporary use until the commit finishes.
+
+        I also merged setPageScaleFactor and setPageScaleFactorLimits into one
+        function on the impl side.  The reason is that setPageFactor must
+        be applied after the limits are updated, but on the other hand setting
+        the limits first may cause an unnecessary clamp of the scale delta.
+        Merging the methods avoids this bind.
+
+        No new tests. (planning to add later: https://bugs.webkit.org/show_bug.cgi?id=71529)
+
+        * platform/graphics/chromium/cc/CCLayerTreeHost.cpp:
+        (WebCore::CCLayerTreeHost::finishCommitOnImplThread):
+        (WebCore::CCLayerTreeHost::applyScrollAndScale):
+        * platform/graphics/chromium/cc/CCLayerTreeHostCommon.h:
+        * platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:
+        (WebCore::CCLayerTreeHostImpl::CCLayerTreeHostImpl):
+        (WebCore::CCLayerTreeHostImpl::setPageScaleFactorAndLimits):
+        (WebCore::CCLayerTreeHostImpl::adjustScrollsForPageScaleChange):
+        (WebCore::CCLayerTreeHostImpl::setScaleDelta):
+        (WebCore::CCLayerTreeHostImpl::applyScaleDeltaToScrollLayer):
+        (WebCore::CCLayerTreeHostImpl::scrollRootLayer):
+        (WebCore::CCLayerTreeHostImpl::processScrollDeltas):
+        * platform/graphics/chromium/cc/CCLayerTreeHostImpl.h:
+
 2011-11-16  Joshua Bell  <jsbell@chromium.org>
 
         IndexedDB: Reduce nested key depth threshold, re-enable flaky test
index aab9d38858607c8459f1cdb7349b09dd4b2325c7..f5c4318dc99ffc17c5af4da4e9318a9c7223d593 100644 (file)
@@ -138,7 +138,7 @@ public:
 
     IntSize scrollDelta() const { return IntSize(); }
 
-    float scaleDelta() const { return 1; }
+    float pageScaleDelta() const { return 1; }
 
     bool doubleSided() const { return m_doubleSided; }
     void setDoubleSided(bool doubleSided) { m_doubleSided = doubleSided; setNeedsCommit(); }
index 9048fdc82d105b3c6260deab1d910d1081eaa466..5dba9d0bfa2213a22c3255ab4f06e12c73b595b7 100644 (file)
@@ -52,7 +52,7 @@ CCLayerImpl::CCLayerImpl(int id)
     , m_usesLayerClipping(false)
     , m_isNonCompositedContent(false)
     , m_drawsContent(false)
-    , m_scaleDelta(1)
+    , m_pageScaleDelta(1)
     , m_targetRenderSurface(0)
     , m_drawDepth(0)
     , m_drawOpacity(0)
@@ -405,10 +405,10 @@ void CCLayerImpl::setScrollDelta(const IntSize& scrollDelta)
     }
 }
 
-void CCLayerImpl::setScaleDelta(float scaleDelta)
+void CCLayerImpl::setPageScaleDelta(float pageScaleDelta)
 {
-    if (m_scaleDelta != scaleDelta) {
-        m_scaleDelta = scaleDelta;
+    if (m_pageScaleDelta != pageScaleDelta) {
+        m_pageScaleDelta = pageScaleDelta;
         noteLayerPropertyChangedForSubtree();
     }
 }
index 524133591f421e58ff6e53983607ceaf7629b311..099f4021ac8b103e46a2843c323b1b94287a6a5e 100644 (file)
@@ -160,8 +160,8 @@ public:
     const IntSize& scrollDelta() const { return m_scrollDelta; }
     void setScrollDelta(const IntSize&);
 
-    float scaleDelta() const { return m_scaleDelta; }
-    void setScaleDelta(float);
+    float pageScaleDelta() const { return m_pageScaleDelta; }
+    void setPageScaleDelta(float);
 
     const IntSize& sentScrollDelta() const { return m_sentScrollDelta; }
     void setSentScrollDelta(const IntSize& sentScrollDelta) { m_sentScrollDelta = sentScrollDelta; }
@@ -253,7 +253,7 @@ private:
     IntSize m_scrollDelta;
     IntSize m_sentScrollDelta;
     IntSize m_maxScrollPosition;
-    float m_scaleDelta;
+    float m_pageScaleDelta;
 
     // Properties owned exclusively by this CCLayerImpl.
     // Debugging.
index 124bc41fd90282b1a9e8c9dc1be534952ec51561..63722b9de3304d4e43e8c0f02fe2f513848f2ceb 100644 (file)
@@ -131,8 +131,7 @@ void CCLayerTreeHost::finishCommitOnImplThread(CCLayerTreeHostImpl* hostImpl)
     hostImpl->setHaveWheelEventHandlers(m_haveWheelEventHandlers);
     hostImpl->setZoomAnimatorTransform(m_zoomAnimatorTransform);
     hostImpl->setViewport(viewportSize());
-    hostImpl->setPageScale(pageScale());
-    hostImpl->setPageScaleFactorLimits(m_minPageScale, m_maxPageScale);
+    hostImpl->setPageScaleFactorAndLimits(pageScale(), m_minPageScale, m_maxPageScale);
 
     // Synchronize trees, if one exists at all...
     if (rootLayer())
@@ -464,7 +463,7 @@ void CCLayerTreeHost::applyScrollAndScale(const CCScrollAndScaleSet& info)
 
     ASSERT(info.scrolls.size() == 1);
     IntSize scrollDelta = info.scrolls[0].scrollDelta;
-    m_client->applyScrollAndScale(scrollDelta, info.pageScale);
+    m_client->applyScrollAndScale(scrollDelta, info.pageScaleDelta);
 }
 
 void CCLayerTreeHost::startRateLimiter(GraphicsContext3D* context)
index 7239191b137d4b87c885a8c81fb8a0775b01a622..3e750f8c1c3c628a2bebfc10143aa31a7b861169 100644 (file)
@@ -198,13 +198,13 @@ static void calculateDrawTransformsAndVisibilityInternal(LayerType* layer, Layer
     TransformationMatrix layerLocalTransform;
     // LT = Tr[origin] * M[zoomAnimator]
     layerLocalTransform.multiply(layer->zoomAnimatorTransform());
-    // LT = Tr[origin] * M[zoomAnimator] * S[scaleDelta]
-    layerLocalTransform.scale(layer->scaleDelta());
-    // LT = Tr[origin] * M[zoomAnimator] * S[scaleDelta] * Tr[origin2anchor]
+    // LT = Tr[origin] * M[zoomAnimator] * S[pageScaleDelta]
+    layerLocalTransform.scale(layer->pageScaleDelta());
+    // LT = Tr[origin] * M[zoomAnimator] * S[pageScaleDelta] * Tr[origin2anchor]
     layerLocalTransform.translate3d(position.x(), position.y(), layer->anchorPointZ());
-    // LT = Tr[origin] * M[zoomAnimator] * S[scaleDelta] * Tr[origin2anchor] * M[layer]
+    // LT = Tr[origin] * M[zoomAnimator] * S[pageScaleDelta] * Tr[origin2anchor] * M[layer]
     layerLocalTransform.multiply(layer->transform());
-    // LT = Tr[origin] * M[zoomAnimator] * S[scaleDelta] * Tr[origin2anchor] * M[layer] * Tr[anchor2center]
+    // LT = Tr[origin] * M[zoomAnimator] * S[pageScaleDelta] * Tr[origin2anchor] * M[layer] * Tr[anchor2center]
     layerLocalTransform.translate3d(centerOffsetX, centerOffsetY, -layer->anchorPointZ());
 
     TransformationMatrix combinedTransform = parentMatrix;
index 529b3b320cf7244e2e4198ef197e5c43dc5b69ac..8a57bfe959373428b6a89b9a8a2c83f798bf59d1 100644 (file)
@@ -53,7 +53,7 @@ public:
 
 struct CCScrollAndScaleSet {
     Vector<CCLayerTreeHostCommon::ScrollUpdateInfo> scrolls;
-    float pageScale;
+    float pageScaleDelta;
 };
 
 template<typename LayerType>
index a060578c27db16b01e4a05b56536213635b56645..b3e9031da4a9af081e3231fa10cd54c2edd91606 100644 (file)
@@ -49,7 +49,8 @@ CCLayerTreeHostImpl::CCLayerTreeHostImpl(const CCSettings& settings, CCLayerTree
     , m_visible(true)
     , m_haveWheelEventHandlers(false)
     , m_pageScale(1)
-    , m_scaleDelta(1)
+    , m_pageScaleDelta(1)
+    , m_sentPageScaleDelta(1)
     , m_minPageScale(0)
     , m_maxPageScale(0)
     , m_pinchGestureActive(false)
@@ -194,27 +195,42 @@ void CCLayerTreeHostImpl::setViewport(const IntSize& viewportSize)
     m_layerRenderer->viewportChanged();
 }
 
-void CCLayerTreeHostImpl::setPageScale(float pageScale)
+void CCLayerTreeHostImpl::setPageScaleFactorAndLimits(float pageScale, float minPageScale, float maxPageScale)
 {
-    if (!pageScale || pageScale == m_pageScale)
+    if (!pageScale)
         return;
 
-    // Maintain constant the product of m_pageScale * m_scaleDelta.
-    float scaleChange = pageScale / m_pageScale;
-    float oldMagnifyScale = m_pageScale * m_scaleDelta;
+    if (m_sentPageScaleDelta == 1 && pageScale == m_pageScale && minPageScale == m_minPageScale && maxPageScale == m_maxPageScale)
+        return;
+
+    m_minPageScale = minPageScale;
+    m_maxPageScale = maxPageScale;
+
+    float pageScaleChange = pageScale / m_pageScale;
     m_pageScale = pageScale;
+    m_sentPageScaleDelta = 1;
 
-    setScaleDelta(oldMagnifyScale / pageScale);
+    // Clamp delta to limits and refresh display matrix.
+    setPageScaleDelta(m_pageScaleDelta);
+    applyPageScaleDeltaToScrollLayer();
+
+    adjustScrollsForPageScaleChange(pageScaleChange);
+}
+
+void CCLayerTreeHostImpl::adjustScrollsForPageScaleChange(float pageScaleChange)
+{
+    if (pageScaleChange == 1)
+        return;
 
     // We also need to convert impl-side scroll deltas to pageScale space.
-    if (m_scrollLayerImpl && m_scrollLayerImpl->scrollable()) {
+    if (m_scrollLayerImpl) {
         IntSize scrollDelta = m_scrollLayerImpl->scrollDelta();
-        scrollDelta.scale(scaleChange);
+        scrollDelta.scale(pageScaleChange);
         m_scrollLayerImpl->setScrollDelta(scrollDelta);
     }
 }
 
-void CCLayerTreeHostImpl::setScaleDelta(float delta)
+void CCLayerTreeHostImpl::setPageScaleDelta(float delta)
 {
     // Clamp to the current min/max limits.
     float finalMagnifyScale = m_pageScale * delta;
@@ -223,24 +239,19 @@ void CCLayerTreeHostImpl::setScaleDelta(float delta)
     else if (m_maxPageScale && finalMagnifyScale > m_maxPageScale)
         delta = m_maxPageScale / m_pageScale;
 
-    if (delta == m_scaleDelta)
+    if (delta == m_pageScaleDelta)
         return;
 
-    m_scaleDelta = delta;
+    m_pageScaleDelta = delta;
 
     updateMaxScrollPosition();
-
-    if (m_scrollLayerImpl && m_scrollLayerImpl->scrollable())
-        m_scrollLayerImpl->setScaleDelta(m_scaleDelta);
+    applyPageScaleDeltaToScrollLayer();
 }
 
-void CCLayerTreeHostImpl::setPageScaleFactorLimits(float minPageScale, float maxPageScale)
+void CCLayerTreeHostImpl::applyPageScaleDeltaToScrollLayer()
 {
-    m_minPageScale = minPageScale;
-    m_maxPageScale = maxPageScale;
-
-    // This will clamp m_scaleDelta and apply side effects if clamped.
-    setScaleDelta(m_scaleDelta);
+    if (m_scrollLayerImpl)
+        m_scrollLayerImpl->setPageScaleDelta(m_pageScaleDelta * m_sentPageScaleDelta);
 }
 
 void CCLayerTreeHostImpl::updateMaxScrollPosition()
@@ -249,7 +260,7 @@ void CCLayerTreeHostImpl::updateMaxScrollPosition()
         return;
 
     FloatSize viewBounds = m_viewportSize;
-    viewBounds.scale(1 / m_scaleDelta);
+    viewBounds.scale(1 / m_pageScaleDelta);
 
     // TODO(aelias): Hardcoding the first child here is weird. Think of
     // a cleaner way to get the contentBounds on the Impl side.
@@ -274,7 +285,7 @@ void CCLayerTreeHostImpl::setZoomAnimatorTransform(const TransformationMatrix& z
 void CCLayerTreeHostImpl::scrollRootLayer(const IntSize& scrollDelta)
 {
     TRACE_EVENT("CCLayerTreeHostImpl::scrollRootLayer", this, 0);
-    if (!m_scrollLayerImpl || !m_scrollLayerImpl->scrollable())
+    if (!m_scrollLayerImpl)
         return;
 
     m_scrollLayerImpl->scrollBy(scrollDelta);
@@ -304,9 +315,9 @@ void CCLayerTreeHostImpl::pinchGestureUpdate(float magnifyDelta,
 
     // Keep the center-of-pinch anchor specified by (x, y) in a stable
     // position over the course of the magnify.
-    FloatPoint prevScaleAnchor(anchor.x() / m_scaleDelta, anchor.y() / m_scaleDelta);
-    setScaleDelta(m_scaleDelta * magnifyDelta);
-    FloatPoint newScaleAnchor(anchor.x() / m_scaleDelta, anchor.y() / m_scaleDelta);
+    FloatPoint prevScaleAnchor(anchor.x() / m_pageScaleDelta, anchor.y() / m_pageScaleDelta);
+    setPageScaleDelta(m_pageScaleDelta * magnifyDelta);
+    FloatPoint newScaleAnchor(anchor.x() / m_pageScaleDelta, anchor.y() / m_pageScaleDelta);
 
     FloatSize move = prevScaleAnchor - newScaleAnchor;
 
@@ -325,14 +336,17 @@ void CCLayerTreeHostImpl::pinchGestureEnd()
 PassOwnPtr<CCScrollAndScaleSet> CCLayerTreeHostImpl::processScrollDeltas()
 {
     OwnPtr<CCScrollAndScaleSet> scrollInfo = adoptPtr(new CCScrollAndScaleSet());
-    bool didMove = m_scrollLayerImpl && (!m_scrollLayerImpl->scrollDelta().isZero() || m_scaleDelta != 1.0f);
+    bool didMove = m_scrollLayerImpl && (!m_scrollLayerImpl->scrollDelta().isZero() || m_pageScaleDelta != 1.0f);
     if (!didMove || m_pinchGestureActive) {
-        scrollInfo->pageScale = m_pageScale;
+        m_sentPageScaleDelta = scrollInfo->pageScaleDelta = 1;
         return scrollInfo.release();
     }
 
+    m_sentPageScaleDelta = scrollInfo->pageScaleDelta = m_pageScaleDelta;
+    m_pageScale = m_pageScale * m_sentPageScaleDelta;
+    setPageScaleDelta(1);
+
     // FIXME: track scrolls from layers other than the root
-    scrollInfo->pageScale = m_pageScale * m_scaleDelta;
     CCLayerTreeHostCommon::ScrollUpdateInfo scroll;
     scroll.layerId = m_scrollLayerImpl->id();
     scroll.scrollDelta = m_scrollLayerImpl->scrollDelta();
@@ -343,6 +357,8 @@ PassOwnPtr<CCScrollAndScaleSet> CCLayerTreeHostImpl::processScrollDeltas()
     m_scrollLayerImpl->setSentScrollDelta(m_scrollLayerImpl->scrollDelta());
     m_scrollLayerImpl->setScrollDelta(IntSize());
 
+    adjustScrollsForPageScaleChange(m_sentPageScaleDelta);
+
     return scrollInfo.release();
 }
 
index 44b9eb93e8a445768d43782701d600c430042fde..c29fe4e0941152e40e0c4d2c2cdea237c49ed9f4 100644 (file)
@@ -99,9 +99,8 @@ public:
     const IntSize& viewportSize() const { return m_viewportSize; }
     void setZoomAnimatorTransform(const TransformationMatrix&);
 
-    void setPageScale(float);
+    void setPageScaleFactorAndLimits(float pageScale, float minPageScale, float maxPageScale);
     float pageScale() const { return m_pageScale; }
-    void setPageScaleFactorLimits(float minPageScale, float maxPageScale);
 
     const CCSettings& settings() const { return m_settings; }
 
@@ -109,7 +108,6 @@ public:
     virtual void pinchGestureUpdate(float, const IntPoint&);
     virtual void pinchGestureEnd();
     PassOwnPtr<CCScrollAndScaleSet> processScrollDeltas();
-    void updateMaxScrollPosition();
 
 protected:
     CCLayerTreeHostImpl(const CCSettings&, CCLayerTreeHostImplClient*);
@@ -118,7 +116,10 @@ protected:
     int m_frameNumber;
 
 private:
-    void setScaleDelta(float);
+    void setPageScaleDelta(float);
+    void applyPageScaleDeltaToScrollLayer();
+    void adjustScrollsForPageScaleChange(float);
+    void updateMaxScrollPosition();
 
     OwnPtr<LayerRendererChromium> m_layerRenderer;
     RefPtr<CCLayerImpl> m_rootLayerImpl;
@@ -129,7 +130,8 @@ private:
     bool m_haveWheelEventHandlers;
 
     float m_pageScale;
-    float m_scaleDelta;
+    float m_pageScaleDelta;
+    float m_sentPageScaleDelta;
     float m_minPageScale, m_maxPageScale;
 
     bool m_pinchGestureActive;
index 729a9f7d0edde082e7c014c57e24267c1d37c495..1c2ecae3a3f67b615db9e298b72276c643bd67a9 100644 (file)
@@ -1,3 +1,25 @@
+2011-11-16  Alexandre Elias  <aelias@google.com>
+
+        [chromium] Improvements for page scale delta during commit
+        https://bugs.webkit.org/show_bug.cgi?id=72471
+
+        Reviewed by James Robinson.
+
+        Page scale now follows the same commit flow as scroll position:
+        the delta is folded into m_pageScale at BFAC time, and a "sent" value
+        is preserved for temporary use until the commit finishes.
+
+        I also merged setPageScaleFactor and setPageScaleFactorLimits into one
+        function on the impl side.  The reason is that setPageFactor must
+        be applied after the limits are updated, but on the other hand setting
+        the limits first may cause an unnecessary clamp of the scale delta.
+        Merging the methods avoids this bind.
+
+        No new tests. (planning to add later: https://bugs.webkit.org/show_bug.cgi?id=71529)
+
+        * src/WebViewImpl.cpp:
+        (WebKit::WebViewImpl::applyScrollAndScale):
+
 2011-11-16  Dave Michael  <dmichael@chromium.org>
 
         Fix up chromium API for creating events, initializing message events
index 8a1231747c36818be39f1f30567ffad615306771..b87995df050ebf40f32cde6fffa8bcbf05ad6981 100644 (file)
@@ -2792,16 +2792,12 @@ void WebViewImpl::animateAndLayout(double frameBeginTime)
     layout();
 }
 
-void WebViewImpl::applyScrollAndScale(const IntSize& scrollDelta, float scaleFactor)
+void WebViewImpl::applyScrollAndScale(const IntSize& scrollDelta, float pageScaleDelta)
 {
     if (!mainFrameImpl() || !mainFrameImpl()->frameView())
         return;
 
-    float oldScale = pageScaleFactor();
-    if (!oldScale)
-        oldScale = 1;
-
-    if (!scaleFactor || oldScale == scaleFactor)
+    if (pageScaleDelta == 1)
         mainFrameImpl()->frameView()->scrollBy(scrollDelta);
     else {
         // The page scale changed, so apply a scale and scroll in a single
@@ -2811,10 +2807,9 @@ void WebViewImpl::applyScrollAndScale(const IntSize& scrollDelta, float scaleFac
         WebSize scrollOffset = mainFrame()->scrollOffset();
         scrollOffset.width += scrollDelta.width();
         scrollOffset.height += scrollDelta.height();
-        float scaleDelta = scaleFactor / oldScale;
-        WebPoint scaledScrollOffset(scrollOffset.width * scaleDelta,
-                                    scrollOffset.height * scaleDelta);
-        setPageScaleFactor(scaleFactor, scaledScrollOffset);
+        WebPoint scaledScrollOffset(scrollOffset.width * pageScaleDelta,
+                                    scrollOffset.height * pageScaleDelta);
+        setPageScaleFactor(pageScaleFactor() * pageScaleDelta, scaledScrollOffset);
     }
 }
 
index d7333876bb446f8be5483a574b5565764151b7ba..06dc6c1df385d5fab0cab57f4d06cf5ff5440f47 100644 (file)
@@ -111,7 +111,7 @@ TEST(CCLayerImplTest, verifyLayerChangesAreTrackedProperly)
     EXECUTE_AND_VERIFY_SUBTREE_CHANGED(root->scrollBy(arbitraryIntSize));
     EXECUTE_AND_VERIFY_SUBTREE_CHANGED(root->setScrollDelta(arbitraryIntSize));
     EXECUTE_AND_VERIFY_SUBTREE_CHANGED(root->setScrollPosition(arbitraryIntPoint));
-    EXECUTE_AND_VERIFY_SUBTREE_CHANGED(root->setScaleDelta(arbitraryNumber));
+    EXECUTE_AND_VERIFY_SUBTREE_CHANGED(root->setPageScaleDelta(arbitraryNumber));
 
     // Changing these properties only affects the layer itself.
     EXECUTE_AND_VERIFY_ONLY_LAYER_CHANGED(root->setContentBounds(arbitraryIntSize));
@@ -149,7 +149,7 @@ TEST(CCLayerImplTest, verifyLayerChangesAreTrackedProperly)
     EXECUTE_AND_VERIFY_SUBTREE_DID_NOT_CHANGE(root->setDoubleSided(false)); // constructor initializes it to "true".
     EXECUTE_AND_VERIFY_SUBTREE_DID_NOT_CHANGE(root->setScrollDelta(arbitraryIntSize));
     EXECUTE_AND_VERIFY_SUBTREE_DID_NOT_CHANGE(root->setScrollPosition(arbitraryIntPoint));
-    EXECUTE_AND_VERIFY_SUBTREE_DID_NOT_CHANGE(root->setScaleDelta(arbitraryNumber));
+    EXECUTE_AND_VERIFY_SUBTREE_DID_NOT_CHANGE(root->setPageScaleDelta(arbitraryNumber));
     EXECUTE_AND_VERIFY_SUBTREE_DID_NOT_CHANGE(root->setContentBounds(arbitraryIntSize));
     EXECUTE_AND_VERIFY_SUBTREE_DID_NOT_CHANGE(root->setOpaque(true));
     EXECUTE_AND_VERIFY_SUBTREE_DID_NOT_CHANGE(root->setOpacity(arbitraryNumber));