Android: Call SetNeedsAnimate() for toolbar animations

chrome/android/java/src/org/chromium/chrome/browser/fullscreen/ChromeFullscreenManager.java

Index: chrome/android/java/src/org/chromium/chrome/browser/fullscreen/ChromeFullscreenManager.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/fullscreen/ChromeFullscreenManager.java b/chrome/android/java/src/org/chromium/chrome/browser/fullscreen/ChromeFullscreenManager.java
index 3a35fb416a540fdde3f8dabc59749e4aa49ebe45..2cb5801850d51349173061361a29cf01e32aa950 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/fullscreen/ChromeFullscreenManager.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/fullscreen/ChromeFullscreenManager.java
@@ -105,8 +105,9 @@ public class ChromeFullscreenManager
         /**
          * Called whenever the content's visible offset changes.
          * @param offset The new offset of the visible content from the top of the screen.
+         * @param needsAnimate Whether the caller is driving an animation with further updates.

[Ted C, 2016/05/31 22:25:46]

Hmm...this name seems odd to me (or maybe too specific to the compositor).

This class doesn't know if the listeners need to animate as a result of this
call.  It only really knows whether this call was triggered by the browser or
the renderer (most likely).

I guess there isn't a great name for this as the checks below don't actually
account for where their is a browser animation whose offsets are being overruled
by the renderer's as they are in a more visible state.  "maybeFromBrowser"
doesn't exactly have a good ring to it either.  I guess I'll just say it looks
weird and leave it at that.

          */
-        public void onVisibleContentOffsetChanged(float offset);
+        public void onVisibleContentOffsetChanged(float offset, boolean needsAnimate);
 
         /**
          * Called when a ContentVideoView is created/destroyed.
@@ -502,8 +503,12 @@ public class ChromeFullscreenManager
                 mControlContainer.getView().setTranslationY(getControlOffset());
             }
 
+            // Whether we need the compositor to draw again to update our animation.
+            // Should be |false| when the top controls are only moved through the page scrolling.
+            boolean needsAnimate = mControlAnimation != null || shouldShowAndroidControls();
             for (int i = 0; i < mListeners.size(); i++) {
-                mListeners.get(i).onVisibleContentOffsetChanged(getVisibleContentOffset());
+                mListeners.get(i).onVisibleContentOffsetChanged(
+                        getVisibleContentOffset(), needsAnimate);
             }
         }