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

Unified Diff: ui/display/chromeos/display_configurator_unittest.cc

Issue 667753002: Treat displays with and without EDID the same way (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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 | « ui/display/chromeos/display_configurator.cc ('k') | ui/display/chromeos/test/test_display_snapshot.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/display/chromeos/display_configurator_unittest.cc
diff --git a/ui/display/chromeos/display_configurator_unittest.cc b/ui/display/chromeos/display_configurator_unittest.cc
index aecaddd020cf6a50581d78c33e3e8462ce9f303d..0254d6a66c93e8c4147dc4de97f3599575b277b6 100644
--- a/ui/display/chromeos/display_configurator_unittest.cc
+++ b/ui/display/chromeos/display_configurator_unittest.cc
@@ -359,7 +359,6 @@ class DisplayConfiguratorTest : public testing::Test {
o->set_type(DISPLAY_CONNECTION_TYPE_INTERNAL);
o->set_is_aspect_preserving_scaling(true);
o->set_display_id(123);
- o->set_has_proper_display_id(true);
o = &outputs_[1];
o->set_current_mode(&big_mode_);
@@ -369,7 +368,6 @@ class DisplayConfiguratorTest : public testing::Test {
o->set_type(DISPLAY_CONNECTION_TYPE_HDMI);
o->set_is_aspect_preserving_scaling(true);
o->set_display_id(456);
- o->set_has_proper_display_id(true);
UpdateOutputs(2, false);
}
@@ -1004,18 +1002,7 @@ TEST_F(DisplayConfiguratorTest, InvalidMultipleDisplayStates) {
EXPECT_EQ(2, observer_.num_failures());
}
-TEST_F(DisplayConfiguratorTest, GetMultipleDisplayStateForDisplaysWithoutId) {
- outputs_[0].set_has_proper_display_id(false);
- UpdateOutputs(2, false);
- configurator_.Init(false);
- state_controller_.set_state(MULTIPLE_DISPLAY_STATE_DUAL_MIRROR);
- configurator_.ForceInitialConfigure(0);
- EXPECT_EQ(MULTIPLE_DISPLAY_STATE_DUAL_EXTENDED,
- configurator_.display_state());
-}
-
-TEST_F(DisplayConfiguratorTest, GetMultipleDisplayStateForDisplaysWithId) {
- outputs_[0].set_has_proper_display_id(true);
+TEST_F(DisplayConfiguratorTest, GetMultipleDisplayStateForMirroredDisplays) {
UpdateOutputs(2, false);
configurator_.Init(false);
state_controller_.set_state(MULTIPLE_DISPLAY_STATE_DUAL_MIRROR);
« no previous file with comments | « ui/display/chromeos/display_configurator.cc ('k') | ui/display/chromeos/test/test_display_snapshot.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698