Commit 5796b828 authored by Miguel Casas's avatar Miguel Casas Committed by Commit Bot

Remove use-monitor-color-space flag

This CL removes the flag to disable this feature, since it
has been in Beta 66.0.3359.30 without any major problems
so far.

Bug: 771345
Change-Id: I7283cc05ded90ca3ec24ba227fc80654ad5f3325
Reviewed-on: https://chromium-review.googlesource.com/912016Reviewed-by: default avatarAvi Drissman <avi@chromium.org>
Commit-Queue: Miguel Casas <mcasas@chromium.org>
Cr-Commit-Position: refs/heads/master@{#544122}
parent 7f2fdcb1
...@@ -3124,10 +3124,6 @@ const FeatureEntry kFeatureEntries[] = { ...@@ -3124,10 +3124,6 @@ const FeatureEntry kFeatureEntries[] = {
MULTI_VALUE_TYPE(kForceColorProfileChoices)}, MULTI_VALUE_TYPE(kForceColorProfileChoices)},
#if defined(OS_CHROMEOS) #if defined(OS_CHROMEOS)
{"use-monitor-color-space", flag_descriptions::kUseMonitorColorSpaceName,
flag_descriptions::kUseMonitorColorSpaceDescription, kOsCrOS,
FEATURE_VALUE_TYPE(features::kUseMonitorColorSpace)},
{"quick-unlock-pin-signin", flag_descriptions::kQuickUnlockPinSignin, {"quick-unlock-pin-signin", flag_descriptions::kQuickUnlockPinSignin,
flag_descriptions::kQuickUnlockPinSigninDescription, kOsCrOS, flag_descriptions::kQuickUnlockPinSigninDescription, kOsCrOS,
FEATURE_VALUE_TYPE(features::kQuickUnlockPinSignin)}, FEATURE_VALUE_TYPE(features::kQuickUnlockPinSignin)},
......
...@@ -64,11 +64,4 @@ namespace features { ...@@ -64,11 +64,4 @@ namespace features {
const base::Feature kHighDynamicRange{"HighDynamicRange", const base::Feature kHighDynamicRange{"HighDynamicRange",
base::FEATURE_ENABLED_BY_DEFAULT}; base::FEATURE_ENABLED_BY_DEFAULT};
#if defined(OS_CHROMEOS)
// Enables using the monitor's provided color space information when rendering.
// TODO(mcasas): remove this flag http://crbug.com/771345.
const base::Feature kUseMonitorColorSpace{"UseMonitorColorSpace",
base::FEATURE_ENABLED_BY_DEFAULT};
#endif
} // namespace features } // namespace features
...@@ -34,10 +34,6 @@ namespace features { ...@@ -34,10 +34,6 @@ namespace features {
DISPLAY_EXPORT extern const base::Feature kHighDynamicRange; DISPLAY_EXPORT extern const base::Feature kHighDynamicRange;
#if defined(OS_CHROMEOS)
DISPLAY_EXPORT extern const base::Feature kUseMonitorColorSpace;
#endif
} // namespace features } // namespace features
#endif // UI_DISPLAY_DISPLAY_SWITCHES_H_ #endif // UI_DISPLAY_DISPLAY_SWITCHES_H_
...@@ -2037,13 +2037,8 @@ Display DisplayManager::CreateDisplayFromDisplayInfoById(int64_t id) { ...@@ -2037,13 +2037,8 @@ Display DisplayManager::CreateDisplayFromDisplayInfoById(int64_t id) {
new_display.set_rotation(display_info.GetActiveRotation()); new_display.set_rotation(display_info.GetActiveRotation());
new_display.set_touch_support(display_info.touch_support()); new_display.set_touch_support(display_info.touch_support());
new_display.set_maximum_cursor_size(display_info.maximum_cursor_size()); new_display.set_maximum_cursor_size(display_info.maximum_cursor_size());
#if defined(OS_CHROMEOS) if (!Display::HasForceColorProfile())
// TODO(mcasas): remove this check, http://crbug.com/771345.
if (base::FeatureList::IsEnabled(features::kUseMonitorColorSpace))
new_display.set_color_space(display_info.color_space()); new_display.set_color_space(display_info.color_space());
#else
new_display.set_color_space(display_info.color_space());
#endif
if (internal_display_has_accelerometer_ && Display::IsInternalDisplayId(id)) { if (internal_display_has_accelerometer_ && Display::IsInternalDisplayId(id)) {
new_display.set_accelerometer_support( new_display.set_accelerometer_support(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment