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

Unified Diff: ash/touch/touch_transformer_controller_unittest.cc

Issue 618283003: Adds special support to the device manager for keyboards devices. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase to master. 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
« no previous file with comments | « ash/touch/touch_transformer_controller.cc ('k') | ash/touch/touchscreen_util.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/touch/touch_transformer_controller_unittest.cc
diff --git a/ash/touch/touch_transformer_controller_unittest.cc b/ash/touch/touch_transformer_controller_unittest.cc
index cff2c5356e8f481bd2c64a505374a39cb0ce1518..bd8f4548abd726a021193f4b508e2c2658182e19 100644
--- a/ash/touch/touch_transformer_controller_unittest.cc
+++ b/ash/touch/touch_transformer_controller_unittest.cc
@@ -15,7 +15,7 @@ namespace ash {
namespace {
DisplayInfo CreateDisplayInfo(int64 id,
- int touch_device_id,
+ unsigned int touch_device_id,
const gfx::Rect& bounds) {
DisplayInfo info(id, std::string(), false);
info.SetBounds(bounds);
@@ -31,7 +31,7 @@ TEST_F(TouchTransformerControllerTest, TouchTransformerMirrorModeLetterboxing) {
// mirror mode it is configured as 1920x1200. This is in letterboxing
// mode.
DisplayInfo internal_display_info =
- CreateDisplayInfo(1, 10, gfx::Rect(0, 0, 1920, 1200));
+ CreateDisplayInfo(1, 10u, gfx::Rect(0, 0, 1920, 1200));
std::vector<DisplayMode> internal_modes;
internal_modes.push_back(
DisplayMode(gfx::Size(2560, 1700), 60, false, true));
@@ -40,7 +40,7 @@ TEST_F(TouchTransformerControllerTest, TouchTransformerMirrorModeLetterboxing) {
internal_display_info.set_display_modes(internal_modes);
DisplayInfo external_display_info =
- CreateDisplayInfo(2, 11, gfx::Rect(0, 0, 1920, 1200));
+ CreateDisplayInfo(2, 11u, gfx::Rect(0, 0, 1920, 1200));
TouchTransformerController* tt_controller =
Shell::GetInstance()->touch_transformer_controller();
@@ -154,8 +154,9 @@ TEST_F(TouchTransformerControllerTest, TouchTransformerExtendedMode) {
// where 2428 = 768 + 60 (hidden gap) + 1600
// and the sceond monitor is translated to Point (0, 828) in the
// framebuffer.
- DisplayInfo display1 = CreateDisplayInfo(1, 5, gfx::Rect(0, 0, 1366, 768));
- DisplayInfo display2 = CreateDisplayInfo(2, 6, gfx::Rect(0, 828, 2560, 1600));
+ DisplayInfo display1 = CreateDisplayInfo(1, 5u, gfx::Rect(0, 0, 1366, 768));
+ DisplayInfo display2 =
+ CreateDisplayInfo(2, 6u, gfx::Rect(0, 828, 2560, 1600));
gfx::Size fb_size(2560, 2428);
TouchTransformerController* tt_controller =
@@ -205,7 +206,7 @@ TEST_F(TouchTransformerControllerTest, TouchTransformerExtendedMode) {
}
TEST_F(TouchTransformerControllerTest, TouchRadiusScale) {
- DisplayInfo display = CreateDisplayInfo(1, 5, gfx::Rect(0, 0, 2560, 1600));
+ DisplayInfo display = CreateDisplayInfo(1, 5u, gfx::Rect(0, 0, 2560, 1600));
std::vector<unsigned int> devices;
devices.push_back(5);
ui::SetUpTouchDevicesForTest(devices);
« no previous file with comments | « ash/touch/touch_transformer_controller.cc ('k') | ash/touch/touchscreen_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698