Compile chrome://settings, part 3: 167 proper errors left

chrome/browser/resources/options/deletable_item_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 List = cr.ui.List;
   /** @const */ var ListItem = cr.ui.ListItem;
 
   /**
    * Creates a deletable list item, which has a button that will trigger a call
@@ skipping 26 matching lines @@
      * @private
      */
     deletable_: true,
 
     /** @override */
     decorate: function() {
       ListItem.prototype.decorate.call(this);
 
       this.classList.add('deletable-item');
 
-      this.contentElement_ = this.ownerDocument.createElement('div');
+      this.contentElement_ = /** @type {HTMLElement} */(
+          this.ownerDocument.createElement('div'));
       this.appendChild(this.contentElement_);
 
-      this.closeButtonElement_ = this.ownerDocument.createElement('button');
+      this.closeButtonElement_ = /** @type {HTMLElement} */(
+          this.ownerDocument.createElement('button'));
       this.closeButtonElement_.className =
           'raw-button row-delete-button custom-appearance';
       this.closeButtonElement_.addEventListener('mousedown',
                                                 this.handleMouseDownUpOnClose_);
       this.closeButtonElement_.addEventListener('mouseup',
                                                 this.handleMouseDownUpOnClose_);
       this.closeButtonElement_.addEventListener('focus',
                                                 this.handleFocus_.bind(this));
       this.closeButtonElement_.tabIndex = -1;
       this.closeButtonElement_.title =
@@ skipping 55 matching lines @@
      * @private
      */
     handleMouseDownUpOnClose_: function(e) {
       if (e.target.disabled)
         return;
       e.stopPropagation();
       e.preventDefault();
     },
   };
 
+  /**
+   * @constructor
+   * @extends {cr.ui.List}
+   */
   var DeletableItemList = cr.ui.define('list');
 
   DeletableItemList.prototype = {
     __proto__: List.prototype,
 
     /** @override */
     decorate: function() {
       List.prototype.decorate.call(this);
-      this.addEventListener('click', this.handleClick_);
+      this.addEventListener('click', this.handleClick);
       this.addEventListener('keydown', this.handleKeyDown_);
     },
 
     /**
      * Callback for onclick events.
      * @param {Event} e The click event object.
-     * @private
+     * @protected

[Dan Beam, 2014/09/11 22:51:08]

@override

[Vitaly Pavlenko, 2014/09/11 23:13:09]

Done.

      */
-    handleClick_: function(e) {
+    handleClick: function(e) {
       if (this.disabled)
         return;
 
       var target = e.target;
       if (target.classList.contains('row-delete-button')) {
         var listItem = this.getListItemAncestor(target);
         var idx = this.getIndexOfListItem(listItem);
         this.deleteItemAtIndex(idx);
       }
     },
@@ skipping 36 matching lines @@
      */
     deleteItemAtIndex: function(index) {
     },
   };
 
   return {
     DeletableItemList: DeletableItemList,
     DeletableItem: DeletableItem,
   };
 });