[Remoting Android] Move setAnimationEnabled to TouchInputHandler

remoting/android/java/src/org/chromium/chromoting/DesktopView.java

Index: remoting/android/java/src/org/chromium/chromoting/DesktopView.java
diff --git a/remoting/android/java/src/org/chromium/chromoting/DesktopView.java b/remoting/android/java/src/org/chromium/chromoting/DesktopView.java
index af4b1d08fa23fd29b04221b947a983c2a64a6e88..43e3907e8c327ba3d630ec73032b6c9d69d0a52b 100644
--- a/remoting/android/java/src/org/chromium/chromoting/DesktopView.java
+++ b/remoting/android/java/src/org/chromium/chromoting/DesktopView.java
@@ -43,6 +43,11 @@ public abstract class DesktopView extends SurfaceView {
     protected final Event.Raisable<SizeChangedEventParameter> mOnHostSizeChanged =
             new Event.Raisable<>();
 
+    /**
+     * Subclass should trigger this event when a frame is rendered.
+     */
+    protected final Event.Raisable<Void> mOnCanvasRendered = new Event.Raisable<>();
+

[Yuwei, 2016/08/26 20:34:32]

DesktopView.onCanvasRendered() is just temporary. Finally all events will be
removed from DesktopView and there will be a simpler way to forward events from
GlDisplay.

[Hzj_jie, 2016/08/29 22:20:23]

I believe mOnTouch is still raised from GlDesktopView, right?

[Yuwei, 2016/08/29 23:24:25]

Yep, but DesktopView to be precise. That's a view's event so we probably can't
get rid of it.

     private final int mTinyFeedbackPixelRadius;
     private final int mSmallFeedbackPixelRadius;
     private final int mLargeFeedbackPixelRadius;
@@ -101,6 +106,11 @@ public abstract class DesktopView extends SurfaceView {
         return mOnHostSizeChanged;
     }
 
+    /** An {@link Event} which is triggered when a frame is rendered. */
+    public final Event<Void> onCanvasRendered() {
+        return mOnCanvasRendered;
+    }
+
     // View overrides.
     /** Called when a software keyboard is requested, and specifies its options. */
     @Override
@@ -172,10 +182,4 @@ public abstract class DesktopView extends SurfaceView {
      * the TouchInputHandler, which requires repainting.
      */
     public abstract void cursorVisibilityChanged(boolean visible);
-
-    /**
-     * Starts or stops an animation. Whilst the animation is running, the DesktopView will
-     * periodically call TouchInputHandler.processAnimation() and repaint itself.
-     */
-    public abstract void setAnimationEnabled(boolean enabled);
 }