Index: ash/display/display_manager_unittest.cc |
diff --git a/ash/display/display_manager_unittest.cc b/ash/display/display_manager_unittest.cc |
index 9156c20f23731587e8f9d1febc51bf940b609fdf..51ccad21711b47eef65d948cfb68127a28099cf5 100644 |
--- a/ash/display/display_manager_unittest.cc |
+++ b/ash/display/display_manager_unittest.cc |
@@ -720,9 +720,9 @@ TEST_F(DisplayManagerTest, DontRememberBestResolution) { |
DisplayInfo native_display_info = |
CreateDisplayInfo(display_id, gfx::Rect(0, 0, 1000, 500)); |
std::vector<Resolution> resolutions; |
- resolutions.push_back(Resolution(gfx::Size(1000, 500), false)); |
- resolutions.push_back(Resolution(gfx::Size(800, 300), false)); |
- resolutions.push_back(Resolution(gfx::Size(400, 500), false)); |
+ resolutions.push_back(Resolution(gfx::Size(1000, 500), 58.0f, false)); |
+ resolutions.push_back(Resolution(gfx::Size(800, 300), 59.0f, false)); |
+ resolutions.push_back(Resolution(gfx::Size(400, 500), 60.0f, false)); |
native_display_info.set_resolutions(resolutions); |
@@ -731,24 +731,32 @@ TEST_F(DisplayManagerTest, DontRememberBestResolution) { |
display_manager()->OnNativeDisplaysChanged(display_info_list); |
gfx::Size selected; |
- EXPECT_FALSE(display_manager()->GetSelectedResolutionForDisplayId( |
- display_id, &selected)); |
+ float refresh_rate = 0.0f; |
+ EXPECT_TRUE(display_manager()->GetSelectedResolutionForDisplayId( |
+ display_id, &selected, &refresh_rate)); |
+ EXPECT_EQ("1000x500", selected.ToString()); |
+ EXPECT_EQ(58.0f, refresh_rate); |
// Unsupported resolution. |
display_manager()->SetDisplayResolution(display_id, gfx::Size(800, 4000)); |
- EXPECT_FALSE(display_manager()->GetSelectedResolutionForDisplayId( |
- display_id, &selected)); |
+ EXPECT_TRUE(display_manager()->GetSelectedResolutionForDisplayId( |
+ display_id, &selected, &refresh_rate)); |
+ EXPECT_EQ("1000x500", selected.ToString()); |
+ EXPECT_EQ(58.0f, refresh_rate); |
// Supported resolution. |
display_manager()->SetDisplayResolution(display_id, gfx::Size(800, 300)); |
EXPECT_TRUE(display_manager()->GetSelectedResolutionForDisplayId( |
- display_id, &selected)); |
+ display_id, &selected, &refresh_rate)); |
EXPECT_EQ("800x300", selected.ToString()); |
+ EXPECT_EQ(59.0f, refresh_rate); |
// Best resolution. |
display_manager()->SetDisplayResolution(display_id, gfx::Size(1000, 500)); |
- EXPECT_FALSE(display_manager()->GetSelectedResolutionForDisplayId( |
- display_id, &selected)); |
+ EXPECT_TRUE(display_manager()->GetSelectedResolutionForDisplayId( |
+ display_id, &selected, &refresh_rate)); |
+ EXPECT_EQ("1000x500", selected.ToString()); |
+ EXPECT_EQ(58.0f, refresh_rate); |
} |
TEST_F(DisplayManagerTest, ResolutionFallback) { |
@@ -756,9 +764,9 @@ TEST_F(DisplayManagerTest, ResolutionFallback) { |
DisplayInfo native_display_info = |
CreateDisplayInfo(display_id, gfx::Rect(0, 0, 1000, 500)); |
std::vector<Resolution> resolutions; |
- resolutions.push_back(Resolution(gfx::Size(1000, 500), false)); |
- resolutions.push_back(Resolution(gfx::Size(800, 300), false)); |
- resolutions.push_back(Resolution(gfx::Size(400, 500), false)); |
+ resolutions.push_back(Resolution(gfx::Size(1000, 500), 58.0f, false)); |
+ resolutions.push_back(Resolution(gfx::Size(800, 300), 59.0f, false)); |
+ resolutions.push_back(Resolution(gfx::Size(400, 500), 60.0f, false)); |
std::vector<Resolution> copy = resolutions; |
native_display_info.set_resolutions(copy); |
@@ -777,12 +785,14 @@ TEST_F(DisplayManagerTest, ResolutionFallback) { |
display_manager()->OnNativeDisplaysChanged(new_display_info_list); |
gfx::Size selected; |
+ float refresh_rate = 0.0f; |
EXPECT_TRUE(display_manager()->GetSelectedResolutionForDisplayId( |
- display_id, &selected)); |
+ display_id, &selected, &refresh_rate)); |
EXPECT_EQ("400x500", selected.ToString()); |
+ EXPECT_EQ(60.0f, refresh_rate); |
} |
{ |
- // Best resolution should not be set. |
+ // Best resolution should find itself on the resolutions list. |
display_manager()->SetDisplayResolution(display_id, gfx::Size(800, 300)); |
DisplayInfo new_native_display_info = |
CreateDisplayInfo(display_id, gfx::Rect(0, 0, 1000, 500)); |
@@ -793,8 +803,11 @@ TEST_F(DisplayManagerTest, ResolutionFallback) { |
display_manager()->OnNativeDisplaysChanged(new_display_info_list); |
gfx::Size selected; |
- EXPECT_FALSE(display_manager()->GetSelectedResolutionForDisplayId( |
- display_id, &selected)); |
+ float refresh_rate = 0.0f; |
+ EXPECT_TRUE(display_manager()->GetSelectedResolutionForDisplayId( |
+ display_id, &selected, &refresh_rate)); |
+ EXPECT_EQ("1000x500", selected.ToString()); |
+ EXPECT_EQ(58.0f, refresh_rate); |
} |
} |