Refactor the client code for the X11 version.

Make ChromotingViews responsible for initializing themselves.
Move all x11-related code into X11View.
Create InputCapturer class manage client input capture.

BUG=none
TEST=ran Win host + X11 client

Review URL: http://codereview.chromium.org/2861047

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52973 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 10 insertions, 5 deletions

diff --git a/remoting/client/chromoting_view.h b/remoting/client/chromoting_view.h
index 503a903..a50b407 100644
--- a/remoting/client/chromoting_view.h
+++ b/remoting/client/chromoting_view.h
@@ -18,6 +18,12 @@ class ChromotingView {
  public:
   virtual ~ChromotingView() {}
 
+  // Initialize the common structures for the view.
+  virtual bool Initialize() = 0;
+
+  // Free up resources allocated by this view.
+  virtual void TearDown() = 0;
+
   // Tells the ChromotingView to paint the current image on the screen.
   // TODO(hclam): Add rects as parameter if needed.
   virtual void Paint() = 0;
@@ -34,12 +40,11 @@ class ChromotingView {
   // extends past the end of the backing store, it is filled with black.
   virtual void SetViewport(int x, int y, int width, int height) = 0;
 
-  // Resize the underlying image that is displayed.  This should match the size
-  // of the output from the decoder.
+  // Resize the underlying image that contains the host screen buffer.
+  // This should match the size of the output from the decoder.
   //
-  // TODO(ajwong): We need a better name.  Look at how Java represents this
-  // stuff?
-  virtual void SetBackingStoreSize(int width, int height) = 0;
+  // TODO(garykac): This handles only 1 screen. We need multi-screen support.
+  virtual void SetHostScreenSize(int width, int height) = 0;
 
   // Handle the BeginUpdateStream message.
   virtual void HandleBeginUpdateStream(HostMessage* msg) = 0;