Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(124)

Unified Diff: ui/wm/core/nested_accelerator_delegate.h

Issue 312483002: More accelerator code leanup (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/wm/core/nested_accelerator_delegate.h
diff --git a/ui/wm/core/nested_accelerator_delegate.h b/ui/wm/core/nested_accelerator_delegate.h
index 45b9e2af5adbe4f326f65cc89349958c68445450..4b13c0d882b9b506637a488cf2ee96d37b87cb26 100644
--- a/ui/wm/core/nested_accelerator_delegate.h
+++ b/ui/wm/core/nested_accelerator_delegate.h
@@ -6,7 +6,7 @@
#define UI_WM_CORE_NESTED_ACCELERATOR_DELEGATE_H_
namespace ui {
-class KeyEvent;
+class Accelerator;
}
namespace wm {
@@ -15,15 +15,18 @@ namespace wm {
// handling.
class NestedAcceleratorDelegate {
public:
- virtual ~NestedAcceleratorDelegate() {}
+ enum Result {
+ RESULT_PROCESSED,
+ RESULT_NOT_PROCESSED,
+ // The key event should be ignored now and instead be reposted so that
+ // next event loop.
+ RESULT_PROCESS_LATER,
+ };
- // If the key event should be ignored now and instead be reposted so that next
- // event loop.
- virtual bool ShouldProcessEventNow(const ui::KeyEvent& key_event) = 0;
+ virtual ~NestedAcceleratorDelegate() {}
- // Attempts to process an accelerator for the key-event.
- // Returns whether an accelerator was triggered and processed.
- virtual bool ProcessEvent(const ui::KeyEvent& key_event) = 0;
+ // Attempts to process the |accelerator|.
+ virtual Result ProcessAccelerator(const ui::Accelerator& accelerator) = 0;
};
} // namespace wm
« no previous file with comments | « ui/wm/core/nested_accelerator_controller_unittest.cc ('k') | ui/wm/core/nested_accelerator_dispatcher_linux.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698