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

Unified Diff: chrome/browser/ui/ash/launcher/chrome_launcher_controller_per_app.cc

Issue 14584004: Remove ash-disable-launcher-per-display flag (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 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
Index: chrome/browser/ui/ash/launcher/chrome_launcher_controller_per_app.cc
diff --git a/chrome/browser/ui/ash/launcher/chrome_launcher_controller_per_app.cc b/chrome/browser/ui/ash/launcher/chrome_launcher_controller_per_app.cc
index 9568380d226c3604d0f2d5094340a63f4308bd3d..22e929430fb3d6f77abf8ee66684b7f735ff6fc3 100644
--- a/chrome/browser/ui/ash/launcher/chrome_launcher_controller_per_app.cc
+++ b/chrome/browser/ui/ash/launcher/chrome_launcher_controller_per_app.cc
@@ -613,11 +613,7 @@ void ChromeLauncherControllerPerApp::SetAppImage(
void ChromeLauncherControllerPerApp::OnAutoHideBehaviorChanged(
ash::ShelfAutoHideBehavior new_behavior) {
- ash::Shell::RootWindowList root_windows;
- if (ash::Shell::IsLauncherPerDisplayEnabled())
- root_windows = ash::Shell::GetAllRootWindows();
- else
- root_windows.push_back(ash::Shell::GetPrimaryRootWindow());
+ ash::Shell::RootWindowList root_windows = ash::Shell::GetAllRootWindows();
for (ash::Shell::RootWindowList::const_iterator iter =
root_windows.begin();
@@ -1500,11 +1496,7 @@ void ChromeLauncherControllerPerApp::SetShelfAutoHideBehaviorPrefs(
}
void ChromeLauncherControllerPerApp::SetShelfAutoHideBehaviorFromPrefs() {
- ash::Shell::RootWindowList root_windows;
- if (ash::Shell::IsLauncherPerDisplayEnabled())
- root_windows = ash::Shell::GetAllRootWindows();
- else
- root_windows.push_back(ash::Shell::GetPrimaryRootWindow());
+ ash::Shell::RootWindowList root_windows = ash::Shell::GetAllRootWindows();
for (ash::Shell::RootWindowList::const_iterator iter = root_windows.begin();
iter != root_windows.end(); ++iter) {
@@ -1518,11 +1510,8 @@ void ChromeLauncherControllerPerApp::SetShelfAlignmentFromPrefs() {
switches::kShowLauncherAlignmentMenu))
return;
- ash::Shell::RootWindowList root_windows;
- if (ash::Shell::IsLauncherPerDisplayEnabled())
- root_windows = ash::Shell::GetAllRootWindows();
- else
- root_windows.push_back(ash::Shell::GetPrimaryRootWindow());
+ ash::Shell::RootWindowList root_windows = ash::Shell::GetAllRootWindows();
+
for (ash::Shell::RootWindowList::const_iterator iter = root_windows.begin();
iter != root_windows.end(); ++iter) {
// See comment in |kShelfAlignment| as to why we consider two prefs.
« no previous file with comments | « chrome/browser/about_flags.cc ('k') | chrome/browser/ui/ash/launcher/chrome_launcher_controller_per_browser.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698