Changeset 97902 in webkit

Timestamp:

Oct 19, 2011, 5:44:28 PM (14 years ago)

Author:

wjmaclean@chromium.org

Message:

[chromium] Shorten animation time for ZoomAnimator.
​https://bugs.webkit.org/show_bug.cgi?id=70419

Reviewed by Kenneth Russell.

Shorten animation time for ZoomAnimator.

Existing test coverage sufficient.

• platform/ScrollAnimatorNone.cpp:

(WebCore::ScrollAnimatorNone::zoom):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/ScrollAnimatorNone.cpp (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-10-19  W. James MacLean  <wjmaclean@chromium.org>
+
+        [chromium] Shorten animation time for ZoomAnimator.
+        https://bugs.webkit.org/show_bug.cgi?id=70419
+
+        Reviewed by Kenneth Russell.
+
+        Shorten animation time for ZoomAnimator.
+
+        Existing test coverage sufficient.
+
+        * platform/ScrollAnimatorNone.cpp:
+        (WebCore::ScrollAnimatorNone::zoom):
+
 2011-10-19  Zhenyao Mo  <zmo@google.com>
 

trunk/Source/WebCore/platform/ScrollAnimatorNone.cpp

@@ -56 +56 @@
 static double kTickTime = 1 / kFrameRate;
 static double kMinimumTimerInterval = .001;
+static double kZoomTicks = 11;
 
 PassOwnPtr<ScrollAnimator> ScrollAnimator::create(ScrollableArea* scrollableArea)
@@ -497 +498 @@
 #endif
         m_zoomData.m_lastAnimationTime = currentTime;
-        m_zoomData.m_animationTime = 50 * kTickTime;
+        m_zoomData.m_animationTime = kZoomTicks * kTickTime;
 
         bool isContinuing = m_zoomData.animateZoom(currentTime);