DOMUI: Make Autofill profile lists multi-select and implementing deleting

chrome/browser/resources/shared/js/cr/ui/list.js

 // Copyright (c) 2010 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.
 
 // require: listselectionmodel.js
 
 /**
  * @fileoverview This implements a list control.
  */
 
@@ skipping 252 matching lines @@
       this.afterFiller_ = this.ownerDocument.createElement('div');
       this.beforeFiller_.className = 'spacer';
       this.afterFiller_.className = 'spacer';
       this.appendChild(this.beforeFiller_);
       this.appendChild(this.afterFiller_);
 
       var length = this.dataModel ? this.dataModel.length : 0;
       this.selectionModel = new ListSelectionModel(length);
 
       this.addEventListener('dblclick', this.handleDoubleClick_);
-      this.addEventListener('mousedown', this.handleMouseDownUp_);
-      this.addEventListener('mouseup', this.handleMouseDownUp_);
+      this.addEventListener('mousedown', this.handleMouseDownUp_, false);
+      this.addEventListener('mouseup', this.handleMouseDownUp_, false);

[stuartmorgan, 2011/01/06 19:27:33]

Isn't false the default? Why do we need this here but not on the other
listeners?

[James Hawkins, 2011/01/06 20:06:43]

Removing per off-list conversation.

       this.addEventListener('keydown', this.handleKeyDown);
       this.addEventListener('scroll', this.redraw.bind(this));
 
       // Make list focusable
       if (!this.hasAttribute('tabindex'))
         this.tabIndex = 0;
     },
 
     /**
      * Returns the height of an item, measuring it if necessary.
@@ skipping 323 matching lines @@
       }
     },
   };
 
   cr.defineProperty(List, 'disabled', cr.PropertyKind.BOOL_ATTR);
 
   return {
     List: List
   }
 });