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

Unified Diff: ash/touch/touchscreen_util_unittest.cc

Issue 2270553002: Move ash::DisplayInfo to ui (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebased 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
« no previous file with comments | « ash/touch/touchscreen_util.cc ('k') | ash/wallpaper/wallpaper_controller.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/touch/touchscreen_util_unittest.cc
diff --git a/ash/touch/touchscreen_util_unittest.cc b/ash/touch/touchscreen_util_unittest.cc
index 3610f25ad25ca645ab4fba1230290a530954c094..f8b638e5cf8163450c5444c628ee912352784c8f 100644
--- a/ash/touch/touchscreen_util_unittest.cc
+++ b/ash/touch/touchscreen_util_unittest.cc
@@ -5,15 +5,17 @@
#include <string>
#include <vector>
-#include "ash/common/display/display_info.h"
#include "ash/test/ash_test_base.h"
#include "ash/test/display_manager_test_api.h"
#include "ash/touch/touchscreen_util.h"
#include "testing/gtest/include/gtest/gtest.h"
+#include "ui/display/manager/managed_display_info.h"
#include "ui/events/devices/input_device.h"
namespace ash {
+using DisplayInfoList = std::vector<display::ManagedDisplayInfo>;
+
class TouchscreenUtilTest : public test::AshTestBase {
public:
TouchscreenUtilTest() {}
@@ -25,41 +27,45 @@ class TouchscreenUtilTest : public test::AshTestBase {
// touchscreen can't be detected, it is then associated to a touch screen
// with matching size.
{
- DisplayInfo display(1, "1", false);
- scoped_refptr<ManagedDisplayMode> mode(new ManagedDisplayMode(
- gfx::Size(1920, 1080), 60.0, false /* interlaced */,
- true /* native */, 1.0 /* ui_scale */,
- 1.0 /* device_scale_factor */));
- DisplayInfo::ManagedDisplayModeList modes(1, mode);
+ display::ManagedDisplayInfo display(1, "1", false);
+ scoped_refptr<display::ManagedDisplayMode> mode(
+ new display::ManagedDisplayMode(gfx::Size(1920, 1080), 60.0,
+ false /* interlaced */,
+ true /* native */, 1.0 /* ui_scale */,
+ 1.0 /* device_scale_factor */));
+ display::ManagedDisplayInfo::ManagedDisplayModeList modes(1, mode);
display.SetManagedDisplayModes(modes);
displays_.push_back(display);
}
{
- DisplayInfo display(2, "2", false);
-
- scoped_refptr<ManagedDisplayMode> mode(new ManagedDisplayMode(
- gfx::Size(800, 600), 60.0, false /* interlaced */, true /* native */,
- 1.0 /* ui_scale */, 1.0 /* device_scale_factor */));
- DisplayInfo::ManagedDisplayModeList modes(1, mode);
+ display::ManagedDisplayInfo display(2, "2", false);
+
+ scoped_refptr<display::ManagedDisplayMode> mode(
+ new display::ManagedDisplayMode(gfx::Size(800, 600), 60.0,
+ false /* interlaced */,
+ true /* native */, 1.0 /* ui_scale */,
+ 1.0 /* device_scale_factor */));
+ display::ManagedDisplayInfo::ManagedDisplayModeList modes(1, mode);
display.SetManagedDisplayModes(modes);
displays_.push_back(display);
}
// Display without native mode. Must not be matched to any touch screen.
{
- DisplayInfo display(3, "3", false);
+ display::ManagedDisplayInfo display(3, "3", false);
displays_.push_back(display);
}
{
- DisplayInfo display(4, "4", false);
-
- scoped_refptr<ManagedDisplayMode> mode(new ManagedDisplayMode(
- gfx::Size(1024, 768), 60.0, false /* interlaced */,
- /* native */ true, 1.0 /* ui_scale */,
- 1.0 /* device_scale_factor */));
- DisplayInfo::ManagedDisplayModeList modes(1, mode);
+ display::ManagedDisplayInfo display(4, "4", false);
+
+ scoped_refptr<display::ManagedDisplayMode> mode(
+ new display::ManagedDisplayMode(gfx::Size(1024, 768), 60.0,
+ false /* interlaced */,
+ /* native */ true, 1.0 /* ui_scale */,
+ 1.0 /* device_scale_factor */));
+ display::ManagedDisplayInfo::ManagedDisplayModeList modes(1, mode);
display.SetManagedDisplayModes(modes);
displays_.push_back(display);
}
@@ -71,7 +77,7 @@ class TouchscreenUtilTest : public test::AshTestBase {
}
protected:
- std::vector<DisplayInfo> displays_;
+ DisplayInfoList displays_;
private:
DISALLOW_COPY_AND_ASSIGN(TouchscreenUtilTest);
@@ -97,7 +103,7 @@ TEST_F(TouchscreenUtilTest, ManyTouchscreens) {
gfx::Size(256, 256), 0));
}
- std::vector<DisplayInfo> displays;
+ DisplayInfoList displays;
displays.push_back(displays_[3]);
test::ScopedSetInternalDisplayId set_internal(displays_[0].id());
@@ -172,7 +178,7 @@ TEST_F(TouchscreenUtilTest, MapWhenSizesDoNotMatch) {
ui::TouchscreenDevice(2, ui::InputDeviceType::INPUT_DEVICE_EXTERNAL, "2",
gfx::Size(802, 600), 0));
- std::vector<DisplayInfo> displays;
+ DisplayInfoList displays;
displays.push_back(displays_[0]);
displays.push_back(displays_[1]);
@@ -194,7 +200,7 @@ TEST_F(TouchscreenUtilTest, MapInternalTouchscreen) {
ui::TouchscreenDevice(2, ui::InputDeviceType::INPUT_DEVICE_INTERNAL, "2",
gfx::Size(9999, 888), 0));
- std::vector<DisplayInfo> displays;
+ DisplayInfoList displays;
displays.push_back(displays_[0]);
displays.push_back(displays_[1]);
« no previous file with comments | « ash/touch/touchscreen_util.cc ('k') | ash/wallpaper/wallpaper_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698