[Remoting Android] Move setAnimationEnabled to TouchInputHandler

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

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.chromoting;
 
 import android.content.Context;
 import android.graphics.Matrix;
 import android.graphics.PointF;
 import android.text.InputType;
@@ skipping 25 matching lines @@
      */
     protected final Event.Raisable<SizeChangedEventParameter> mOnClientSizeChanged =
             new Event.Raisable<>();
 
     /**
      * Subclass should trigger this event when the host (desktop frame) size is changed.
      */
     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;
 
     /** The parent Desktop activity. */
     private final Desktop mDesktop;
 
     private final Event.Raisable<TouchEventParameter> mOnTouch = new Event.Raisable<>();
 
     public DesktopView(Desktop desktop, Client client) {
@@ skipping 38 matching lines @@
     /** An {@link Event} which is triggered when the client size is changed. */
     public final Event<SizeChangedEventParameter> onClientSizeChanged() {
         return mOnClientSizeChanged;
     }
 
     /** An {@link Event} which is triggered when the host size is changed. */
     public final Event<SizeChangedEventParameter> onHostSizeChanged() {
         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
     public final InputConnection onCreateInputConnection(EditorInfo outAttrs) {
         // Disables rich input support and instead requests simple key events.
         outAttrs.inputType = InputType.TYPE_NULL;
 
         // Prevents most third-party IMEs from ignoring our Activity's adjustResize preference.
         outAttrs.imeOptions |= EditorInfo.IME_FLAG_NO_FULLSCREEN;
 
@@ skipping 51 matching lines @@
      * Informs the view that the cursor has been moved by the TouchInputHandler, which requires
      * repainting.
      */
     public abstract void cursorMoved(PointF position);
 
     /**
      * Informs the view that the cursor visibility has been changed (for different input mode) by
      * 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);
 }