DOMUI: Implement the new-style Autofill options page.

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 251 matching lines @@
       this.beforeFiller_ = this.ownerDocument.createElement('div');
       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('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.
      * @private
      */
     getItemHeight_: function() {
       if (!this.itemHeight_)
         this.itemHeight_ = measureItem(this);
       return this.itemHeight_;
     },
 
     /**
+     * Callback for the double click event.
+     * @param {Event} e The mouse event object.
+     * @private
+     */
+    handleDoubleClick_: function(e) {
+      if (this.disabled)
+        return;
+
+      var target = this.getListItemAncestor(e.target);
+      var index = target ? this.getIndexOfListItem(target) : -1;
+      this.activateItemAtIndex(index);

[Evan Stade, 2011/01/12 00:46:12]

this is causing js errors in lists that don't implement the function. We might
add a stub in List, but currently only Autofill lists use this, and AFAIK other
list implementations don't need it. Can we just move it to the autofill list
class?

[James Hawkins, 2011/01/12 00:54:04]

I guess I forgot to stub it out; I'll add the stub.

+    },
+
+    /**
      * Callback for mousedown and mouseup events.
      * @param {Event} e The mouse event object.
      * @private
      */
     handleMouseDownUp_: function(e) {
       if (this.disabled)
         return;
 
       var target = e.target;
 
@@ skipping 290 matching lines @@
       }
     },
   };
 
   cr.defineProperty(List, 'disabled', cr.PropertyKind.BOOL_ATTR);
 
   return {
     List: List
   }
 });