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

Unified Diff: chrome/browser/ui/webui/options/chromeos/display_options_handler.cc

Issue 2445583002: Relocate display_manager from ash to ui (Closed)
Patch Set: fix windows build Created 4 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
Index: chrome/browser/ui/webui/options/chromeos/display_options_handler.cc
diff --git a/chrome/browser/ui/webui/options/chromeos/display_options_handler.cc b/chrome/browser/ui/webui/options/chromeos/display_options_handler.cc
index 68c5fa355d25010149657b733dcf75ad690d80b1..43b01ab76e51877e55b79c106332e2c1eaf3bb09 100644
--- a/chrome/browser/ui/webui/options/chromeos/display_options_handler.cc
+++ b/chrome/browser/ui/webui/options/chromeos/display_options_handler.cc
@@ -13,7 +13,6 @@
#include "ash/common/strings/grit/ash_strings.h"
#include "ash/display/display_configuration_controller.h"
-#include "ash/display/display_manager.h"
#include "ash/display/resolution_notification_controller.h"
#include "ash/display/window_tree_host_manager.h"
#include "ash/shell.h"
@@ -34,6 +33,7 @@
#include "ui/display/display.h"
#include "ui/display/manager/display_layout.h"
#include "ui/display/manager/display_layout_builder.h"
+#include "ui/display/manager/display_manager.h"
#include "ui/display/screen.h"
#include "ui/gfx/geometry/rect.h"
#include "ui/gfx/geometry/size_conversions.h"
@@ -42,7 +42,7 @@ namespace chromeos {
namespace options {
namespace {
-ash::DisplayManager* GetDisplayManager() {
+display::DisplayManager* GetDisplayManager() {
return ash::Shell::GetInstance()->display_manager();
}
@@ -305,19 +305,19 @@ void DisplayOptionsHandler::OnDisplayConfigurationChanged() {
}
void DisplayOptionsHandler::SendAllDisplayInfo() {
- ash::DisplayManager* display_manager = GetDisplayManager();
+ display::DisplayManager* display_manager = GetDisplayManager();
std::vector<display::Display> displays;
for (size_t i = 0; i < display_manager->GetNumDisplays(); ++i)
displays.push_back(display_manager->GetDisplayAt(i));
- ash::DisplayManager::MultiDisplayMode display_mode;
+ display::DisplayManager::MultiDisplayMode display_mode;
if (display_manager->IsInMirrorMode())
- display_mode = ash::DisplayManager::MIRRORING;
+ display_mode = display::DisplayManager::MIRRORING;
else if (display_manager->IsInUnifiedMode())
- display_mode = ash::DisplayManager::UNIFIED;
+ display_mode = display::DisplayManager::UNIFIED;
else
- display_mode = ash::DisplayManager::EXTENDED;
+ display_mode = display::DisplayManager::EXTENDED;
base::FundamentalValue mode(static_cast<int>(display_mode));
int64_t primary_id = display::Screen::GetScreen()->GetPrimaryDisplay().id();
@@ -381,7 +381,7 @@ void DisplayOptionsHandler::UpdateDisplaySettingsEnabled() {
if (chrome::IsRunningInMash())
return;
- ash::DisplayManager* display_manager = GetDisplayManager();
+ display::DisplayManager* display_manager = GetDisplayManager();
bool disable_multi_display_layout =
base::CommandLine::ForCurrentProcess()->HasSwitch(
chromeos::switches::kDisableMultiDisplayLayout);
@@ -431,7 +431,7 @@ void DisplayOptionsHandler::HandleSetDisplayLayout(
NOTREACHED();
content::RecordAction(base::UserMetricsAction("Options_DisplayRearrange"));
- ash::DisplayManager* display_manager = GetDisplayManager();
+ display::DisplayManager* display_manager = GetDisplayManager();
display::DisplayLayoutBuilder builder(
display_manager->GetCurrentDisplayLayout());
builder.ClearPlacements();
@@ -493,7 +493,7 @@ void DisplayOptionsHandler::HandleSetDisplayMode(const base::ListValue* args) {
content::RecordAction(
base::UserMetricsAction("Options_DisplaySetResolution"));
- ash::DisplayManager* display_manager = GetDisplayManager();
+ display::DisplayManager* display_manager = GetDisplayManager();
scoped_refptr<display::ManagedDisplayMode> current_mode =
display_manager->GetActiveModeForDisplayId(display_id);
if (!display_manager->SetDisplayMode(display_id, mode)) {
@@ -580,7 +580,8 @@ void DisplayOptionsHandler::HandleSetUnifiedDesktopEnabled(
NOTREACHED();
GetDisplayManager()->SetDefaultMultiDisplayModeForCurrentDisplays(
- enable ? ash::DisplayManager::UNIFIED : ash::DisplayManager::EXTENDED);
+ enable ? display::DisplayManager::UNIFIED
+ : display::DisplayManager::EXTENDED);
}
} // namespace options

Powered by Google App Engine
This is Rietveld 408576698