Make explicit which TextIndicator animations are driven manually, and which run autom...
[WebKit-https.git] / Source / WebCore / ChangeLog
index 89f9aa249c4527690f41771680a3150fc0c12d0b..27c23b24cb178397c4d2aa9e09480a7d70d29043 100644 (file)
@@ -1,3 +1,25 @@
+2015-01-06  Timothy Horton  <timothy_horton@apple.com>
+
+        Make explicit which TextIndicator animations are driven manually, and which run automatically
+        https://bugs.webkit.org/show_bug.cgi?id=140113
+        <rdar://problem/19383425>
+
+        Reviewed by Darin Adler.
+
+        * page/TextIndicator.cpp:
+        (WebCore::TextIndicator::wantsManualAnimation):
+        Add wantsManualAnimation(). The old transitions (Bounce and BounceAndCrossfade)
+        run automatically, and the new ones (FadeIn and Crossfade) are driven manually.
+
+        * page/TextIndicator.h:
+        Add a comment explaining which animations are manual and which are not.
+
+        * page/mac/TextIndicatorWindow.mm:
+        (-[WebTextIndicatorView present]):
+        Make use of wantsManualAnimation(). The previous (wantsFadeIn || wantsCrossfade)
+        was wrong, because it was true for BounceAndCrossfade, and would cause
+        BounceAndCrossfade animations (which aren't driven manually) to stall at progress=0.
+
 2015-01-06  Anders Carlsson  <andersca@apple.com>
 
         Give empty pages a valid database provider.