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

Unified Diff: services/window_manager/window_manager_app.h

Issue 924653002: Use StrongBinding instead of InterfaceImpl in wm event dispatcher (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 10 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: services/window_manager/window_manager_app.h
diff --git a/services/window_manager/window_manager_app.h b/services/window_manager/window_manager_app.h
index ef8f555cc0d9239163c984a718232e2081ce44ea..a384600df415b93d85614f0d337851fb7d4dc7bd 100644
--- a/services/window_manager/window_manager_app.h
+++ b/services/window_manager/window_manager_app.h
@@ -55,6 +55,7 @@ class WindowManagerApp
public CaptureControllerObserver,
public mojo::InterfaceFactory<mojo::WindowManager>,
public mojo::InterfaceFactory<mojo::WindowManagerInternal>,
+ public mojo::InterfaceFactory<mojo::NativeViewportEventDispatcher>,
public mojo::WindowManagerInternal {
public:
WindowManagerApp(ViewManagerDelegate* view_manager_delegate,
@@ -153,6 +154,11 @@ class WindowManagerApp
void Create(mojo::ApplicationConnection* connection,
mojo::InterfaceRequest<mojo::WindowManager> request) override;
+ // InterfaceFactory<NativeViewportEventDispatcher>:
+ void Create(mojo::ApplicationConnection* connection,
+ mojo::InterfaceRequest<mojo::NativeViewportEventDispatcher>
+ request) override;
+
// WindowManagerInternal:
void CreateWindowManagerForViewManagerClient(
uint16_t connection_id,
@@ -162,10 +168,6 @@ class WindowManagerApp
mojo::Shell* shell_;
- mojo::InterfaceFactoryImplWithContext<NativeViewportEventDispatcherImpl,
- WindowManagerApp>
- native_viewport_event_dispatcher_factory_;
-
ViewManagerDelegate* wrapped_view_manager_delegate_;
WindowManagerDelegate* window_manager_delegate_;
« no previous file with comments | « services/window_manager/native_viewport_event_dispatcher_impl.cc ('k') | services/window_manager/window_manager_app.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698