Move handling of dialog preferences to Preferences class

chrome/browser/resources/options2/browser_options.js

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 cr.define('options', function() {
   var OptionsPage = options.OptionsPage;
   var ArrayDataModel = cr.ui.ArrayDataModel;
   var RepeatingButton = cr.ui.RepeatingButton;
 
   //
@@ skipping 347 matching lines @@
 
       // Web Content section.
       $('fontSettingsCustomizeFontsButton').onclick = function(event) {
         OptionsPage.navigateToPage('fonts');
         chrome.send('coreOptionsUserMetricsAction', ['Options_FontSettings']);
       };
       $('defaultFontSize').onchange = function(event) {
         var value = event.target.options[event.target.selectedIndex].value;
         Preferences.setIntegerPref(
              'webkit.webprefs.default_fixed_font_size',
-             value - OptionsPage.SIZE_DIFFERENCE_FIXED_STANDARD, '');
+             value - OptionsPage.SIZE_DIFFERENCE_FIXED_STANDARD,
+             true);

[James Hawkins, 2012/08/02 20:33:32]

Optional nit: Condense parameters to save a line, i.e., 'true' can fit on the
line above.

[bartfab (slow), 2012/08/31 14:14:30]

Done.

         chrome.send('defaultFontSizeAction', [String(value)]);
       };
       $('defaultZoomFactor').onchange = function(event) {
         chrome.send('defaultZoomFactorAction',
             [String(event.target.options[event.target.selectedIndex].value)]);
       };
 
       // Languages section.
       $('language-button').onclick = function(event) {
         OptionsPage.navigateToPage('languages');
@@ skipping 966 matching lines @@
     BrowserOptions.getLoggedInUsername = function() {
       return BrowserOptions.getInstance().username_;
     };
   }
 
   // Export
   return {
     BrowserOptions: BrowserOptions
   };
 });