Converted LocalInputMonitor into a SessionController instance.

remoting/host/session_controller.h

Index: remoting/host/session_controller.h
diff --git a/remoting/host/session_controller.h b/remoting/host/session_controller.h
index d4479afdbf2037d711af9b4410a97121a558eebc..aa23d91c63ef4e920cdc86af11905e9fe49aea34 100644
--- a/remoting/host/session_controller.h
+++ b/remoting/host/session_controller.h
@@ -6,7 +6,6 @@
 #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"
 
@@ -16,10 +15,7 @@ class ScreenResolution;
 
 class SessionController {

[Sergey Ulanov, 2013/03/20 06:31:06]

This interface is used for two unrelated features (1) monitoring mouse moves and
(2) updating screen resolution. LocalInputMonitor doesn't care about resolution,
while ResizingHostObserver doesn't touch delegate. So I wonder why we need this
interface at all? Can Delegate be moved to LocalInputMonitor?

Or is the intent here to provide a generic interface for optional features that
may or may not be enabled? In that case it may be better to name it something
like SessionExtension.

Also pleas add comments for this interface.

[alexeypa (please no reviews), 2013/03/20 17:05:29]

It is indeed used for two unrelated purposes yet I think splitting it into two
separate interfaces will introduce more noise, but little or not gain:

- DesktopEnvironment will get another CreateXxx() method meanisng that it will
be added to all 4 variations of DesktopEnvironment and 4 mocked
DesktopEnvironment's are well.
- ClientSession and DesktopSessoinAgent will get another pointer with life-time
identical to session_controller_.

So I think mixing the two here is less evil.
No. This delegate will be used by the disconnect and continue windows as well.
I picked "SessionController" implying that objects implementing it control the
client session in one of two ways:

- They invoke SessionController::Delegate methods directly telling session what
to do.
- They modify the underlying desktop environment in some way affecting the
client session (resolution change).
Done.

I also added PauseSession() to SessionController::Delegate that will be used by
the continue window code.

  public:
-  // TODO(alexeypa): remove MouseMoveObserver entirely and move
-  // OnLocalMouseMoved() method to SessionController::Delegate.
-  // See http://crbug.com/104544.
-  class Delegate : public MouseMoveObserver {
+  class Delegate {
    public:
     virtual ~Delegate() {}