Compile chrome://settings, part 2: reduce from 950 to 400 errors

chrome/browser/resources/options/pref_ui.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 Preferences = options.Preferences;
 
   /**
    * Allows an element to be disabled for several reasons.
@@ skipping 106 matching lines @@
      * true, the default handler is suppressed.
      * @param {Event} event Input element change event.
      */
     customPrefChangeHandler: function(event) {
       return false;
     },
   };
 
   /**
    * The name of the associated preference.
-   * @type {string}
    */
   cr.defineProperty(PrefInputElement, 'pref', cr.PropertyKind.ATTR);
 
   /**
    * The data type of the associated preference, only relevant for derived
    * classes that support different data types.
-   * @type {string}
    */
   cr.defineProperty(PrefInputElement, 'dataType', cr.PropertyKind.ATTR);
 
   /**
    * Whether this input element is part of a dialog. If so, changes take effect
    * in the settings UI immediately but are only actually committed when the
    * user confirms the dialog. If the user cancels the dialog instead, the
    * changes are rolled back in the settings UI and never committed.
-   * @type {boolean}
    */
   cr.defineProperty(PrefInputElement, 'dialogPref', cr.PropertyKind.BOOL_ATTR);
 
   /**
    * Whether the associated preference is controlled by a source other than the
    * user's setting (can be 'policy', 'extension', 'recommended' or unset).
-   * @type {string}
    */
   cr.defineProperty(PrefInputElement, 'controlledBy', cr.PropertyKind.ATTR);
 
   /**
    * The user metric string.
-   * @type {string}
    */
   cr.defineProperty(PrefInputElement, 'metric', cr.PropertyKind.ATTR);
 
   /////////////////////////////////////////////////////////////////////////////
   // PrefCheckbox class:
 
   // Define a constructor that uses an input element as its underlying element.
   var PrefCheckbox = cr.ui.define('input');
 
   PrefCheckbox.prototype = {
@@ skipping 32 matching lines @@
     updateStateFromPref_: function(event) {
       if (this.customPrefChangeHandler(event))
         return;
       var value = Boolean(event.value.value);
       this.checked = this.inverted_pref ? !value : value;
     },
   };
 
   /**
    * Whether the mapping between checkbox state and associated pref is inverted.
-   * @type {boolean}
    */
   cr.defineProperty(PrefCheckbox, 'inverted_pref', cr.PropertyKind.BOOL_ATTR);
 
   /////////////////////////////////////////////////////////////////////////////
   // PrefNumber class:
 
   // Define a constructor that uses an input element as its underlying element.
   var PrefNumber = cr.ui.define('input');
 
   PrefNumber.prototype = {
@@ skipping 192 matching lines @@
      * Update the selected item when the associated pref changes.
      * @param {Event} event Pref change event.
      * @private
      */
     updateStateFromPref_: function(event) {
       if (this.customPrefChangeHandler(event))
         return;
 
       // Make sure the value is a string, because the value is stored as a
       // string in the HTMLOptionElement.
-      value = String(event.value.value);
+      var value = String(event.value.value);
 
       var found = false;
       for (var i = 0; i < this.options.length; i++) {
         if (this.options[i].value == value) {
           this.selectedIndex = i;
           found = true;
         }
       }
 
       // Item not found, select first item.
@@ skipping 121 matching lines @@
     /**
      * See |updateDisabledState_| above.
      */
     setDisabled: function(reason, disabled) {
       updateDisabledState_(this, reason, disabled);
     },
   };
 
   /**
    * The name of the associated preference.
-   * @type {string}
    */
   cr.defineProperty(PrefButton, 'pref', cr.PropertyKind.ATTR);
 
   /**
    * Whether the associated preference is controlled by a source other than the
    * user's setting (can be 'policy', 'extension', 'recommended' or unset).
-   * @type {string}
    */
   cr.defineProperty(PrefButton, 'controlledBy', cr.PropertyKind.ATTR);
 
   // Export
   return {
     PrefCheckbox: PrefCheckbox,
     PrefNumber: PrefNumber,
     PrefRadio: PrefRadio,
     PrefRange: PrefRange,
     PrefSelect: PrefSelect,
     PrefTextField: PrefTextField,
     PrefPortNumber: PrefPortNumber,
     PrefButton: PrefButton
   };
 
 });