New issue
Advanced search Search tips

Issue 774795 link

Starred by 2 users

Issue metadata

Status: Started
Owner:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug

Blocking:
issue 774794



Sign in to add a comment

Clean up old code for dual display mirroring

Project Member Reported by weidongg@chromium.org, Oct 14 2017

Issue description

Clean up old code for dual display mirroring once |kEnableMultiDisplayMirroring| display switch is enabled by default after everything gets stable.
 
Blocking: 774794
Project Member

Comment 2 by bugdroid1@chromium.org, Jan 6 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f1e5a894ebcfa33dda1323704a17d3f6f64a370f

commit f1e5a894ebcfa33dda1323704a17d3f6f64a370f
Author: Weidong Guo <weidongg@chromium.org>
Date: Sat Jan 06 03:55:46 2018

Enable multi-mirroring by default

Changes:
1. Change from kEnableMultiMirroring to kDisableMultiMirroring.
2. Remove related tests.

Bug: 774795
Change-Id: I793504b1a9d89c2f6129c1e4edb45cb32a82c845
Reviewed-on: https://chromium-review.googlesource.com/847792
Commit-Queue: Weidong Guo <weidongg@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Reviewed-by: Ahmed Fakhry <afakhry@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527493}
[modify] https://crrev.com/f1e5a894ebcfa33dda1323704a17d3f6f64a370f/ash/display/display_manager_unittest.cc
[modify] https://crrev.com/f1e5a894ebcfa33dda1323704a17d3f6f64a370f/ash/display/mirror_window_controller_unittest.cc
[modify] https://crrev.com/f1e5a894ebcfa33dda1323704a17d3f6f64a370f/ash/system/screen_layout_observer_unittest.cc
[modify] https://crrev.com/f1e5a894ebcfa33dda1323704a17d3f6f64a370f/chrome/browser/about_flags.cc
[modify] https://crrev.com/f1e5a894ebcfa33dda1323704a17d3f6f64a370f/chrome/browser/chromeos/display/display_prefs_unittest.cc
[modify] https://crrev.com/f1e5a894ebcfa33dda1323704a17d3f6f64a370f/chrome/browser/flag_descriptions.cc
[modify] https://crrev.com/f1e5a894ebcfa33dda1323704a17d3f6f64a370f/chrome/browser/flag_descriptions.h
[modify] https://crrev.com/f1e5a894ebcfa33dda1323704a17d3f6f64a370f/chrome/browser/ui/webui/settings/md_settings_localized_strings_provider.cc
[modify] https://crrev.com/f1e5a894ebcfa33dda1323704a17d3f6f64a370f/tools/metrics/histograms/enums.xml
[modify] https://crrev.com/f1e5a894ebcfa33dda1323704a17d3f6f64a370f/ui/display/display_switches.cc
[modify] https://crrev.com/f1e5a894ebcfa33dda1323704a17d3f6f64a370f/ui/display/display_switches.h
[modify] https://crrev.com/f1e5a894ebcfa33dda1323704a17d3f6f64a370f/ui/display/manager/chromeos/display_configurator.cc
[modify] https://crrev.com/f1e5a894ebcfa33dda1323704a17d3f6f64a370f/ui/display/manager/display_manager.cc

Status: Fixed (was: Assigned)

Comment 4 by msw@chromium.org, Mar 22 2018

Status: Started (was: Fixed)
Hey Wei, there are still TODOs to remove tests referencing this bug.

Sign in to add a comment