Fix JavaScript errors in options2

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() {
   const OptionsPage = options.OptionsPage;
   const ArrayDataModel = cr.ui.ArrayDataModel;
   const RepeatingButton = cr.ui.RepeatingButton;
 
   //
@@ skipping 332 matching lines @@
     },
 
     /**
      * Called to set the Instant field trial status.
      * @param {boolean} enabled If true, the experiment is enabled.
      * @private
      */
     setInstantFieldTrialStatus_: function(enabled) {
       $('instantEnabledCheckbox').hidden = enabled;
       $('instantFieldTrialCheckbox').hidden = !enabled;
-      $('instantLabel').htmlFor = enabled ? 'instantFieldTrialCheckbox'
-                                          : 'instantEnabledCheckbox';
+      $('instantLabel').htmlFor = enabled ? 'instantFieldTrialCheckbox' :
+                                            'instantEnabledCheckbox';
     },
 
     /**
      * Update the Default Browsers section based on the current state.
      * @param {string} statusString Description of the current default state.
      * @param {boolean} isDefault Whether or not the browser is currently
      *     default.
      * @param {boolean} canBeDefault Whether or not the browser can be default.
      * @private
      */
@@ skipping 32 matching lines @@
         var option = new Option(engine['name'], engine['index']);
         if (defaultValue == option.value)
           defaultIndex = i;
         engineSelect.appendChild(option);
       }
       if (defaultIndex >= 0)
         engineSelect.selectedIndex = defaultIndex;
     },
 
     /**
-     * Returns true if the custom startup page control block should
-     * be enabled.
-     * @returns {boolean} Whether the startup page controls should be
-     *     enabled.
+     * Returns true if the custom startup page control block should be enabled.
+     * @return {boolean} Whether the startup page controls should be enabled.
      */
     shouldEnableCustomStartupPageControls: function(pages) {
       return $('startupShowPagesButton').checked &&
           !this.startup_pages_pref_.disabled;
     },
 
     /**
      * Sets the enabled state of the custom startup page list controls
      * based on the current startup radio button selection.
      * @private
@@ skipping 104 matching lines @@
 
     /**
      * Adds all |profiles| to the list.
      * @param {Array.<Object>} An array of profile info objects.
      *     each object is of the form:
      *       profileInfo = {
      *         name: "Profile Name",
      *         iconURL: "chrome://path/to/icon/image",
      *         filePath: "/path/to/profile/data/on/disk",
      *         isCurrentProfile: false
-     *       };
+     *       }.

[James Hawkins, 2012/02/03 01:28:24]

Whatsup with this change?

[Tyler Breisacher (Chromium), 2012/02/03 01:40:01]

The description is a sentence so the linter wants it to end with a period.

[James Hawkins, 2012/02/03 01:41:58]

That's pretty ridiculous.  This is quoted code, and ending that code with a
period is not valid.

      * @private
      */
     setProfilesInfo_: function(profiles) {
       this.setProfileViewSingle_(profiles.length);
       // add it to the list, even if the list is hidden so we can access it
       // later.
       $('profiles-list').dataModel = new ArrayDataModel(profiles);
       this.setProfileViewButtonsStatus_();
     },
 
@@ skipping 70 matching lines @@
       return BrowserOptions.getInstance().username_;
     };
   }
 
   // Export
   return {
     BrowserOptions: BrowserOptions
   };
 
 });