Add tests for InputDeviceServer/InputDeviceClient.

services/ui/public/cpp/input_devices/input_device_client.cc

 // 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.
 
 #include "services/ui/public/cpp/input_devices/input_device_client.h"
 
 #include "base/logging.h"
 
 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());
+}
 
-  server->AddObserver(binding_.CreateInterfacePtrAndBind());
+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);
+}
+
+InputDeviceClient::InputDeviceClient(bool is_input_device_manager)
+    : binding_(this) {

[sadrul, 2016/08/02 15:16:14]

Initialize |is_input_device_manager_| here

[kylechar, 2016/08/02 17:46:57]

Done.

+  if (is_input_device_manager) {
+    InputDeviceManager::SetInstance(this);
+    is_input_device_manager_ = true;
+  }
+}
+
+mojom::InputDeviceObserverMojoPtr InputDeviceClient::GetIntefacePtr() {
+  return binding_.CreateInterfacePtrAndBind();
 }
 
 void InputDeviceClient::OnKeyboardDeviceConfigurationChanged(
     const std::vector<ui::InputDevice>& devices) {
   keyboard_devices_ = devices;
   FOR_EACH_OBSERVER(ui::InputDeviceEventObserver, observers_,
                     OnKeyboardDeviceConfigurationChanged());
 }
 
 void InputDeviceClient::OnTouchscreenDeviceConfigurationChanged(
@@ skipping 32 matching lines @@
   if (!touchpad_devices.empty())
     OnTouchpadDeviceConfigurationChanged(touchpad_devices);
 
   if (!device_lists_complete_) {
     device_lists_complete_ = true;
     FOR_EACH_OBSERVER(ui::InputDeviceEventObserver, observers_,
                       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