| Index: ui/events/devices/x11/device_data_manager_x11_unittest.cc
|
| diff --git a/ui/events/devices/x11/device_data_manager_x11_unittest.cc b/ui/events/devices/x11/device_data_manager_x11_unittest.cc
|
| index 444611f4fcc6b25da67d9168c54d9ba1020fce74..3cce253f3454467c9f46ba8688973e0085dc338d 100644
|
| --- a/ui/events/devices/x11/device_data_manager_x11_unittest.cc
|
| +++ b/ui/events/devices/x11/device_data_manager_x11_unittest.cc
|
| @@ -78,9 +78,9 @@ TEST_F(DeviceDataManagerX11Test, NotifyOnDisable) {
|
| TestInputDeviceObserver observer(manager);
|
| std::vector<ui::KeyboardDevice> keyboards;
|
| keyboards.push_back(
|
| - ui::KeyboardDevice(1u, ui::InputDeviceType::INPUT_DEVICE_INTERNAL));
|
| + ui::KeyboardDevice(1, ui::InputDeviceType::INPUT_DEVICE_INTERNAL));
|
| keyboards.push_back(
|
| - ui::KeyboardDevice(2u, ui::InputDeviceType::INPUT_DEVICE_INTERNAL));
|
| + ui::KeyboardDevice(2, ui::InputDeviceType::INPUT_DEVICE_INTERNAL));
|
| SetKeyboardDevices(keyboards);
|
| EXPECT_TRUE(observer.change_notified());
|
| std::vector<KeyboardDevice> devices = manager->keyboard_devices();
|
| @@ -88,16 +88,16 @@ TEST_F(DeviceDataManagerX11Test, NotifyOnDisable) {
|
| observer.Reset();
|
| // Disable the device, should be notified that the device list contains one
|
| // less device.
|
| - manager->DisableDevice(2u);
|
| + manager->DisableDevice(2);
|
| EXPECT_TRUE(observer.change_notified());
|
| devices = manager->keyboard_devices();
|
| EXPECT_EQ(1u, devices.size());
|
| KeyboardDevice device = devices.front();
|
| - EXPECT_EQ(1u, device.id);
|
| + EXPECT_EQ(1, device.id);
|
| observer.Reset();
|
| // Reenable the device, should be notified that the device list contains one
|
| // more device.
|
| - manager->EnableDevice(2u);
|
| + manager->EnableDevice(2);
|
| EXPECT_TRUE(observer.change_notified());
|
| devices = manager->keyboard_devices();
|
| EXPECT_EQ(keyboards.size(), devices.size());
|
| @@ -109,9 +109,9 @@ TEST_F(DeviceDataManagerX11Test, TestMultipleDisable) {
|
| TestInputDeviceObserver observer(manager);
|
| std::vector<ui::KeyboardDevice> keyboards;
|
| keyboards.push_back(
|
| - ui::KeyboardDevice(1u, ui::InputDeviceType::INPUT_DEVICE_INTERNAL));
|
| + ui::KeyboardDevice(1, ui::InputDeviceType::INPUT_DEVICE_INTERNAL));
|
| keyboards.push_back(
|
| - ui::KeyboardDevice(2u, ui::InputDeviceType::INPUT_DEVICE_INTERNAL));
|
| + ui::KeyboardDevice(2, ui::InputDeviceType::INPUT_DEVICE_INTERNAL));
|
| SetKeyboardDevices(keyboards);
|
| EXPECT_TRUE(observer.change_notified());
|
| std::vector<KeyboardDevice> devices = manager->keyboard_devices();
|
| @@ -119,25 +119,25 @@ TEST_F(DeviceDataManagerX11Test, TestMultipleDisable) {
|
| observer.Reset();
|
| // Disable the device, should be notified that the device list contains one
|
| // less device.
|
| - manager->DisableDevice(1u);
|
| + manager->DisableDevice(1);
|
| EXPECT_TRUE(observer.change_notified());
|
| devices = manager->keyboard_devices();
|
| EXPECT_EQ(1u, devices.size());
|
| observer.Reset();
|
| // Disable the second device, should be notified that the device list empty.
|
| - manager->DisableDevice(2u);
|
| + manager->DisableDevice(2);
|
| EXPECT_TRUE(observer.change_notified());
|
| devices = manager->keyboard_devices();
|
| EXPECT_EQ(0u, devices.size());
|
| observer.Reset();
|
| // Enable the first device, should be notified that one device present.
|
| - manager->EnableDevice(1u);
|
| + manager->EnableDevice(1);
|
| EXPECT_TRUE(observer.change_notified());
|
| devices = manager->keyboard_devices();
|
| EXPECT_EQ(1u, devices.size());
|
| observer.Reset();
|
| // Enable the second device, should be notified that both devices present.
|
| - manager->EnableDevice(2u);
|
| + manager->EnableDevice(2);
|
| EXPECT_TRUE(observer.change_notified());
|
| devices = manager->keyboard_devices();
|
| EXPECT_EQ(2u, devices.size());
|
| @@ -148,16 +148,16 @@ TEST_F(DeviceDataManagerX11Test, UnblockOnDeviceUnplugged) {
|
| TestInputDeviceObserver observer(manager);
|
| std::vector<ui::KeyboardDevice> all_keyboards;
|
| all_keyboards.push_back(
|
| - ui::KeyboardDevice(1u, ui::InputDeviceType::INPUT_DEVICE_INTERNAL));
|
| + ui::KeyboardDevice(1, ui::InputDeviceType::INPUT_DEVICE_INTERNAL));
|
| all_keyboards.push_back(
|
| - ui::KeyboardDevice(2u, ui::InputDeviceType::INPUT_DEVICE_INTERNAL));
|
| + ui::KeyboardDevice(2, ui::InputDeviceType::INPUT_DEVICE_INTERNAL));
|
| SetKeyboardDevices(all_keyboards);
|
| EXPECT_TRUE(observer.change_notified());
|
| std::vector<KeyboardDevice> devices = manager->keyboard_devices();
|
| EXPECT_EQ(all_keyboards.size(), devices.size());
|
| observer.Reset();
|
| // Expect to be notified that the device is no longer available.
|
| - manager->DisableDevice(2u);
|
| + manager->DisableDevice(2);
|
| EXPECT_TRUE(observer.change_notified());
|
| devices = manager->keyboard_devices();
|
| EXPECT_EQ(1u, devices.size());
|
| @@ -166,7 +166,7 @@ TEST_F(DeviceDataManagerX11Test, UnblockOnDeviceUnplugged) {
|
| // did not change.
|
| std::vector<ui::KeyboardDevice> subset_keyboards;
|
| subset_keyboards.push_back(
|
| - ui::KeyboardDevice(1u, ui::InputDeviceType::INPUT_DEVICE_INTERNAL));
|
| + ui::KeyboardDevice(1, ui::InputDeviceType::INPUT_DEVICE_INTERNAL));
|
| SetKeyboardDevices(subset_keyboards);
|
| EXPECT_FALSE(observer.change_notified());
|
| // Replug in the first device. Should be notified of the new device.
|
|
|