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

Unified Diff: mojo/services/window_manager/native_viewport_event_dispatcher_impl.cc

Issue 685013002: Refactors event dispatching of NativeViewport into its own interface (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: cleanup Created 6 years, 2 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: mojo/services/window_manager/native_viewport_event_dispatcher_impl.cc
diff --git a/mojo/services/window_manager/window_manager_internal_service_impl.cc b/mojo/services/window_manager/native_viewport_event_dispatcher_impl.cc
similarity index 54%
rename from mojo/services/window_manager/window_manager_internal_service_impl.cc
rename to mojo/services/window_manager/native_viewport_event_dispatcher_impl.cc
index 2e321b8d8e77b39259660d7ac2d10ec1f66b3d69..30764b402d32daca7d36bfb0620ae4e4eb26eb4c 100644
--- a/mojo/services/window_manager/window_manager_internal_service_impl.cc
+++ b/mojo/services/window_manager/native_viewport_event_dispatcher_impl.cc
@@ -2,31 +2,28 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "mojo/services/window_manager/window_manager_internal_service_impl.h"
+#include "mojo/services/window_manager/native_viewport_event_dispatcher_impl.h"
#include "mojo/aura/window_tree_host_mojo.h"
#include "mojo/converters/input_events/input_events_type_converters.h"
#include "mojo/services/window_manager/window_manager_app.h"
-#include "mojo/services/window_manager/window_manager_delegate.h"
namespace mojo {
-WindowManagerInternalServiceImpl::WindowManagerInternalServiceImpl(
+NativeViewportEventDispatcherImpl::NativeViewportEventDispatcherImpl(
WindowManagerApp* app)
: app_(app) {
}
-
-WindowManagerInternalServiceImpl::~WindowManagerInternalServiceImpl() {
+NativeViewportEventDispatcherImpl::~NativeViewportEventDispatcherImpl() {
}
-void WindowManagerInternalServiceImpl::OnViewInputEvent(mojo::EventPtr event) {
+void NativeViewportEventDispatcherImpl::OnEvent(
+ mojo::EventPtr event,
+ const mojo::Callback<void()>& callback) {
scoped_ptr<ui::Event> ui_event = event.To<scoped_ptr<ui::Event>>();
if (ui_event)
app_->host()->SendEventToProcessor(ui_event.get());
-}
-
-void WindowManagerInternalServiceImpl::OnConnectionEstablished() {
- app_->set_window_manager_client(client());
+ callback.Run();
}
} // namespace mojo

Powered by Google App Engine
This is Rietveld 408576698