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

Unified Diff: ash/display/display_color_manager_chromeos_unittest.cc

Issue 1945353004: Revert of ash: reset color management when new screens are hotplugged (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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/display/display_color_manager_chromeos.cc ('k') | ui/ozone/platform/drm/gpu/drm_device.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/display/display_color_manager_chromeos_unittest.cc
diff --git a/ash/display/display_color_manager_chromeos_unittest.cc b/ash/display/display_color_manager_chromeos_unittest.cc
index f6b745aeff7ae96550b84170b27c95c1ba723217..9766c10983a600dc75dd80f784994039198d1900 100644
--- a/ash/display/display_color_manager_chromeos_unittest.cc
+++ b/ash/display/display_color_manager_chromeos_unittest.cc
@@ -26,7 +26,6 @@
namespace {
-const char kResetGammaAction[] = "*set_color_correction(id=123)";
const char kSetGammaAction[] =
"set_color_correction(id=123,gamma[0]*gamma[255]=???????????\?)";
const char kSetFullCTMAction[] =
@@ -192,9 +191,8 @@
configurator_.OnConfigurationChanged();
EXPECT_TRUE(test_api_.TriggerConfigureTimeout());
- EXPECT_TRUE(
- base::MatchPattern(log_->GetActionsAndClear(), kResetGammaAction));
-
+
+ log_->GetActionsAndClear();
WaitOnColorCalibration();
EXPECT_TRUE(base::MatchPattern(log_->GetActionsAndClear(), kSetGammaAction));
}
@@ -216,12 +214,10 @@
outputs.push_back(&snapshot);
native_display_delegate_->set_outputs(outputs);
- log_->GetActionsAndClear();
- configurator_.OnConfigurationChanged();
- EXPECT_TRUE(test_api_.TriggerConfigureTimeout());
- EXPECT_TRUE(
- base::MatchPattern(log_->GetActionsAndClear(), kResetGammaAction));
-
+ configurator_.OnConfigurationChanged();
+ EXPECT_TRUE(test_api_.TriggerConfigureTimeout());
+
+ log_->GetActionsAndClear();
WaitOnColorCalibration();
EXPECT_TRUE(base::MatchPattern(log_->GetActionsAndClear(), kSetGammaAction));
}
@@ -245,9 +241,8 @@
configurator_.OnConfigurationChanged();
EXPECT_TRUE(test_api_.TriggerConfigureTimeout());
- EXPECT_TRUE(
- base::MatchPattern(log_->GetActionsAndClear(), kResetGammaAction));
-
+
+ log_->GetActionsAndClear();
WaitOnColorCalibration();
EXPECT_TRUE(
base::MatchPattern(log_->GetActionsAndClear(), kSetFullCTMAction));
@@ -272,9 +267,8 @@
configurator_.OnConfigurationChanged();
EXPECT_TRUE(test_api_.TriggerConfigureTimeout());
- EXPECT_TRUE(
- base::MatchPattern(log_->GetActionsAndClear(), kResetGammaAction));
-
+
+ log_->GetActionsAndClear();
WaitOnColorCalibration();
EXPECT_STREQ("", log_->GetActionsAndClear().c_str());
}
@@ -298,9 +292,8 @@
configurator_.OnConfigurationChanged();
EXPECT_TRUE(test_api_.TriggerConfigureTimeout());
- EXPECT_TRUE(
- base::MatchPattern(log_->GetActionsAndClear(), kResetGammaAction));
-
+
+ log_->GetActionsAndClear();
WaitOnColorCalibration();
EXPECT_STREQ("", log_->GetActionsAndClear().c_str());
}
@@ -324,9 +317,8 @@
configurator_.OnConfigurationChanged();
EXPECT_TRUE(test_api_.TriggerConfigureTimeout());
- EXPECT_TRUE(
- base::MatchPattern(log_->GetActionsAndClear(), kResetGammaAction));
-
+
+ log_->GetActionsAndClear();
WaitOnColorCalibration();
EXPECT_STREQ("", log_->GetActionsAndClear().c_str());
}
« no previous file with comments | « ash/display/display_color_manager_chromeos.cc ('k') | ui/ozone/platform/drm/gpu/drm_device.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698