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

chrome/browser/resources/options/language_list.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 */ var ArrayDataModel = cr.ui.ArrayDataModel;
   /** @const */ var DeletableItem = options.DeletableItem;
   /** @const */ var DeletableItemList = options.DeletableItemList;
   /** @const */ var List = cr.ui.List;
   /** @const */ var ListItem = cr.ui.ListItem;
@@ skipping 58 matching lines @@
     if (!this.languageCodeToLanguageInfo_) {
       this.languageCodeToLanguageInfo_ = {};
       var languageList = loadTimeData.getValue('languageList');
       for (var i = 0; i < languageList.length; i++) {
         var languageInfo = languageList[i];
         this.languageCodeToLanguageInfo_[languageInfo.code] = languageInfo;
       }
     }
 
     return this.languageCodeToLanguageInfo_[languageCode];
-  }
+  };
 
   /**
    * Returns true if the given language code is valid.
    * @param {string} languageCode Language code (ex. "fr").
    */
   LanguageList.isValidLanguageCode = function(languageCode) {
     // Having the display name for the language code means that the
     // language code is valid.
     if (LanguageList.getLanguageInfoFromLanguageCode(languageCode)) {
       return true;
     }
     return false;
-  }
+  };
 
   LanguageList.prototype = {
     __proto__: DeletableItemList.prototype,
 
     // The list item being dragged.
     draggedItem: null,
     // The drop position information: "below" or "above".
     dropPos: null,
     // The preference is a CSV string that describes preferred languages
     // in Chrome OS. The language list is used for showing the language
@@ skipping 40 matching lines @@
     updateDeletable: function() {
       var items = this.items;
       for (var i = 0; i < items.length; ++i) {
         var item = items[i];
         var languageCode = item.languageCode;
         var languageOptions = options.LanguageOptions.getInstance();
         item.deletable = languageOptions.languageIsDeletable(languageCode);
       }
     },
 
-    /*
+    /**
      * Adds a language to the language list.
      * @param {string} languageCode language code (ex. "fr").
      */
     addLanguage: function(languageCode) {
       // It shouldn't happen but ignore the language code if it's
       // null/undefined, or already present.
       if (!languageCode || this.dataModel.indexOf(languageCode) >= 0) {
         return;
       }
       this.dataModel.push(languageCode);
       // Select the last item, which is the language added.
       this.selectionModel.selectedIndex = this.dataModel.length - 1;
 
       this.savePreference_();
     },
 
-    /*
+    /**
      * Gets the language codes of the currently listed languages.
      */
     getLanguageCodes: function() {
       return this.dataModel.slice();
     },
 
-    /*
+    /**
      * Clears the selection
      */
     clearSelection: function() {
       this.selectionModel.unselectAll();
     },
 
-    /*
+    /**
      * Gets the language code of the selected language.
      */
     getSelectedLanguageCode: function() {
       return this.selectedItem;
     },
 
-    /*
+    /**
      * Selects the language by the given language code.
-     * @returns {boolean} True if the operation is successful.
+     * @return {boolean} True if the operation is successful.
      */
     selectLanguageByCode: function(languageCode) {
       var index = this.dataModel.indexOf(languageCode);
       if (index >= 0) {
         this.selectionModel.selectedIndex = index;
         return true;
       }
       return false;
     },
 
     /** @override */
     deleteItemAtIndex: function(index) {
       if (index >= 0) {
         this.dataModel.splice(index, 1);
         // Once the selected item is removed, there will be no selected item.
         // Select the item pointed by the lead index.
         index = this.selectionModel.leadIndex;
         this.savePreference_();
       }
       return index;
     },
 
-    /*
+    /**
      * Computes the target item of drop event.
      * @param {Event} e The drop or dragover event.
      * @private
      */
     getTargetFromDropEvent_: function(e) {
       var target = e.target;
       // e.target may be an inner element of the list item
       while (target != null && !(target instanceof ListItem)) {
         target = target.parentNode;
       }
       return target;
     },
 
-    /*
+    /**
      * Handles the dragstart event.
      * @param {Event} e The dragstart event.
      * @private
      */
     handleDragStart_: function(e) {
       var target = e.target;
       // ListItem should be the only draggable element type in the page,
       // but just in case.
       if (target instanceof ListItem) {
         this.draggedItem = target;
         e.dataTransfer.effectAllowed = 'move';
         // We need to put some kind of data in the drag or it will be
         // ignored.  Use the display name in case the user drags to a text
         // field or the desktop.
         e.dataTransfer.setData('text/plain', target.title);
       }
     },
 
-    /*
+    /**
      * Handles the dragenter event.
      * @param {Event} e The dragenter event.
      * @private
      */
     handleDragEnter_: function(e) {
       e.preventDefault();
     },
 
-    /*
+    /**
      * Handles the dragover event.
      * @param {Event} e The dragover event.
      * @private
      */
     handleDragOver_: function(e) {
       var dropTarget = this.getTargetFromDropEvent_(e);
       // Determines whether the drop target is to accept the drop.
       // The drop is only successful on another ListItem.
       if (!(dropTarget instanceof ListItem) ||
           dropTarget == this.draggedItem) {
         this.hideDropMarker_();
         return;
       }
       // Compute the drop postion. Should we move the dragged item to
       // below or above the drop target?
       var rect = dropTarget.getBoundingClientRect();
       var dy = e.clientY - rect.top;
       var yRatio = dy / rect.height;
       var dropPos = yRatio <= .5 ? 'above' : 'below';
       this.dropPos = dropPos;
       this.showDropMarker_(dropTarget, dropPos);
       e.preventDefault();
     },
 
-    /*
+    /**
      * Handles the drop event.
      * @param {Event} e The drop event.
      * @private
      */
     handleDrop_: function(e) {
       var dropTarget = this.getTargetFromDropEvent_(e);
       this.hideDropMarker_();
 
       // Delete the language from the original position.
       var languageCode = this.draggedItem.languageCode;
       var originalIndex = this.dataModel.indexOf(languageCode);
       this.dataModel.splice(originalIndex, 1);
       // Insert the language to the new position.
       var newIndex = this.dataModel.indexOf(dropTarget.languageCode);
       if (this.dropPos == 'below')
         newIndex += 1;
       this.dataModel.splice(newIndex, 0, languageCode);
       // The cursor should move to the moved item.
       this.selectionModel.selectedIndex = newIndex;
       // Save the preference.
       this.savePreference_();
     },
 
-    /*
+    /**
      * Handles the dragleave event.
      * @param {Event} e The dragleave event
      * @private
      */
     handleDragLeave_: function(e) {
       this.hideDropMarker_();
     },
 
-    /*
+    /**
      * Shows and positions the marker to indicate the drop target.
      * @param {HTMLElement} target The current target list item of drop
      * @param {string} pos 'below' or 'above'
      * @private
      */
     showDropMarker_: function(target, pos) {
       window.clearTimeout(this.hideDropMarkerTimer_);
       var marker = $('language-options-list-dropmarker');
       var rect = target.getBoundingClientRect();
       var markerHeight = 8;
       if (pos == 'above') {
         marker.style.top = (rect.top - markerHeight / 2) + 'px';
       } else {
         marker.style.top = (rect.bottom - markerHeight / 2) + 'px';
       }
       marker.style.width = rect.width + 'px';
       marker.style.left = rect.left + 'px';
       marker.style.display = 'block';
     },
 
-    /*
+    /**
      * Hides the drop marker.
      * @private
      */
     hideDropMarker_: function() {
       // Hide the marker in a timeout to reduce flickering as we move between
       // valid drop targets.
       window.clearTimeout(this.hideDropMarkerTimer_);
       this.hideDropMarkerTimer_ = window.setTimeout(function() {
         $('language-options-list-dropmarker').style.display = '';
       }, 100);
@@ skipping 83 matching lines @@
       }
       return filteredLanguageCodes;
     },
   };
 
   return {
     LanguageList: LanguageList,
     LanguageListItem: LanguageListItem
   };
 });