web-ui settings: Explicitly set preferences for sans-serif font and minimum l...

chrome/browser/resources/options/font_settings.js

 // Copyright (c) 2011 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;
 
   /**
    * FontSettings class
@@ skipping 25 matching lines @@
       serifFontRange.continuous = false;
       serifFontRange.fontSampleEl = $('serif-font-sample');
       serifFontRange.notifyChange = this.rangeChanged_.bind(this);
 
       var minimumFontRange = $('minimum-font-size');
       minimumFontRange.valueMap = [9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 20,
           22, 24];
       minimumFontRange.continuous = false;
       minimumFontRange.fontSampleEl = $('minimum-font-sample');
       minimumFontRange.notifyChange = this.rangeChanged_.bind(this);
+      minimumFontRange.notifyPrefChange =
+          this.minimumFontSizeChanged_.bind(this);
+
+      // Add an additional listener to the font select menu for the
+      // 'sansserif_font_family' preference.
+      $('serif-font-family').addEventListener('change',
+          function(e) {
+            Preferences.setStringPref('webkit.webprefs.sansserif_font_family',
+                this.options[this.selectedIndex].value, '');
+          });
     },
 
     /**
-     * @inheritDoc
+     * Called as the user changes a non-continuous slider.  This allows for
+     * reflecting the change in the UI before the preference has been changed.
+     * @param {Element} el The slider input element.
+     * @param {number} value The mapped value currently set by the slider.
+     * @private
      */
     rangeChanged_: function(el, value) {
       this.setupFontSample_(
           el.fontSampleEl, value, el.fontSampleEl.style.fontFamily);
     },
 
     /**
+     * Sets the 'minimum_logical_font_size' preference when the minimum font
+     * size has been changed by the user.
+     * @param {Element} el The slider input element.
+     * @param {number} value The mapped value that has been saved.
+     * @private
+     */
+    minimumFontSizeChanged_: function(el, value) {
+      Preferences.setIntegerPref('webkit.webprefs.minimum_logical_font_size',
+          value, '');
+    },
+
+    /**
      * Sets the text, font size and font family of the sample text.
      * @param {Element} el The div containing the sample text.
      * @param {number} size The font size of the sample text.
      * @param {string} font The font family of the sample text.
      * @private
      */
     setupFontSample_: function(el, size, font) {
       el.textContent =
           size + "pt: " + localStrings.getString('fontSettingsLoremIpsum');
       el.style.fontSize = size + "pt";
@@ skipping 17 matching lines @@
     FontSettings.getInstance().setupFontSample_(
         $('minimum-font-sample'), size);
   };
 
   // Export
   return {
     FontSettings: FontSettings
   };
 });