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

Unified Diff: chrome/browser/resources/options2/browser_options.js

Issue 10544171: Add OptionsUI and its handler for multiple displays. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix Created 8 years, 6 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/resources/options2/browser_options.js
diff --git a/chrome/browser/resources/options2/browser_options.js b/chrome/browser/resources/options2/browser_options.js
index f8a0b2b6b03fc55f3ac9fd3379526e9009a32335..fcc581cd0230ec98d658790fad1d0a97ea1972d7 100644
--- a/chrome/browser/resources/options2/browser_options.js
+++ b/chrome/browser/resources/options2/browser_options.js
@@ -482,6 +482,15 @@ cr.define('options', function() {
};
}
+ // Display management section (CrOS only).
+ if (cr.isChromeOS) {
+ $('displayOptionsButton').onclick = function(event) {
+ OptionsPage.navigateToPage('display');
+ chrome.send('coreOptionsUserMetricsAction',
+ ['Options_Display']);
+ }
+ }
+
// Background mode section.
if ($('backgroundModeCheckbox')) {
cr.defineProperty($('backgroundModeCheckbox'),
@@ -1258,6 +1267,22 @@ cr.define('options', function() {
},
/**
+ * Activate the display options button on the System settings page.
+ * @private
+ */
+ showDisplayOptions_: function() {
+ $('display-options-section').hidden = false;
James Hawkins 2012/06/18 15:13:49 Please copy showTouchpadControls_ here instead of
Jun Mukai 2012/06/19 08:37:44 Done.
+ },
+
+ /**
+ * Deactivate the display options button on the System settings page.
+ * @private
+ */
+ hideDisplayOptions_: function() {
+ $('display-options-section').hidden = true;
+ },
+
+ /**
* Activate the bluetooth settings section on the System settings page.
* @private
*/
@@ -1357,6 +1382,7 @@ cr.define('options', function() {
'addBluetoothDevice',
'getStartStopSyncButton',
'hideBluetoothSettings',
+ 'hideDisplayOptions',
'removeCloudPrintConnectorSection',
'removeBluetoothDevice',
'setAutoOpenFileTypesDisplayed',
@@ -1379,6 +1405,7 @@ cr.define('options', function() {
'setupProxySettingsSection',
'setVirtualKeyboardCheckboxState',
'showBluetoothSettings',
+ 'showDisplayOptions',
'showMouseControls',
'showTouchpadControls',
'updateAccountPicture',

Powered by Google App Engine
This is Rietveld 408576698