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

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

 // Copyright (c) 2012 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.
 
 // <include src="../../assert.js">
 
 cr.exportPath('cr.ui');
 
 /**
  * Enum for type of hide. Delayed is used when called by clicking on a
@@ skipping 232 matching lines @@
             break;
         case 'Enter':
         case 'U+0020': // Space
           if (!this.isMenuShown())
             this.showMenu(true);
           e.preventDefault();
           break;
         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;
       }
     }
   };
 
   /**
    * Helper for styling a menu button with a drop-down arrow indicator.
    * Creates a new 2D canvas context and draws a downward-facing arrow into it.
    * @param {string} canvasName The name of the canvas. The canvas can be
@@ skipping 37 matching lines @@
         'drop-down-arrow-hover', ARROW_WIDTH, ARROW_HEIGHT, opt_hoverColor);
     createDropDownArrowCanvas(
         'drop-down-arrow-active', ARROW_WIDTH, ARROW_HEIGHT, opt_activeColor);
   };
 
   // Export
   return {
     MenuButton: MenuButton,
   };
 });