Copy and Cut option of the context menu is not working via keyboard in chrome://bookmarks

chrome/browser/resources/bookmark_manager/js/main.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.
 
 (function() {
 /** @const */ var BookmarkList = bmm.BookmarkList;
 /** @const */ var BookmarkTree = bmm.BookmarkTree;
 /** @const */ var Command = cr.ui.Command;
 /** @const */ var LinkKind = cr.LinkKind;
 /** @const */ var ListItem = cr.ui.ListItem;
@@ skipping 1241 matching lines @@
   return filteredIds;
 }
 
 /**
  * Handler for the command event. This is used for context menu of list/tree
  * and organized menu.
  * @param {!Event} e The event object.
  */
 function handleCommand(e) {
   var command = e.command;
-  var target;
+  var target = assertInstanceof(e.target, HTMLElement);
   switch (command.id) {
     case 'import-menu-command':
       recordUserAction('Import');
       chrome.bookmarks.import();
       break;
 
     case 'export-menu-command':
       recordUserAction('Export');
       chrome.bookmarks.export();
       break;
 
     case 'undo-command':
       if (performGlobalUndo) {
         recordUserAction('UndoGlobal');
         performGlobalUndo();
       } else {
         recordUserAction('UndoNone');
       }
       break;
 
     case 'show-in-folder-command':
       recordUserAction('ShowInFolder');
       showInFolder();
       break;
 
     case 'open-in-new-tab-command':
     case 'open-in-background-tab-command':
       recordUserAction('OpenInNewTab');
-      openBookmarks(LinkKind.BACKGROUND_TAB,
-          assertInstanceof(e.target, HTMLElement));
+      openBookmarks(LinkKind.BACKGROUND_TAB, target);
       break;
 
     case 'open-in-new-window-command':
       recordUserAction('OpenInNewWindow');
-      openBookmarks(LinkKind.WINDOW,
-          assertInstanceof(e.target, HTMLElement));
+      openBookmarks(LinkKind.WINDOW, target);
       break;
 
     case 'open-incognito-window-command':
       recordUserAction('OpenIncognito');
-      openBookmarks(LinkKind.INCOGNITO,
-          assertInstanceof(e.target, HTMLElement));
+      openBookmarks(LinkKind.INCOGNITO, target);
       break;
 
     case 'delete-from-folders-menu-command':
-      target = bmm.tree;
     case 'delete-command':
       recordUserAction('Delete');
-      deleteBookmarks(target || assertInstanceof(e.target, HTMLElement));
+      deleteBookmarks(target);
       break;
 
     case 'copy-from-folders-menu-command':
-      target = bmm.tree;
     case 'copy-command':
       recordUserAction('Copy');
       chrome.bookmarkManagerPrivate.copy(getSelectedBookmarkIds(target),
                                          updatePasteCommand);
       break;
 
     case 'cut-from-folders-menu-command':
-      target = bmm.tree;
     case 'cut-command':
       recordUserAction('Cut');
       chrome.bookmarkManagerPrivate.cut(getSelectedBookmarkIds(target),
                                         function() {
                                           updatePasteCommand();
                                           updateSearchResults();
                                         });
       break;
 
     case 'paste-from-organize-menu-command':
       pasteBookmark(bmm.list.parentId);
       break;
 
     case 'paste-from-folders-menu-command':
       pasteBookmark(bmm.tree.selectedItem.bookmarkId);
       break;
 
     case 'paste-from-context-menu-command':
       pasteBookmark(getSelectedId());
       break;
 
     case 'sort-command':
       recordUserAction('Sort');
       chrome.bookmarkManagerPrivate.sortChildren(bmm.list.parentId);
       break;
 
     case 'rename-folder-command':
-      editItem(assertInstanceof(e.target, HTMLElement));
-      break;
-
     case 'rename-folder-from-folders-menu-command':
-      editItem(bmm.tree);

[Bernhard Bauer, 2015/04/20 10:28:27]

You want to keep this, so that the tree is always what is being edited. What you
could do is swap those two cases, and assign `target = bmm.tree;` then fall
through, like below.

+      editItem(target);
       break;
 
     case 'edit-command':
       recordUserAction('Edit');
       editItem();
       break;
 
     case 'new-folder-from-folders-menu-command':
-      target = bmm.tree;

[Bernhard Bauer, 2015/04/20 10:28:27]

You also want to keep this.

     case 'new-folder-command':
       recordUserAction('NewFolder');
       newFolder(target);
       break;
 
     case 'add-new-bookmark-command':
       recordUserAction('AddPage');
       addPage();
       break;
 
@@ skipping 144 matching lines @@
 
   cr.ui.FocusOutlineManager.forDocument(document);
   initializeSplitter();
   bmm.addBookmarkModelListeners();
   dnd.init(selectItemsAfterUserAction);
   bmm.tree.reload();
 }
 
 initializeBookmarkManager();
 })();