[MD Bookmarks] Add edit folder.

chrome/browser/resources/md_bookmarks/list.js

 // Copyright 2016 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.
 
 Polymer({
   is: 'bookmarks-list',
 
   properties: {
     /** @type {BookmarkTreeNode} */
     menuItem_: Object,
 
     /** @type {Array<BookmarkTreeNode>} */
     displayedList: Array,
 
     searchTerm: String,
+
+    hideUrlInput_: Boolean,
   },
 
   listeners: {
     'open-item-menu': 'onOpenItemMenu_',
   },
 
   /**
    * @param {Event} e
    * @private
    */
   onOpenItemMenu_: function(e) {
     this.menuItem_ = e.detail.item;
     var menu = /** @type {!CrActionMenuElement} */ (
         this.$.dropdown);
     menu.showAt(/** @type {!Element} */ (e.detail.target));
   },
 
   // TODO(jiaxi): change these dummy click event handlers later.

[tsergeant, 2017/02/01 06:04:21]

Can you get rid of this TODO while you're here? It doesn't apply anymore.

[angelayang, 2017/02/01 07:19:33]

Done.

   /** @private */
   onEditTap_: function() {
     this.closeDropdownMenu_();
-    if (this.menuItem_.url)
-      this.$.editBookmark.showModal();
+    this.hideUrlInput_ = !this.menuItem_.url;
+    this.$.editBookmark.showModal();
   },
 
   /** @private */
   onCopyURLTap_: function() {
     var idList = [this.menuItem_.id];
     chrome.bookmarkManagerPrivate.copy(idList, function() {
       // TODO(jiaxi): Add toast later.
     });
     this.closeDropdownMenu_();
   },
 
   /** @private */
   onDeleteTap_: function() {
     if (this.menuItem_.url) {
       chrome.bookmarks.remove(this.menuItem_.id, function() {
         // TODO(jiaxi): Add toast later.
       }.bind(this));
     } else {
       chrome.bookmarks.removeTree(this.menuItem_.id, function() {
         // TODO(jiaxi): Add toast later.
       }.bind(this));
     }
     this.closeDropdownMenu_();
   },
 
   /** @private */
   onSaveEditTap_: function() {
     chrome.bookmarks.update(this.menuItem_.id, {

[tsergeant, 2017/02/01 06:04:21]

I think you need to make it so that 'url' isn't sent for folder renames. I just
tried running chrome.bookmarks.update in devtools to change the URL of a folder,
and it completely crashed my browser, which is something we'd like to avoid.

[angelayang, 2017/02/01 07:19:33]

Done.

       'title': this.menuItem_.title,
       'url': this.menuItem_.url,
     });
     this.$.editBookmark.close();
   },
 
   /** @private */
   onCancelEditTap_: function() {
     this.$.editBookmark.cancel();
   },
 
   /** @private */
   closeDropdownMenu_: function() {
     var menu = /** @type {!CrActionMenuElement} */ (
         this.$.dropdown);
     menu.close();
   },
 
   /** @private */
+  getEditorTitle_: function() {
+    var title = this.menuItem_.url ? 'editBookmarkTitle' : 'editFolderTitle';
+    return loadTimeData.getString(title);
+  },
+
+  /** @private */
   emptyListMessage_: function() {
     var emptyListMessage = this.searchTerm ? 'noSearchResults' : 'emptyList';
     return loadTimeData.getString(emptyListMessage);
   },
 
   /** @private */
   isEmptyList_: function() {
     return this.displayedList.length == 0;
   },
 });