Move chromeos::EventRewriter to //ui/chromeos/events

ui/chromeos/events/event_rewriter.h

Index: ui/chromeos/events/event_rewriter.h
diff --git a/chrome/browser/chromeos/events/event_rewriter.h b/ui/chromeos/events/event_rewriter.h
similarity index 64%
rename from chrome/browser/chromeos/events/event_rewriter.h
rename to ui/chromeos/events/event_rewriter.h
index e887a2c8707df412425cbecf8e66d1db9970a359..1c36719c0fe242a81b86a80ae1262bcb805129e6 100644
--- a/chrome/browser/chromeos/events/event_rewriter.h
+++ b/ui/chromeos/events/event_rewriter.h
@@ -2,31 +2,31 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef CHROME_BROWSER_CHROMEOS_EVENTS_EVENT_REWRITER_H_
-#define CHROME_BROWSER_CHROMEOS_EVENTS_EVENT_REWRITER_H_
+#ifndef UI_CHROMEOS_EVENTS_EVENT_REWRITER_H_
+#define UI_CHROMEOS_EVENTS_EVENT_REWRITER_H_
 
 #include <map>
 #include <memory>
 #include <set>
 #include <string>
 
-#include "base/compiler_specific.h"
-#include "base/containers/hash_tables.h"
 #include "base/macros.h"
+#include "ui/chromeos/ui_chromeos_export.h"
 #include "ui/events/event.h"
 #include "ui/events/event_rewriter.h"
 #include "ui/events/keycodes/dom/dom_key.h"
 
-class PrefService;
+namespace chromeos {
+namespace input_method {
+class ImeKeyboard;
+}  // namespace input_method
+}  // namespace chromeos
 
 namespace ui {
+
 enum class DomCode;
-};
 
 namespace chromeos {

[sadrul, 2017/03/14 01:15:41]

It doesn't look like any other code in //ui/chromeos uses namespace chromeos. So
let's remove this nested namespace?

[Peng, 2017/03/14 16:06:52]

Without it, ui::chromeos::EventRewriter will conflict with ui::EventRewriter.

[sadrul, 2017/03/16 05:02:43]

Can you rename it EventRewriterChromeOS instead, then?

[Peng, 2017/03/16 15:44:58]

Done.

-namespace input_method {
-class ImeKeyboard;
-}
 
 // EventRewriter makes various changes to keyboard-related events,
 // including KeyEvents and some other events with keyboard modifier flags:
@@ -38,7 +38,7 @@ class ImeKeyboard;
 // - handles various key combinations like Search+Backspace -> Delete
 //   and Search+number to Fnumber;
 // - handles key/pointer combinations like Alt+Button1 -> Button3.
-class EventRewriter : public ui::EventRewriter {
+class UI_CHROMEOS_EXPORT EventRewriter : public ui::EventRewriter {
  public:
   enum DeviceType {
     kDeviceUnknown = 0,
@@ -55,10 +55,83 @@ class EventRewriter : public ui::EventRewriter {
     ui::KeyboardCode key_code;
   };
 
-  // Does not take ownership of the |sticky_keys_controller|, which may also
-  // be NULL (for testing without ash), in which case sticky key operations
+  class StickyKeysController {

[sadrul, 2017/03/14 01:15:41]

I thought we can get rid of this now?

[Peng, 2017/03/14 16:06:52]

Oops! Forgot remove it. Done

+   public:
+    StickyKeysController() {}
+    virtual ~StickyKeysController() {}
+
+    // Handles keyboard event. Returns an |EventRewriteStatus|, and may
+    // modify |flags|:
+    // - Returns ui::EVENT_REWRITE_DISCARD, and leaves |flags| untouched,
+    //   if the event is consumed (i.e. a sticky modifier press or release);
+    // - Returns ui::EVENT_REWRITE_REWRITTEN if the event needs to be modified
+    //   according to the returned |flags| (i.e. a sticky-modified key);
+    // - Returns ui::EVENT_REWRITE_DISPATCH_ANOTHER if the event needs to be
+    //   modified according to the returned |flags|, and there are delayed
+    //   modifier-up   events now to be retrieved using |NextDispatchEvent()|
+    //   (i.e. a sticky-modified key that ends a sticky state);
+    // - Otherwise returns ui::EVENT_REWRITE_CONTINUE and leaves |flags|
+    //   unchanged.
+    virtual ui::EventRewriteStatus RewriteKeyEvent(const ui::KeyEvent& event,
+                                                   ui::KeyboardCode key_code,
+                                                   int* flags) = 0;
+
+    // Handles mouse event.
+    virtual ui::EventRewriteStatus RewriteMouseEvent(
+        const ui::MouseEvent& event,
+        int* flags) = 0;
+
+    // Handles scroll event.
+    virtual ui::EventRewriteStatus RewriteScrollEvent(
+        const ui::ScrollEvent& event,
+        int* flags) = 0;
+
+    // Obtains a pending modifier-up event. If the immediately previous
+    // call to |Rewrite...Event()| or |NextDispatchEvent()| returned
+    // ui::EVENT_REWRITE_DISPATCH_ANOTHER, this sets |new_event| and returns:
+    // - ui::EVENT_REWRITE_DISPATCH_ANOTHER if there is at least one more
+    //   pending modifier-up event;
+    // - ui::EVENT_REWRITE_REWRITE if this is the last or only modifier-up
+    // event; Otherwise, there is no pending modifier-up event, and this
+    // function returns ui::EVENT_REWRITE_CONTINUE and sets |new_event| to NULL.
+    virtual ui::EventRewriteStatus NextDispatchEvent(
+        std::unique_ptr<ui::Event>* new_event) = 0;
+
+   private:
+    DISALLOW_COPY_AND_ASSIGN(StickyKeysController);
+  };
+
+  class Delegate {
+   public:
+    Delegate() {}
+    virtual ~Delegate() {}
+
+    virtual bool RewriteModifierKeys() = 0;

[sadrul, 2017/03/14 01:15:41]

Document.

[Peng, 2017/03/14 16:06:52]

Done.

+
+    // Returns true if get keyboard remapped preference value successfully and
+    // the value will be stored in |value|.
+    virtual bool GetKeyboardRemappedPrefValue(const std::string& pref_name,
+                                              int* value) const = 0;
+
+    // Returns true if the target would prefer to receive raw
+    // function keys instead of having them rewritten into back, forward,
+    // brightness, volume, etc. or if the user has specified that they desire
+    // top-row keys to be treated as function keys globally.
+    virtual bool TopRowKeysAreFunctionKeys() const = 0;
+
+    // Retunrs true if the |key_code| and |flags| have been resgistered for

[sadrul, 2017/03/14 01:15:41]

Returns

If |key_code| and |flags| are registered by extensions, does that mean the event
is not rewritten?

[Peng, 2017/03/14 16:06:52]

Yes. If key_code and flags combination has been registered by extension, the
rewriter will not rewrite the event.

+    // extensions.
+    virtual bool IsExtensionCommandRegistered(ui::KeyboardCode key_code,
+                                              int flags) const = 0;
+
+   private:
+    DISALLOW_COPY_AND_ASSIGN(Delegate);
+  };
+
+  // Does not take ownership of the |sticky_keys_controller|, which may also be
+  // nullptr (for testing without ash), in which case sticky key operations
   // don't happen.
-  explicit EventRewriter(ui::EventRewriter* sticky_keys_controller);
+  EventRewriter(Delegate* delegate, ui::EventRewriter* sticky_keys_controller);
   ~EventRewriter() override;
 
   // Calls KeyboardDeviceAddedInternal.
@@ -76,11 +149,8 @@ class EventRewriter : public ui::EventRewriter {
   void set_last_keyboard_device_id_for_testing(int device_id) {
     last_keyboard_device_id_ = device_id;
   }
-  void set_pref_service_for_testing(const PrefService* pref_service) {
-    pref_service_for_testing_ = pref_service;
-  }
   void set_ime_keyboard_for_testing(
-      chromeos::input_method::ImeKeyboard* ime_keyboard) {
+      ::chromeos::input_method::ImeKeyboard* ime_keyboard) {
     ime_keyboard_for_testing_ = ime_keyboard;
   }
 
@@ -102,9 +172,6 @@ class EventRewriter : public ui::EventRewriter {
  private:
   void DeviceKeyPressedOrReleased(int device_id);
 
-  // Returns the PrefService that should be used.
-  const PrefService* GetPrefService() const;
-
   // Adds a device to |device_id_to_type_|.
   DeviceType KeyboardDeviceAdded(int device_id);
 
@@ -122,16 +189,9 @@ class EventRewriter : public ui::EventRewriter {
   // Returns true if |last_keyboard_device_id_| is of given |device_type|.
   bool IsLastKeyboardOfType(DeviceType device_type) const;
 
-  // Returns true if the target for |event| would prefer to receive raw function
-  // keys instead of having them rewritten into back, forward, brightness,
-  // volume, etc. or if the user has specified that they desire top-row keys to
-  // be treated as function keys globally.
-  bool TopRowKeysAreFunctionKeys(const ui::KeyEvent& event) const;
-
   // Given modifier flags |original_flags|, returns the remapped modifiers
   // according to user preferences and/or event properties.
-  int GetRemappedModifierMasks(const PrefService& pref_service,
-                               const ui::Event& event,
+  int GetRemappedModifierMasks(const ui::Event& event,
                                int original_flags) const;
 
   // Rewrite a particular kind of event.
@@ -170,8 +230,9 @@ class EventRewriter : public ui::EventRewriter {
   // used to interpret modifiers on pointer events.
   int last_keyboard_device_id_;
 
-  chromeos::input_method::ImeKeyboard* ime_keyboard_for_testing_;
-  const PrefService* pref_service_for_testing_;
+  ::chromeos::input_method::ImeKeyboard* ime_keyboard_for_testing_;
+
+  Delegate* const delegate_;
 
   // The sticky keys controller is not owned here;
   // at time of writing it is a singleton in ash::Shell.
@@ -201,5 +262,6 @@ class EventRewriter : public ui::EventRewriter {
 };
 
 }  // namespace chromeos
+}  // namespace ui
 
-#endif  // CHROME_BROWSER_CHROMEOS_EVENTS_EVENT_REWRITER_H_
+#endif  // UI_CHROMEOS_EVENTS_EVENT_REWRITER_H_