Converted LocalInputMonitor into a SessionController instance.

remoting/host/session_controller.h

 // Copyright (c) 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.
 
 #ifndef REMOTING_HOST_SESSION_CONTROLLER_H_
 #define REMOTING_HOST_SESSION_CONTROLLER_H_
 
 #include "base/basictypes.h"
-#include "remoting/host/mouse_move_observer.h"
 #include "third_party/skia/include/core/SkPoint.h"
 #include "third_party/skia/include/core/SkSize.h"
 
 namespace remoting {
 
 class ScreenResolution;
 
+// The SessionController interface should be implemented by objects
+// (representing some aspects of the desktop environment) that can control

[Sergey Ulanov, 2013/03/20 17:23:31]

nit: drop the parentheses

[alexeypa (please no reviews), 2013/03/20 17:34:31]

Done.

+// the client session (disconnect, pause, or block the remote input).
+//
+// Objects implementing SessionController are created by
+// DektopEnvironment::CreateSessionController() method. They are created after
+// a client session has been authenticated and destroyed along with the session.
+//
+// The desired screen resolution of is also selected via SessionController.
+// Strictly speaking, it could be implemented via a separate interface. Adding
+// it here however saves us lots of hassle with adding another CreateXxx()
+// method to all implementations of DesktopEnvironment (eight at the moment of

[Sergey Ulanov, 2013/03/20 17:23:31]

nit: no need to be so wordy about it - maybe remove the stuff in parentheses

[alexeypa (please no reviews), 2013/03/20 17:34:31]

Done.

+// writing).
 class SessionController {
  public:
-  // TODO(alexeypa): remove MouseMoveObserver entirely and move
-  // OnLocalMouseMoved() method to SessionController::Delegate.
-  // See http://crbug.com/104544.
-  class Delegate : public MouseMoveObserver {
+  // Provides nobs for controlling the client session.
+  class Delegate {
    public:
     virtual ~Delegate() {}
 
     // Disconnects the client session, tears down transport resources and stops
     // scheduler components.
     virtual void DisconnectSession() = 0;
 
     // Called when local mouse movement is detected.
     virtual void OnLocalMouseMoved(const SkIPoint& position) = 0;
+
+    // Pauses or resumes the session.
+    virtual void PauseSession(bool pause) = 0;
   };
 
   virtual ~SessionController() {}
 
   // Attempts to set new screen resolution in the session.
-  virtual void SetScreenResolution(const ScreenResolution& resolution) = 0;
+  virtual void SetScreenResolution(const ScreenResolution& resolution) {}
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_SESSION_CONTROLLER_H_