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

Side by Side Diff: chrome/browser/resources/options2/browser_options.js

Issue 10201014: Implement High Contrast mode for Chrome OS (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Code review fixes Created 8 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 cr.define('options', function() { 5 cr.define('options', function() {
6 var OptionsPage = options.OptionsPage; 6 var OptionsPage = options.OptionsPage;
7 var ArrayDataModel = cr.ui.ArrayDataModel; 7 var ArrayDataModel = cr.ui.ArrayDataModel;
8 var RepeatingButton = cr.ui.RepeatingButton; 8 var RepeatingButton = cr.ui.RepeatingButton;
9 9
10 // 10 //
(...skipping 448 matching lines...) Expand 10 before | Expand all | Expand 10 after
459 $('cloudPrintManageButton').onclick = function(event) { 459 $('cloudPrintManageButton').onclick = function(event) {
460 chrome.send('showCloudPrintManagePage'); 460 chrome.send('showCloudPrintManagePage');
461 }; 461 };
462 462
463 // Accessibility section (CrOS only). 463 // Accessibility section (CrOS only).
464 if (cr.isChromeOS) { 464 if (cr.isChromeOS) {
465 $('accessibility-spoken-feedback-check').onchange = function(event) { 465 $('accessibility-spoken-feedback-check').onchange = function(event) {
466 chrome.send('spokenFeedbackChange', 466 chrome.send('spokenFeedbackChange',
467 [$('accessibility-spoken-feedback-check').checked]); 467 [$('accessibility-spoken-feedback-check').checked]);
468 }; 468 };
469
470 $('accessibility-high-contrast-check').onchange = function(event) {
471 chrome.send('highContrastChange',
472 [$('accessibility-high-contrast-check').checked]);
James Hawkins 2012/05/17 16:07:37 nit: The parameter indentation on this line and li
Zachary Kuznia 2012/05/21 05:50:05 Done.
473 };
469 } 474 }
470 475
471 // Background mode section. 476 // Background mode section.
472 if ($('backgroundModeCheckbox')) { 477 if ($('backgroundModeCheckbox')) {
473 cr.defineProperty($('backgroundModeCheckbox'), 478 cr.defineProperty($('backgroundModeCheckbox'),
474 'controlledBy', 479 'controlledBy',
475 cr.PropertyKind.ATTR); 480 cr.PropertyKind.ATTR);
476 $('backgroundModeCheckbox').onclick = function(event) { 481 $('backgroundModeCheckbox').onclick = function(event) {
477 chrome.send('backgroundModeAction', 482 chrome.send('backgroundModeAction',
478 [String($('backgroundModeCheckbox').checked)]); 483 [String($('backgroundModeCheckbox').checked)]);
(...skipping 715 matching lines...) Expand 10 before | Expand all | Expand 10 after
1194 */ 1199 */
1195 setSpokenFeedbackCheckboxState_: function(checked) { 1200 setSpokenFeedbackCheckboxState_: function(checked) {
1196 $('accessibility-spoken-feedback-check').checked = checked; 1201 $('accessibility-spoken-feedback-check').checked = checked;
1197 }, 1202 },
1198 1203
1199 /** 1204 /**
1200 * Set the initial state of the high contrast checkbox. 1205 * Set the initial state of the high contrast checkbox.
1201 * @private 1206 * @private
1202 */ 1207 */
1203 setHighContrastCheckboxState_: function(checked) { 1208 setHighContrastCheckboxState_: function(checked) {
1204 // TODO(zork): Update UI 1209 $('accessibility-high-contrast-check').checked = checked;
1205 }, 1210 },
1206 1211
1207 /** 1212 /**
1208 * Set the initial state of the screen magnifier checkbox. 1213 * Set the initial state of the screen magnifier checkbox.
1209 * @private 1214 * @private
1210 */ 1215 */
1211 setScreenMagnifierCheckboxState_: function(checked) { 1216 setScreenMagnifierCheckboxState_: function(checked) {
1212 // TODO(zork): Update UI 1217 // TODO(zork): Update UI
1213 }, 1218 },
1214 1219
(...skipping 168 matching lines...) Expand 10 before | Expand all | Expand 10 after
1383 BrowserOptions.getLoggedInUsername = function() { 1388 BrowserOptions.getLoggedInUsername = function() {
1384 return BrowserOptions.getInstance().username_; 1389 return BrowserOptions.getInstance().username_;
1385 }; 1390 };
1386 } 1391 }
1387 1392
1388 // Export 1393 // Export
1389 return { 1394 return {
1390 BrowserOptions: BrowserOptions 1395 BrowserOptions: BrowserOptions
1391 }; 1396 };
1392 }); 1397 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698