DOMUI Prefs: Implement inline editability for startup pages

chrome/browser/resources/options/browser_options_startup_page_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.
 
 cr.define('options.browser_options', function() {
-  const DeletableItemList = options.DeletableItemList;
-  const DeletableItem = options.DeletableItem;
+  const InlineEditableItem = options.InlineEditableItem;
+  const InlineEditableItemList = options.InlineEditableItemList;
 
   /**
    * Creates a new startup page list item.
    * @param {Object} pageInfo The page this item represents.
    * @constructor
    * @extends {cr.ui.ListItem}
    */
   function StartupPageListItem(pageInfo) {
     var el = cr.doc.createElement('div');
     el.pageInfo_ = pageInfo;
     StartupPageListItem.decorate(el);
     return el;
   }
 
   /**
    * Decorates an element as a startup page list item.
    * @param {!HTMLElement} el The element to decorate.
    */
   StartupPageListItem.decorate = function(el) {
     el.__proto__ = StartupPageListItem.prototype;
     el.decorate();
   };
 
   StartupPageListItem.prototype = {
-    __proto__: DeletableItem.prototype,
+    __proto__: InlineEditableItem.prototype,
+
+    /**
+     * Input field for editing the page url.
+     * @type {HTMLElement}
+     * @private
+     */
+    urlField_: null,
 
     /** @inheritDoc */
     decorate: function() {
-      DeletableItem.prototype.decorate.call(this);
+      InlineEditableItem.prototype.decorate.call(this);
 
-      var titleEl = this.ownerDocument.createElement('span');
+      var titleEl = this.ownerDocument.createElement('div');
       titleEl.className = 'title';
       titleEl.classList.add('favicon-cell');
       titleEl.textContent = this.pageInfo_['title'];
       titleEl.style.backgroundImage = url('chrome://favicon/' +
                                           this.pageInfo_['url']);
       titleEl.title = this.pageInfo_['tooltip'];
 
       this.contentElement.appendChild(titleEl);
+
+      var urlEl = this.createEditableTextCell(this.pageInfo_['url']);
+      urlEl.className = 'url';
+      this.contentElement.appendChild(urlEl);
+
+      this.urlField_ = urlEl.querySelector('input');
+      this.urlField_.required = true;
+
+      this.addEventListener('commitedit', this.onEditCommitted_.bind(this));
+    },
+
+    /** @inheritDoc */
+    get currentInputIsValid() {
+      return this.urlField_.validity.valid;
+    },
+
+    /** @inheritDoc */
+    get hasBeenEdited() {
+      return this.urlField_.value != this.pageInfo_['url'];
+    },
+
+    /**
+     * Called when committing an edit; updates the model.
+     * @param {Event} e The end event.
+     * @private
+     */
+    onEditCommitted_: function(e) {
+      chrome.send('editStartupPage',
+                  [ this.pageInfo_['modelIndex'], this.urlField_.value ]);

[Evan Stade, 2011/01/13 23:12:46]

are the spaces around the [] typical? you don't use them below, for example

[stuartmorgan, 2011/01/13 23:35:37]

Done.

     },
   };
 
   var StartupPageList = cr.ui.define('list');
 
   StartupPageList.prototype = {
-    __proto__: DeletableItemList.prototype,
+    __proto__: InlineEditableItemList.prototype,
 
     /** @inheritDoc */
     createItem: function(pageInfo) {
       return new StartupPageListItem(pageInfo);
     },
 
     /** @inheritDoc */
     deleteItemAtIndex: function(index) {
       chrome.send('removeStartupPages', [String(index)]);
     },
   };
 
   return {
     StartupPageList: StartupPageList
   };
 });