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

Unified Diff: services/ui/public/cpp/input_devices/input_device_client.cc

Issue 2196563004: Add tests for InputDeviceServer/InputDeviceClient. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Switch test runner. Created 4 years, 4 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/ui/public/cpp/input_devices/input_device_client.cc
diff --git a/services/ui/public/cpp/input_devices/input_device_client.cc b/services/ui/public/cpp/input_devices/input_device_client.cc
index 4137e2b40ddc7c5511d98ee8bce7b63883adbc8e..ad52b7a5f539eac8d0a70ad1673f9e2f68680100 100644
--- a/services/ui/public/cpp/input_devices/input_device_client.cc
+++ b/services/ui/public/cpp/input_devices/input_device_client.cc
@@ -8,18 +8,63 @@
namespace ui {
-InputDeviceClient::InputDeviceClient() : binding_(this) {
- InputDeviceManager::SetInstance(this);
-}
+InputDeviceClient::InputDeviceClient() : InputDeviceClient(true) {}
InputDeviceClient::~InputDeviceClient() {
- InputDeviceManager::ClearInstance();
+ if (is_input_device_manager_)
+ InputDeviceManager::ClearInstance();
}
void InputDeviceClient::Connect(mojom::InputDeviceServerPtr server) {
DCHECK(server.is_bound());
+ server->AddObserver(GetIntefacePtr());
+}
+
+const std::vector<ui::InputDevice>& InputDeviceClient::GetKeyboardDevices()
+ const {
+ return keyboard_devices_;
+}
+
+const std::vector<ui::TouchscreenDevice>&
+InputDeviceClient::GetTouchscreenDevices() const {
+ return touchscreen_devices_;
+}
+
+const std::vector<ui::InputDevice>& InputDeviceClient::GetMouseDevices() const {
+ return mouse_devices_;
+}
+
+const std::vector<ui::InputDevice>& InputDeviceClient::GetTouchpadDevices()
+ const {
+ return touchpad_devices_;
+}
+
+bool InputDeviceClient::AreDeviceListsComplete() const {
+ return device_lists_complete_;
+}
+
+bool InputDeviceClient::AreTouchscreensEnabled() const {
+ // TODO(kylechar): This obviously isn't right. We either need to pass this
+ // state around or modify the interface.
+ return true;
+}
+
+void InputDeviceClient::AddObserver(ui::InputDeviceEventObserver* observer) {
+ observers_.AddObserver(observer);
+}
+
+void InputDeviceClient::RemoveObserver(ui::InputDeviceEventObserver* observer) {
+ observers_.RemoveObserver(observer);
+}
- server->AddObserver(binding_.CreateInterfacePtrAndBind());
+InputDeviceClient::InputDeviceClient(bool is_input_device_manager)
+ : binding_(this), is_input_device_manager_(is_input_device_manager) {
+ if (is_input_device_manager_)
+ InputDeviceManager::SetInstance(this);
+}
+
+mojom::InputDeviceObserverMojoPtr InputDeviceClient::GetIntefacePtr() {
+ return binding_.CreateInterfacePtrAndBind();
}
void InputDeviceClient::OnKeyboardDeviceConfigurationChanged(
@@ -72,41 +117,4 @@ void InputDeviceClient::OnDeviceListsComplete(
}
}
-void InputDeviceClient::AddObserver(ui::InputDeviceEventObserver* observer) {
- observers_.AddObserver(observer);
-}
-
-void InputDeviceClient::RemoveObserver(ui::InputDeviceEventObserver* observer) {
- observers_.RemoveObserver(observer);
-}
-
-const std::vector<ui::InputDevice>& InputDeviceClient::GetKeyboardDevices()
- const {
- return keyboard_devices_;
-}
-
-const std::vector<ui::TouchscreenDevice>&
-InputDeviceClient::GetTouchscreenDevices() const {
- return touchscreen_devices_;
-}
-
-const std::vector<ui::InputDevice>& InputDeviceClient::GetMouseDevices() const {
- return mouse_devices_;
-}
-
-const std::vector<ui::InputDevice>& InputDeviceClient::GetTouchpadDevices()
- const {
- return touchpad_devices_;
-}
-
-bool InputDeviceClient::AreDeviceListsComplete() const {
- return device_lists_complete_;
-}
-
-bool InputDeviceClient::AreTouchscreensEnabled() const {
- // TODO(kylechar): This obviously isn't right. We either need to pass this
- // state around or modify the interface.
- return true;
-}
-
} // namespace ui
« no previous file with comments | « services/ui/public/cpp/input_devices/input_device_client.h ('k') | testing/buildbot/chromium.chromiumos.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698