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

Unified Diff: components/mus/public/cpp/input_devices/input_device_client.h

Issue 1992443002: Add Mojo IPC based input-device service. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@cleanup_ddm
Patch Set: Fixes for comments. Created 4 years, 6 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/cpp/input_devices/input_device_client.h
diff --git a/components/mus/public/cpp/input_devices/input_device_client.h b/components/mus/public/cpp/input_devices/input_device_client.h
new file mode 100644
index 0000000000000000000000000000000000000000..2691380d4a38a084fc6b0a7cced0ddd8f38646f6
--- /dev/null
+++ b/components/mus/public/cpp/input_devices/input_device_client.h
@@ -0,0 +1,86 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef COMPONENTS_MUS_PUBLIC_CPP_INPUT_DEVICES_INPUT_DEVICE_CLIENT_H_
+#define COMPONENTS_MUS_PUBLIC_CPP_INPUT_DEVICES_INPUT_DEVICE_CLIENT_H_
+
+#include <vector>
+
+#include "base/macros.h"
+#include "base/observer_list.h"
+#include "components/mus/public/interfaces/input_devices/input_devices.mojom.h"
+#include "mojo/public/cpp/bindings/array.h"
+#include "mojo/public/cpp/bindings/binding.h"
+#include "ui/events/devices/input_device.h"
+#include "ui/events/devices/input_device_event_observer.h"
+#include "ui/events/devices/input_device_manager.h"
+#include "ui/events/devices/touchscreen_device.h"
+
+namespace input_device {
+
+// Allows in-process client code to register as a ui:::InputDeviceEventObserver
+// and get information about input-devices. InputDeviceClient itself acts as an
+// InputDeviceObserverMojo and registers to get updates from InputDeviceServer.
+// Essentially, InputDeviceClient forwards input-device events and caches
+// input-device state.
+//
+// One instance of InputDeviceClient should be instantiated in each process that
+// doesn't have a DeviceDataManager. The Mojo app should call Connect() with the
+// mojo shell connector to register as an observer with mus. Client code should
+// use InputDeviceManager and doesn't need to be aware it's not dealing with
+// DeviceDataManager directly.
+class InputDeviceClient : public mojom::InputDeviceObserverMojo,
+ public ui::InputDeviceManager {
+ public:
+ InputDeviceClient();
+ ~InputDeviceClient() override;
+
+ // Connects to mojo:mus as an observer on InputDeviceServer to receive input
+ // device updates.
+ void Connect(mojom::InputDeviceServerPtr server);
+
+ // ui::InputDeviceManager:
sadrul 2016/06/08 16:31:09 Make these privates too?
kylechar 2016/06/08 18:13:16 They seem more appropriate as public. The class ex
+ const std::vector<ui::InputDevice>& GetKeyboardDevices() const override;
+ const std::vector<ui::TouchscreenDevice>& GetTouchscreenDevices()
+ const override;
+ const std::vector<ui::InputDevice>& GetMouseDevices() const override;
+ const std::vector<ui::InputDevice>& GetTouchpadDevices() const override;
+
+ bool AreDeviceListsComplete() const override;
+ bool AreTouchscreensEnabled() const override;
+
+ void AddObserver(ui::InputDeviceEventObserver* observer) override;
+ void RemoveObserver(ui::InputDeviceEventObserver* observer) override;
+
+ private:
+ // mojom::InputDeviceObserverMojo:
+ void OnKeyboardDeviceConfigurationChanged(
+ mojo::Array<ui::InputDevice> devices) override;
+ void OnTouchscreenDeviceConfigurationChanged(
+ mojo::Array<ui::TouchscreenDevice> devices) override;
+ void OnMouseDeviceConfigurationChanged(
+ mojo::Array<ui::InputDevice> devices) override;
+ void OnTouchpadDeviceConfigurationChanged(
+ mojo::Array<ui::InputDevice> devices) override;
+ void OnDeviceListsComplete() override;
+
+ mojo::Binding<mojom::InputDeviceObserverMojo> binding_;
+
+ // Holds the list of input devices and signal that we have received the lists
+ // after initialization.
+ std::vector<ui::InputDevice> keyboard_devices_;
+ std::vector<ui::TouchscreenDevice> touchscreen_devices_;
+ std::vector<ui::InputDevice> mouse_devices_;
+ std::vector<ui::InputDevice> touchpad_devices_;
+ bool device_lists_complete_ = false;
+
+ // List of observers in this process.
+ base::ObserverList<ui::InputDeviceEventObserver> observers_;
+
+ DISALLOW_COPY_AND_ASSIGN(InputDeviceClient);
+};
+
+} // namespace input_device
+
+#endif // COMPONENTS_MUS_PUBLIC_CPP_INPUT_DEVICES_INPUT_DEVICE_CLIENT_H_

Powered by Google App Engine
This is Rietveld 408576698