[MD settings] observe changes to default font size in Appearance

chrome/browser/resources/settings/appearance_page/appearance_fonts_page.js

 // Copyright 2015 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.
 
 (function() {
   'use strict';
 
-  /**
-   * This is the absolute difference maintained between standard and
-   * fixed-width font sizes. http://crbug.com/91922.
-   * @const @private {number}
-   */
-  var SIZE_DIFFERENCE_FIXED_STANDARD_ = 3;
-
   /** @const @private {!Array<number>} */
   var FONT_SIZE_RANGE_ = [
     9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 20, 22, 24, 26, 28, 30, 32, 34, 36,
     40, 44, 48, 56, 64, 72,
   ];
 
   /** @const @private {!Array<number>} */
   var MINIMUM_FONT_SIZE_RANGE_ = [
     6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 20, 22, 24
   ];
 
   /**
    * 'settings-appearance-fonts-page' is the settings page containing appearance
    * settings.
-   *
-   * Example:
-   *
-   *   <settings-appearance-fonts-page prefs="{{prefs}}">
-   *   </settings-appearance-fonts-page>
    */
   Polymer({
     is: 'settings-appearance-fonts-page',
 
     behaviors: [I18nBehavior, WebUIListenerBehavior],
 
     properties: {
       /** @private */
       advancedExtensionSublabel_: String,
 
@@ skipping 37 matching lines @@
 
     /** @private {?settings.FontsBrowserProxy} */
     browserProxy_: null,
 
     /** @private {boolean} */
     advancedExtensionInstalled_: false,
 
     /** @private {?string} */
     advancedExtensionUrl_: null,
 
-    observers: [
-      'fontSizeChanged_(prefs.webkit.webprefs.default_font_size.value)',
-    ],
-
     /** @override */
     created: function() {
       this.browserProxy_ = settings.FontsBrowserProxyImpl.getInstance();
     },
 
     /** @override */
     ready: function() {
       this.addWebUIListener('advanced-font-settings-installed',
           this.setAdvancedExtensionInstalled_.bind(this));
       this.browserProxy_.observeAdvancedFontExtensionAvailable();
@@ skipping 32 matching lines @@
         fontMenuOptions.push({
           value: response.fontList[i][0],
           name: response.fontList[i][1]
         });
       }
       this.fontOptions_ = fontMenuOptions;
       this.advancedExtensionUrl_ = response.extensionUrl;
     },
 
     /**
-     * @param {number} value The changed font size slider value.
-     * @private
-     */
-    fontSizeChanged_: function(value) {
-      // TODO(michaelpg): Whitelist this pref in prefs_utils.cc so it is
-      // included in the <settings-prefs> getAllPrefs call, otherwise this path
-      // is invalid and nothing happens. See crbug.com/612535.
-      this.set('prefs.webkit.webprefs.default_fixed_font_size.value',
-          value - SIZE_DIFFERENCE_FIXED_STANDARD_);
-    },
-
-    /**
      * Get the minimum font size, accounting for unset prefs.
      * @return {?}
      * @private
      */
     computeMinimumFontSize_: function() {
       return this.get('prefs.webkit.webprefs.minimum_font_size.value') ||
           MINIMUM_FONT_SIZE_RANGE_[0];
     },
   });
 })();