MD Bookmarks: Enable the delete button in the toolbar overlay

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(
@@ skipping 13 matching lines @@
                 createItem('13', {url: 'http://13/'}),
               ]),
           createFolder(
               '2',
               [
                 createFolder('21', []),
               ]))
     });
     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);
-    };
+    commandManager = new TestCommandManager();
     replaceBody(commandManager);
 
     Polymer.dom.flush();
   });
 
   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'])));
   });
@@ skipping 15 matching lines @@
     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']);
+    commandManager.assertLastCommand('copy', ['13']);
 
     // Doesn't trigger when a folder is selected.
     store.data.selection.items = new Set(['11']);
     store.notifyObservers();
 
     MockInteractions.pressAndReleaseKeyOn(document, 67, modifier, 'c');
-    assertLastCommand(null);
+    commandManager.assertLastCommand(null);
 
     // Doesn't trigger when nothing is selected.
     store.data.selection.items = new Set();
     store.notifyObservers();
 
     MockInteractions.pressAndReleaseKeyOn(document, 67, modifier, 'c');
-    assertLastCommand(null);
+    commandManager.assertLastCommand(null);
   });
 
   test('delete command triggers', function() {
     store.data.selection.items = new Set(['12', '13']);
     store.notifyObservers();
 
     MockInteractions.pressAndReleaseKeyOn(document, 46, '', 'Delete');
-    assertLastCommand('delete', ['12', '13']);
+    commandManager.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']);
+    commandManager.assertLastCommand('edit', ['11']);
   });
 
   test('does not delete children at same time as ancestor', function() {
     var lastDelete = null;
     chrome.bookmarkManagerPrivate.removeTrees = function(idArray) {
       lastDelete = idArray.sort();
     };
 
     var parentAndChildren = new Set(['1', '2', '12', '111']);
     assertTrue(commandManager.canExecute(Command.DELETE, parentAndChildren));
@@ skipping 13 matching lines @@
   test('shift-enter opens URLs in new window', function() {
     store.data.selection.items = new Set(['12', '13']);
     store.notifyObservers();
 
     var lastCreate;
     chrome.windows.create = function(createConfig) {
       lastCreate = createConfig;
     };
 
     MockInteractions.pressAndReleaseKeyOn(document, 13, 'shift', 'Enter');
-    assertLastCommand(Command.OPEN_NEW_WINDOW, ['12', '13']);
+    commandManager.assertLastCommand(Command.OPEN_NEW_WINDOW, ['12', '13']);
     assertDeepEquals(['http://121/', 'http://13/'], lastCreate.url);
     assertFalse(lastCreate.incognito);
   });
 
   test('cannot execute "Open in New Tab" on folders with no items', function() {
     var items = new Set(['2']);
     assertFalse(commandManager.canExecute(Command.OPEN_NEW_TAB, items));
 
     store.data.selection.items = items;
 
@@ skipping 83 matching lines @@
     assertOpenedTabs(['http://111/', 'http://12/']);
   });
 
   test('control-double click opens full selection', function() {
     customClick(items[0]);
     simulateDoubleClick(items[2], {ctrlKey: true});
 
     assertOpenedTabs(['http://111/', 'http://13/']);
   });
 });