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

Unified Diff: components/mus/public/interfaces/window_tree_host.mojom

Issue 1656123002: Moves accelerator registration to WindowManager (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix mandoline Created 4 years, 11 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: components/mus/public/interfaces/window_tree_host.mojom
diff --git a/components/mus/public/interfaces/window_tree_host.mojom b/components/mus/public/interfaces/window_tree_host.mojom
index cc29d711c25cefb6c585596e52798d8e25044e6a..63286bf63c20dba6db6d7203b66389eed3b9e25b 100644
--- a/components/mus/public/interfaces/window_tree_host.mojom
+++ b/components/mus/public/interfaces/window_tree_host.mojom
@@ -5,8 +5,6 @@
module mus.mojom;
import "components/mus/public/interfaces/input_event_constants.mojom";
-import "components/mus/public/interfaces/input_event_matcher.mojom";
-import "components/mus/public/interfaces/input_events.mojom";
import "components/mus/public/interfaces/input_key_codes.mojom";
import "components/mus/public/interfaces/window_tree.mojom";
import "ui/mojo/geometry/geometry.mojom";
@@ -20,15 +18,6 @@ interface WindowTreeHost {
// Sets a title string to be displayed on the platform window.
SetTitle(string title);
- // Add and remove accelerators. When accelerators are registered the
- // WindowTreeHostClient receives the event via OnAccelerator() rather than the
- // target window. The id is defined by the client and can be used to more
- // easily identify the accelerator's action. If an accelerator with the same
- // id or the same matcher already exists, then the accelerator is not added,
- // and returns false.
- AddAccelerator(uint32 id, EventMatcher matcher) => (bool success);
- RemoveAccelerator(uint32 id);
-
// Enables (or disables) child windows of |window_id| to be activated.
AddActivationParent(uint32 window_id);
RemoveActivationParent(uint32 window_id);
@@ -47,9 +36,8 @@ interface WindowTreeHost {
mojo.Insets hit_area);
};
+// TODO(sky): get rid of this.
interface WindowTreeHostClient {
- // An accelerator registered via AddAccelerator() has been triggered.
- OnAccelerator(uint32 id, Event event);
};
interface WindowTreeHostFactory {
« no previous file with comments | « components/mus/public/interfaces/window_manager.mojom ('k') | components/mus/ws/window_manager_client_apptest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698