options: prettier fix for bug 281069 that can be committed to trunk later.

chrome/browser/resources/options/language_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.
 
 // TODO(kochi): Generalize the notification as a component and put it
 // in js/cr/ui/notification.js .
 
 cr.define('options', function() {
   /** @const */ var OptionsPage = options.OptionsPage;
   /** @const */ var LanguageList = options.LanguageList;
@@ skipping 408 matching lines @@
       if (!cr.isMac)
         this.updateSpellCheckLanguageButton_(languageCode);
 
       if (cr.isChromeOS)
         this.updateInputMethodList_(languageCode);
 
       this.updateLanguageListInAddLanguageOverlay_();
     },
 
     /**
-     * Happens when a user changes back to the language they're currently using.
-     */
-    currentLocaleWasReselected: function() {
-      this.updateUiLanguageButton_(
-          loadTimeData.getString('currentUiLanguageCode'));
-    },
-
-    /**
      * Handles languageOptionsList's save event.
      * @param {Event} e Save event.
      * @private
      */
     handleLanguageOptionsListSave_: function(e) {
       if (cr.isChromeOS) {
         // Sort the preload engines per the saved languages before save.
         this.preloadEngines_ = this.sortPreloadEngines_(this.preloadEngines_);
         this.savePreloadEnginesPref_();
       }
@@ skipping 399 matching lines @@
       if (checkbox.checked) {
         chrome.send('inputMethodEnable', [checkbox.inputMethodId]);
       } else {
         chrome.send('inputMethodDisable', [checkbox.inputMethodId]);
       }
       this.updatePreloadEnginesFromCheckboxes_();
       this.preloadEngines_ = this.sortPreloadEngines_(this.preloadEngines_);
       this.savePreloadEnginesPref_();
     },
 
-    handleAddLanguageOkButtonClick_: function() {
+    /**
+     * Handles clicks on the "OK" button of the "Add language" dialog.
+     * @param {Event} e Click event.
+     * @private
+     */
+    handleAddLanguageOkButtonClick_: function(e) {
       var languagesSelect = $('add-language-overlay-language-list');
       var selectedIndex = languagesSelect.selectedIndex;
       if (selectedIndex >= 0) {
         var selection = languagesSelect.options[selectedIndex];
         var langCode = String(selection.value);
         $('language-options-list').addLanguage(langCode);
         this.addBlockedLanguage_(langCode);
         OptionsPage.closeOverlay();
       }
     },
@@ skipping 318 matching lines @@
       notificationElement.onmouseout = delayedHide;
       actionLink.onfocus = show;
       actionLink.onblur = delayedHide;
       // Enable tabbing to the link now that it is shown.
       actionLink.tabIndex = 0;
 
       show();
       delayedHide();
     },
 
+    /**
+     * Chrome callback for when the UI language preference is saved.
+     * @param {string} languageCode The newly selected language to use.
+     * @private
+     */
+    uiLanguageSaved_: function(languageCode) {
+      this.prospectiveUiLanguageCode_ = languageCode;
+
+      // If the user is no longer on the same language code, ignore.
+      if ($('language-options-list').getSelectedLanguageCode() != languageCode)
+        return;
+
+      // Special case for when a user changes to a different language, and
+      // changes back to the same language without having restarted Chrome or
+      // logged in/out of ChromeOS.
+      if (languageCode == loadTimeData.getString('currentUiLanguageCode')) {
+        this.updateUiLanguageButton_(languageCode);
+        return;
+      }
+
+      // Otherwise, show a notification telling the user that their changes will
+      // only take effect after restart.
+      showMutuallyExclusiveNodes([$('language-options-ui-language-button'),
+                                  $('language-options-ui-notification-bar')],
+                                 1);
+    },
+
+    /**
+     * A handler for when dictionary for |languageCode| begins downloading.
+     * @param {string} languageCode The language of the dictionary that just
+     *     began downloading.
+     * @private
+     */
     onDictionaryDownloadBegin_: function(languageCode) {
       this.spellcheckDictionaryDownloadStatus_[languageCode] =
           DOWNLOAD_STATUS.IN_PROGRESS;
       if (!cr.isMac &&
           languageCode ==
               $('language-options-list').getSelectedLanguageCode()) {
         this.updateSpellCheckLanguageButton_(languageCode);
       }
     },
 
+    /**
+     * A handler for when dictionary for |languageCode| successfully downloaded.
+     * @param {string} languageCode The language of the dictionary that
+     *     succeeded downloading.
+     * @private
+     */
     onDictionaryDownloadSuccess_: function(languageCode) {
       delete this.spellcheckDictionaryDownloadStatus_[languageCode];
       this.spellcheckDictionaryDownloadFailures_ = 0;
       if (!cr.isMac &&
           languageCode ==
               $('language-options-list').getSelectedLanguageCode()) {
         this.updateSpellCheckLanguageButton_(languageCode);
       }
     },
 
+    /**
+     * A handler for when dictionary for |languageCode| fails to download.
+     * @param {string} languageCode The language of the dictionary that failed
+     *     to download.
+     * @private
+     */
     onDictionaryDownloadFailure_: function(languageCode) {
       this.spellcheckDictionaryDownloadStatus_[languageCode] =
           DOWNLOAD_STATUS.FAILED;
       this.spellcheckDictionaryDownloadFailures_++;
       if (!cr.isMac &&
           languageCode ==
               $('language-options-list').getSelectedLanguageCode()) {
         this.updateSpellCheckLanguageButton_(languageCode);
       }
     },
@@ skipping 35 matching lines @@
    * @param {number} index The index of |nodes| to show.
    */
   function showMutuallyExclusiveNodes(nodes, index) {
     assert(index >= 0 && index < nodes.length);
     for (var i = 0; i < nodes.length; ++i) {
       assert(nodes[i] instanceof HTMLElement);  // TODO(dbeam): Ignore null?
       nodes[i].hidden = i != index;
     }
   }
 
-  /**
-   * Chrome callback for when the UI language preference is saved.
-   * @param {string} languageCode The newly selected language to use.
-   */
   LanguageOptions.uiLanguageSaved = function(languageCode) {
-    this.getInstance().prospectiveUiLanguageCode_ = languageCode;
-
-    // If the user is no longer on the same language code, ignore.
-    if ($('language-options-list').getSelectedLanguageCode() != languageCode)
-      return;
-
-    // Special case for when a user changes to a different language, and changes
-    // back to the same language without having restarted Chrome or logged
-    // in/out of ChromeOS.
-    if (languageCode == loadTimeData.getString('currentUiLanguageCode')) {
-      LanguageOptions.getInstance().currentLocaleWasReselected();
-      return;
-    }
-
-    // Otherwise, show a notification telling the user that their changes will
-    // only take effect after restart.
-    showMutuallyExclusiveNodes([$('language-options-ui-language-button'),
-                                $('language-options-ui-notification-bar')], 1);
+    LanguageOptions.getInstance().uiLanguageSaved_(languageCode);
   };
 
   LanguageOptions.onDictionaryDownloadBegin = function(languageCode) {
     LanguageOptions.getInstance().onDictionaryDownloadBegin_(languageCode);
   };
 
   LanguageOptions.onDictionaryDownloadSuccess = function(languageCode) {
     LanguageOptions.getInstance().onDictionaryDownloadSuccess_(languageCode);
   };
 
   LanguageOptions.onDictionaryDownloadFailure = function(languageCode) {
     LanguageOptions.getInstance().onDictionaryDownloadFailure_(languageCode);
   };
 
   LanguageOptions.onComponentManagerInitialized = function(componentImes) {
     LanguageOptions.getInstance().appendComponentExtensionIme_(componentImes);
   };
 
   // Export
   return {
     LanguageOptions: LanguageOptions
   };
 });