Compile chrome://settings, part 1: remove all warnings

chrome/browser/resources/options/chromeos/internet_detail.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.
 
 // NOTE(stevenjb): This code is in the process of being converted to be
 // compatible with the networkingPrivate extension API:
 // * The network property dictionaries are being converted to use ONC values.
 // * chrome.send calls will be replaced with an API object that simulates the
 //   networkingPrivate API. See network_config.js.
 // See crbug.com/279351 for more info.
@@ skipping 132 matching lines @@
    * @param {string} selector CSS selector for extracting a list of elements.
    * @param {bool} hidden New hidden value.
    */
   function updateHidden(selector, hidden) {
     var elements = cr.doc.querySelectorAll(selector);
     for (var i = 0, el; el = elements[i]; i++) {
       el.hidden = hidden;
     }
   }
 
-  /*
+  /**
    * Helper function to update the properties of the data object from the
    * properties in the update object.
-   * @param {object} data object to update.
-   * @param {object} object containing the updated properties.
+   * @param {Object} data Object to update.
+   * @param {Object} update Object containing the updated properties.
    */
   function updateDataObject(data, update) {
     for (var prop in update) {
       if (prop in data)
         data[prop] = update[prop];
     }
   }
 
   /**
    * Monitor pref change of given element.
@@ skipping 348 matching lines @@
           model.value = model.automatic;
           fields[i].model = model;
         }
       }
       if (!checked)
         $('ip-address').focus();
     },
 
     /**
      * Handler for when the name server selection changes.
-     * @param {Event} e The click event.
+     * @param {Event} event The click event.
      * @private
      */
     handleNameServerTypeChange_: function(event) {
       var type = event.target.value;
       DetailsInternetPage.updateNameServerDisplay(type);
     },
 
     /**
      * Creates an indicator event for controlled properties using
      * the same dictionary format as CoreOptionsHandler::CreateValueForPref.
      * @param {string} name The name for the Event.
-     * @param {Object} data Property dictionary with |value|, |controlledBy|,
-     *  and |recommendedValue| properties set.
+     * @param {Object} propData Property dictionary with |value|,
+     *     |controlledBy|, and |recommendedValue| properties set.
      * @private
      */
     createControlledEvent_: function(name, propData) {
       var event = new Event(name);
       event.value = {
         value: propData.value,
         controlledBy: propData.controlledBy,
         recommendedValue: propData.recommendedValue
       };
       return event;
     },
 
     /**
      * Creates an indicator event for controlled properties using
      * the ONC getManagedProperties dictionary format.
      * @param {string} name The name for the Event.
-     * @param {Object} data ONC managed network property dictionary.
+     * @param {Object} propData ONC managed network property dictionary.
      * @private
      */
     createManagedEvent_: function(name, propData) {
       var event = new Event(name);
       event.value = {};
 
       // Set the current value and recommended value.
       var activeValue = propData['Active'];
       var effective = propData['Effective'];
       if (activeValue == undefined)
@@ skipping 921 matching lines @@
 
     // Don't show page name in address bar and in history to prevent people
     // navigate here by hand and solve issue with page session restore.
     PageManager.showPageByName('detailsInternetPage', false);
   };
 
   return {
     DetailsInternetPage: DetailsInternetPage
   };
 });