MD Bookmarks: Prevent keyboard shortcuts when the toolbar/dialogs are focused

chrome/browser/resources/md_bookmarks/list.js

Index: chrome/browser/resources/md_bookmarks/list.js
diff --git a/chrome/browser/resources/md_bookmarks/list.js b/chrome/browser/resources/md_bookmarks/list.js
index 75810eaaddfd5c0710b7cc919e29a4d1c6105137..8a2d92233e722b1b625d060703606effca3440ff 100644
--- a/chrome/browser/resources/md_bookmarks/list.js
+++ b/chrome/browser/resources/md_bookmarks/list.js
@@ -177,6 +177,16 @@ Polymer({
       }
     }
 
+    // Prevent the iron-list from changing focus on enter.
+    if (e.path[0] instanceof HTMLButtonElement && e.key == 'Enter') {

[calamity, 2017/06/06 07:15:11]

🤡🎪

Remove braces.

Might be worth writing a test for this.

[tsergeant, 2017/06/06 08:10:48]

Done.

+      handled = true;
+    }
+
+    if (!handled) {
+      handled = bookmarks.CommandManager.getInstance().handleKeyEvent(
+          e, this.getState().selection.items);
+    }
+
     if (handled)
       e.stopPropagation();
   },