Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(536)

Unified Diff: chrome/test/data/webui/md_bookmarks/toolbar_test.js

Issue 2893833002: MD Bookmarks: Enable the delete button in the toolbar overlay (Closed)
Patch Set: Fix nit Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/test/data/webui/md_bookmarks/toolbar_test.js
diff --git a/chrome/test/data/webui/md_bookmarks/toolbar_test.js b/chrome/test/data/webui/md_bookmarks/toolbar_test.js
index 1cd78ffb9c9a0bae3d3f9f503ffb0e212e8e7ca5..8e5d470d38f24d0d61ec43de90435a956eb2ea9e 100644
--- a/chrome/test/data/webui/md_bookmarks/toolbar_test.js
+++ b/chrome/test/data/webui/md_bookmarks/toolbar_test.js
@@ -5,6 +5,11 @@
suite('<bookmarks-toolbar>', function() {
var toolbar;
var store;
+ var commandManager;
+
+ suiteSetup(function() {
+ chrome.bookmarkManagerPrivate.removeTrees = function() {};
+ });
setup(function() {
store = new bookmarks.TestStore({
@@ -21,6 +26,9 @@ suite('<bookmarks-toolbar>', function() {
toolbar = document.createElement('bookmarks-toolbar');
replaceBody(toolbar);
+
+ commandManager = new TestCommandManager();
+ document.body.appendChild(commandManager);
});
test('selecting multiple items shows toolbar overlay', function() {
@@ -34,4 +42,15 @@ suite('<bookmarks-toolbar>', function() {
store.notifyObservers();
assertTrue(toolbar.showSelectionOverlay);
});
+
+ test('clicking overlay delete button triggers a delete command', function() {
+ store.data.selection.items = new Set(['2', '3']);
+ store.notifyObservers();
+
+ Polymer.dom.flush();
+ MockInteractions.tap(
+ toolbar.$$('cr-toolbar-selection-overlay').deleteButton);
+
+ commandManager.assertLastCommand(Command.DELETE, ['2', '3']);
+ });
});

Powered by Google App Engine
This is Rietveld 408576698