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

Unified Diff: ui/display/win/screen_win_unittest.cc

Issue 2361283002: Add GetDisplayWithDisplayId to display::Screen. (Closed)
Patch Set: rebase Created 4 years, 1 month 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: ui/display/win/screen_win_unittest.cc
diff --git a/ui/display/win/screen_win_unittest.cc b/ui/display/win/screen_win_unittest.cc
index bf9552186175b576000a236b621e88339fd0c498..f5d2f76a5182b90729741166223dbc91599c93aa 100644
--- a/ui/display/win/screen_win_unittest.cc
+++ b/ui/display/win/screen_win_unittest.cc
@@ -353,6 +353,16 @@ TEST_F(ScreenWinTestSingleDisplay1x, GetDisplays) {
EXPECT_EQ(gfx::Rect(0, 0, 1920, 1100), displays[0].work_area());
}
+TEST_F(ScreenWinTestSingleDisplay1x, GetDisplayById) {
+ Screen* screen = GetScreen();
+ int64_t primary_display_id = screen->GetPrimaryDisplay().id();
+ Display display;
+ bool has_display =
+ screen->GetDisplayWithDisplayId(primary_display_id, &display);
+ EXPECT_TRUE(has_display);
sky 2016/11/19 15:01:40 no need for the temporary. This comment holds for
+ EXPECT_EQ(screen->GetPrimaryDisplay(), display);
+}
+
TEST_F(ScreenWinTestSingleDisplay1x, GetNumDisplays) {
EXPECT_EQ(1, GetScreen()->GetNumDisplays());
}

Powered by Google App Engine
This is Rietveld 408576698