Context menu is not getting hided in chrome://bookmarks after delete key press.

ui/webui/resources/js/cr/ui/menu_button.js

Index: ui/webui/resources/js/cr/ui/menu_button.js
diff --git a/ui/webui/resources/js/cr/ui/menu_button.js b/ui/webui/resources/js/cr/ui/menu_button.js
index 8e27b8af40ba8b55c84600664f72bf3b2e438930..df6bd8f9e3e7c174ce741db65cdedbe9330dbf5a 100644
--- a/ui/webui/resources/js/cr/ui/menu_button.js
+++ b/ui/webui/resources/js/cr/ui/menu_button.js
@@ -250,6 +250,7 @@ cr.define('cr.ui', function() {
         case 'Esc':
         case 'U+001B': // Maybe this is remote desktop playing a prank?
         case 'U+0009': // Tab
+        case 'U+007F': // Delete

[Bernhard Bauer, 2015/04/20 10:46:40]

This doesn't seem like the right place to hide the menu. Note that this is a
general-purpose context menu which doesn't know that pressing the Delete key
will delete a bookmark.

Overall, I think the behavior for context menus spawned by right-clicking (where
the menu gets all key events) is the correct one -- see also the comment in
context_menu_handler.js: "If the menu is visible we let it handle all the
keyboard events". That comment also appears in this file in line 124, but the
code does not actually match that behavior (preventDefault() and
stopPropagation() is only called if the menu does not handle the key press), so
I think that is what we should fix.

[Deepak, 2015/04/20 12:38:35]

Thanks for giving me good understanding, I got your point.
Earlier deletion was happening as event get bubbled up as 'delete' key was not
handled. Now we are preventing this.
Now we have same behavior for selecting 'Delete' key by keyboard when we have
context menu on 'Action' button by left/right mouse click selection.
I have updated description also.

           this.hideMenu();
           break;
       }