Removes mouse listeners from UserInputMonitor.

media/base/user_input_monitor.h

 // 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.
 
 #ifndef MEDIA_BASE_USER_INPUT_MONITOR_H_
 #define MEDIA_BASE_USER_INPUT_MONITOR_H_
 
 #include <stddef.h>
 
 #include <memory>
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
-#include "base/observer_list_threadsafe.h"
 #include "base/synchronization/lock.h"
 #include "media/base/media_export.h"
 
-struct SkIPoint;
-
 namespace base {
 class SingleThreadTaskRunner;
 }  // namespace base
 
 namespace media {
 
-// Monitors and notifies about mouse movements and keyboard events.
-// Thread safe. The listeners are called on the thread where the listeners are
-// added.
+// Monitors and notifies about keyboard events.
+// Thread safe.
 class MEDIA_EXPORT UserInputMonitor {
  public:
-  // The interface to receive mouse movement events.
-  class MEDIA_EXPORT MouseEventListener {
-   public:
-    // |position| is the new mouse position.
-    virtual void OnMouseMoved(const SkIPoint& position) = 0;
-
-   protected:
-    virtual ~MouseEventListener() {}
-  };
-  typedef base::ObserverListThreadSafe<UserInputMonitor::MouseEventListener>
-      MouseListenerList;
-
-  UserInputMonitor();
+  UserInputMonitor() : key_press_counter_references_(0) {}

[xhwang, 2017/01/10 20:13:16]

wez: The style guide recommends inlining constructors when the class only
contains POD members. In this case, we do have a base::Lock member. Is it a
concern?

https://www.chromium.org/developers/coding-style/cpp-dos-and-donts#TOC-Stop-i...

[Wez, 2017/01/10 23:19:24]

I agree, it's probably best to keep the constructor in the .cc.

[CJ, 2017/01/11 21:40:27]

Done.

   virtual ~UserInputMonitor();
 
   // Creates a platform-specific instance of UserInputMonitor.
   // |io_task_runner| is the task runner for an IO thread.
   // |ui_task_runner| is the task runner for a UI thread.
   static std::unique_ptr<UserInputMonitor> Create(
       const scoped_refptr<base::SingleThreadTaskRunner>& io_task_runner,
       const scoped_refptr<base::SingleThreadTaskRunner>& ui_task_runner);
 
-  // The same |listener| should only be added once.
-  // The clients should make sure to call Remove*Listener before |listener| is
-  // destroyed.
-  void AddMouseListener(MouseEventListener* listener);
-  void RemoveMouseListener(MouseEventListener* listener);
-
   // A caller must call EnableKeyPressMonitoring and
   // DisableKeyPressMonitoring in pair.
   void EnableKeyPressMonitoring();
   void DisableKeyPressMonitoring();
 
   // Returns the number of keypresses. The starting point from when it is
   // counted is not guaranteed, but consistent within the pair of calls of
   // EnableKeyPressMonitoring and DisableKeyPressMonitoring. So a caller can
   // use the difference between the values returned at two times to get the
   // number of keypresses happened within that time period, but should not make
   // any assumption on the initial value.
   virtual size_t GetKeyPressCount() const = 0;
 
- protected:
-  scoped_refptr<MouseListenerList> mouse_listeners() {
-    return mouse_listeners_;
-  }
-
  private:
   virtual void StartKeyboardMonitoring() = 0;
   virtual void StopKeyboardMonitoring() = 0;
-  virtual void StartMouseMonitoring() = 0;
-  virtual void StopMouseMonitoring() = 0;
 
   base::Lock lock_;
   size_t key_press_counter_references_;
-  size_t mouse_listeners_count_;
-  scoped_refptr<MouseListenerList> mouse_listeners_;
 
   DISALLOW_COPY_AND_ASSIGN(UserInputMonitor);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_USER_INPUT_MONITOR_H_