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

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

Issue 10829034: Store secondary display layout to the user's preferences (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 5 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/options2/chromeos/display_options_handler.cc
diff --git a/chrome/browser/ui/webui/options2/chromeos/display_options_handler.cc b/chrome/browser/ui/webui/options2/chromeos/display_options_handler.cc
index 46656e00c5a1901ef692a88a4bd1480980222175..739f6118a947bf3255922e95703f4e645d61655c 100644
--- a/chrome/browser/ui/webui/options2/chromeos/display_options_handler.cc
+++ b/chrome/browser/ui/webui/options2/chromeos/display_options_handler.cc
@@ -12,6 +12,9 @@
#include "base/json/json_value_converter.h"
#include "base/logging.h"
#include "base/values.h"
+#include "chrome/browser/prefs/pref_service.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/common/pref_names.h"
#include "chromeos/display/output_configurator.h"
#include "content/public/browser/web_ui.h"
#include "grit/generated_resources.h"
@@ -116,10 +119,9 @@ void DisplayOptionsHandler::SendDisplayInfo() {
displays.Set(i, js_display);
}
- DisplayController* display_controller =
- ash::Shell::GetInstance()->display_controller();
- base::FundamentalValue layout(static_cast<int>(
- display_controller->secondary_display_layout()));
+ PrefService* pref_service = Profile::FromWebUI(web_ui())->GetPrefs();
+ base::FundamentalValue layout(
+ pref_service->GetInteger(prefs::kSecondaryDisplayLayout));
web_ui()->CallJavascriptFunction(
"options.DisplayOptions.setDisplayInfo",
@@ -152,8 +154,8 @@ void DisplayOptionsHandler::HandleDisplayLayout(const base::ListValue* args) {
DCHECK_LE(DisplayController::TOP, layout);
DCHECK_GE(DisplayController::LEFT, layout);
- ash::Shell::GetInstance()->display_controller()->SetSecondaryDisplayLayout(
- static_cast<DisplayController::SecondaryDisplayLayout>(layout));
+ PrefService* pref_service = Profile::FromWebUI(web_ui())->GetPrefs();
+ pref_service->SetInteger(prefs::kSecondaryDisplayLayout, layout);
SendDisplayInfo();
}

Powered by Google App Engine
This is Rietveld 408576698