[Remoting Android] Placeholder for DesktopView

remoting/android/java/src/org/chromium/chromoting/AbstractDesktopView.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.Point;
+import android.view.SurfaceView;
 
 import org.chromium.chromoting.jni.Client;
 
 /**
  * Callback interface to allow the TouchInputHandler to request actions on the DesktopView.
  */
-public interface DesktopViewInterface {
+public abstract class AbstractDesktopView extends SurfaceView {
+    public AbstractDesktopView(Context context) {
+        super(context);
+    }
+
     /**
      * Initializes the instance. Implementations can assume this function will be called exactly
      * once after constructor but before other functions.
      */
-    void init(Desktop desktop, Client client);
+    public abstract void init(Desktop desktop, Client client);
 
     /** Triggers a brief animation to indicate the existence and location of an input event. */
-    void showInputFeedback(DesktopView.InputFeedbackType feedbackToShow, Point pos);
+    public abstract void showInputFeedback(DesktopView.InputFeedbackType feedbackToShow, Point pos);
 
     /** Shows the action bar. */
-    void showActionBar();
+    public abstract void showActionBar();

[Lambros, 2016/07/08 22:02:36]

Do showActionBar() and showKeyboard() need to be part of this interface? I'm not
sure why the View should care about these?
Doesn't matter for this CL, but maybe these can be refactored out somehow?

[Yuwei, 2016/07/08 23:07:54]

Agreed. These functions are only being used by TouchInputHandler. showKeyboard
can be easily moved into TouchInputHandler while showActionBar needs reference
to the desktop activity. May need to pass the activity from DesktopView to the
constructor of TouchInputHandler.

 
     /** Shows the software keyboard. */
-    void showKeyboard();
+    public abstract void showKeyboard();
 
     /**
      * Informs the view that its transformation matrix (for rendering the remote desktop bitmap)
      * has been changed by the TouchInputHandler, which requires repainting.
      */
-    void transformationChanged();
+    public abstract void transformationChanged();
 
     /**
      * Informs the view that the cursor has been moved by the TouchInputHandler, which requires
      * repainting.
      */
-    void cursorMoved();
+    public abstract void cursorMoved();
 
     /**
      * Informs the view that the cursor visibility has been changed (for different input mode) by
      * the TouchInputHandler, which requires repainting.
      */
-    void cursorVisibilityChanged();
+    public abstract void cursorVisibilityChanged();
 
     /**
      * Starts or stops an animation. Whilst the animation is running, the DesktopView will
      * periodically call TouchInputHandler.processAnimation() and repaint itself.
      */
-    void setAnimationEnabled(boolean enabled);
+    public abstract void setAnimationEnabled(boolean enabled);
 
     /**
      * An {@link Event} which is triggered when the view is being painted. Adding handlers to this
      * event causes painting to be triggered continuously until they are all removed.
      */
-    Event<PaintEventParameter> onPaint();
+    public abstract Event<PaintEventParameter> onPaint();
 
     /** An {@link Event} which is triggered when the client size is changed. */
-    Event<SizeChangedEventParameter> onClientSizeChanged();
+    public abstract Event<SizeChangedEventParameter> onClientSizeChanged();
 
     /** An {@link Event} which is triggered when the host size is changed. */
-    Event<SizeChangedEventParameter> onHostSizeChanged();
+    public abstract Event<SizeChangedEventParameter> onHostSizeChanged();
 
     /** An {@link Event} which is triggered when user touchs the screen. */

[Lambros, 2016/07/08 22:02:36]

s/touchs/touches/

[Yuwei, 2016/07/08 23:46:01]

Done.

-    Event<TouchEventParameter> onTouch();
+    public abstract Event<TouchEventParameter> onTouch();
 }