MD Bookmarks: Add keyboard shortcut support to bookmarks-command-manager

chrome/test/data/webui/md_bookmarks/command_manager_test.js

 // Copyright 2017 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.
 
 suite('<bookmarks-command-manager>', function() {
   var commandManager;
   var store;
+  var lastCommand;
+  var lastCommandIds;
+
+  function assertLastCommand(command, ids) {
+    assertEquals(lastCommand, command);
+    if (ids)
+      assertDeepEquals(normalizeSet(lastCommandIds), ids);
+    lastCommand = null;
+    lastCommandIds = null;
+  }
+
+  suiteSetup(function() {
+    // Overwrite bookmarkManagerPrivate APIs which will crash if called with
+    // fake data.
+    chrome.bookmarkManagerPrivate.copy = function() {};
+    chrome.bookmarkManagerPrivate.removeTrees = function() {};
+  });
 
   setup(function() {
     store = new bookmarks.TestStore({
       nodes: testTree(createFolder(
           '1',
           [
             createFolder('11', []),
             createFolder('12', []),
             createItem('13'),
           ])),
     });
     bookmarks.Store.instance_ = store;
 
     commandManager = document.createElement('bookmarks-command-manager');
+
+    var realHandle = commandManager.handle.bind(commandManager);
+    commandManager.handle = function(command, itemIds) {
+      lastCommand = command;
+      lastCommandIds = itemIds;
+      realHandle(command, itemIds);
+    };
     replaceBody(commandManager);
   });
 
   test('can only copy single URL items', function() {
     assertFalse(commandManager.canExecute(Command.COPY, new Set(['11'])));
     assertFalse(commandManager.canExecute(Command.COPY, new Set(['11', '13'])));
     assertTrue(commandManager.canExecute(Command.COPY, new Set(['13'])));
   });
 
   test('context menu hides invalid commands', function() {
     store.data.selection.items = new Set(['11', '13']);
     store.notifyObservers();
 
     commandManager.openCommandMenuAtPosition(0, 0);
     var commandHidden = {};
     commandManager.root.querySelectorAll('.dropdown-item').forEach(element => {
       commandHidden[element.getAttribute('command')] = element.hidden;
     });
 
     // With a folder and an item selected, the only available context menu item
     // is 'Delete'.
     assertTrue(commandHidden['edit']);
     assertTrue(commandHidden['copy']);
     assertFalse(commandHidden['delete']);
   });
+
+  test('keyboard shortcuts trigger when valid', function() {
+    var modifier = cr.isMac ? 'meta' : 'ctrl';
+
+    store.data.selection.items = new Set(['13']);
+    store.notifyObservers();
+
+    MockInteractions.pressAndReleaseKeyOn(document, 67, modifier, 'c');
+    assertLastCommand('copy', ['13']);
+
+    store.data.selection.items = new Set(['11']);
+    store.notifyObservers();
+
+    MockInteractions.pressAndReleaseKeyOn(document, 67, modifier, 'c');
+    assertLastCommand(null);

[calamity, 2017/04/26 10:07:58]

Add a case for empty selection too.

[tsergeant, 2017/04/27 01:42:51]

Done.

+  });
+
+  test('delete command triggers', function() {

[tsergeant, 2017/04/26 08:10:55]

I'm not totally convinced how useful this test and the next are, but I guess
they're worth at least having.

+    store.data.selection.items = new Set(['12', '13']);
+    store.notifyObservers();
+
+    MockInteractions.pressAndReleaseKeyOn(document, 46, '', 'Delete');
+    assertLastCommand('delete', ['12', '13']);
+  });
+
+  test('edit command triggers', function() {
+    var key = cr.isMac ? 'Enter' : 'F2';
+    var keyCode = cr.isMac ? 13 : 113;
+
+    store.data.selection.items = new Set(['11']);
+    store.notifyObservers();
+
+    MockInteractions.pressAndReleaseKeyOn(document, keyCode, '', key);
+    assertLastCommand('edit', ['11']);
+  });
 });