Fix Ctrl+Alt+Esc keyboard shortcut in Me2Me host.

remoting/host/local_input_monitor.h

 // Copyright (c) 2012 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_LOCAL_INPUT_MONITOR_H_
 #define REMOTING_LOCAL_INPUT_MONITOR_H_
 
+#include "base/callback_forward.h"
 #include "base/memory/scoped_ptr.h"
 
 namespace remoting {
 
-class ChromotingHost;
+class MouseMoveObserver;
 
 class LocalInputMonitor {
  public:
   virtual ~LocalInputMonitor() {}
 
-  virtual void Start(ChromotingHost* host) = 0;
+  virtual void Start(MouseMoveObserver* mouse_move_observer,

[Wez, 2012/05/25 23:32:18]

MouseMoveObserver only has a single method, so why not make it a base::Callback,
akin to disconnect_callback, rather than an interface.

[Sergey Ulanov, 2012/05/26 01:06:15]

You can't store callbacks in stl containers and the windows implementation of
LocalInputMonitor needs it.
Also interface looks cleaner than callback in this case IMHO.

[Wez, 2012/05/29 16:57:33]

An alternative would be to allow LocalInputMonitor to accept multiple
LocalInputObservers, or to allow multiple instances to be created, so that both
methods then go on the LocalInputObserver interface.

+                     const base::Closure& disconnect_callback) = 0;

[Wez, 2012/05/25 23:32:18]

This looks a little odd; can you add a comment to explain what "disconnect
callback" is for, since in the context of LocalInputMonitor it's not obvious
(there is nothing to disconnect here!).

[Sergey Ulanov, 2012/05/26 01:06:15]

Done.

   virtual void Stop() = 0;
 
-  // TODO(sergeyu): This is a short-term hack to disable disconnection
-  // shortcut on Mac.
-  virtual void DisableShortcutOnMac() {};
-
   static scoped_ptr<LocalInputMonitor> Create();
 };
 
 }  // namespace remoting
 
 #endif