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/chrome
Peng
2017/03/14 16:06:52
Without it, ui::chromeos::EventRewriter will confl
sadrul
2017/03/16 05:02:43
Can you rename it EventRewriterChromeOS instead, t
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
Peng
2017/03/14 16:06:52
Yes. If key_code and flags combination has been re
|
+ // 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_ |