Pressing 'Delete' key in bookmark manager's search box deletes bookmark.

chrome/browser/resources/bookmark_manager/js/main.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.
 
 (function() {
 /** @const */ var BookmarkList = bmm.BookmarkList;
 /** @const */ var BookmarkTree = bmm.BookmarkTree;
 /** @const */ var Command = cr.ui.Command;
 /** @const */ var LinkKind = cr.LinkKind;
 /** @const */ var ListItem = cr.ui.ListItem;
@@ skipping 452 matching lines @@
     else if (document.activeElement == bmm.tree)
       cmd = 'paste-from-folders-menu-command';
 
     if (cmd)
       update('paste-from-context-menu-command', !$(cmd).disabled);
 
     if (opt_f) opt_f();
   });
 }
 
+function handleCanExecuteForSearchBox(e) {
+  var command = e.command;
+  switch (command.id) {
+    case 'delete-command':
+    case 'undo-command':
+      // Do nothing when delete is selected in search box.
+      // Pass the undo command through (fixes http://crbug.com/278112).
+      // Otherwise, because the global undo command has no visible UI, always

[Bernhard Bauer, 2015/04/15 14:54:45]

Remove this sentence (see my comment above).

[Deepak, 2015/04/15 15:04:15]

Done.

+      // enable it, and just make it a no-op if undo is not possible.
+      e.canExecute = false;
+      break;
+  }
+}
+
 function handleCanExecuteForDocument(e) {
   var command = e.command;
   switch (command.id) {
     case 'import-menu-command':
       e.canExecute = canEdit;
       break;
 
     case 'export-menu-command':
       // We can always execute the export-menu command.
       e.canExecute = true;
       break;
 
     case 'sort-command':
       e.canExecute = !bmm.list.isSearch() &&
           bmm.list.dataModel && bmm.list.dataModel.length > 1 &&
           !isUnmodifiable(bmm.tree.getBookmarkNodeById(bmm.list.parentId));
       break;
 
-    case 'undo-command':
-      // If the search box is active, pass the undo command through
-      // (fixes http://crbug.com/278112). Otherwise, because
-      // the global undo command has no visible UI, always enable it, and
-      // just make it a no-op if undo is not possible.
-      e.canExecute = e.currentTarget.activeElement !== $('term');

[Bernhard Bauer, 2015/04/15 14:54:45]

For the undo command we need to keep the behavior if the search field is _not_
selected. Thanks to the way events are propagated, this only gets invoked if the
search field handler did not set canExecute, so you know that the search field
is not selected, so you can set it unconditionally to true. You can also keep
the second part of the comment (everything after "Otherwise").

[Deepak, 2015/04/15 15:04:14]

Done.

-      break;
-
     default:
       canExecuteForList(e);
       if (!e.defaultPrevented)
         canExecuteForTree(e);
       break;
   }
 }
 
 /**
  * Helper function for handling canExecute for the list and the tree.
@@ skipping 948 matching lines @@
   // when // the user goes back and forward in the history.
   window.addEventListener('hashchange', processHash);
 
   document.querySelector('header form').onsubmit =
       /** @type {function(Event=)} */(function(e) {
     setSearch($('term').value);
     e.preventDefault();
   });
 
   $('term').addEventListener('search', handleSearch);
+  $('term').addEventListener('canExecute', handleCanExecuteForSearchBox);
 
   $('folders-button').addEventListener('click', handleMenuButtonClicked);
   $('organize-button').addEventListener('click', handleMenuButtonClicked);
 
   document.addEventListener('canExecute', handleCanExecuteForDocument);
   document.addEventListener('command', handleCommand);
 
   // Listen to copy, cut and paste events and execute the associated commands.
   installEventHandlerForCommand('copy', 'copy-command');
   installEventHandlerForCommand('cut', 'cut-command');
@@ skipping 29 matching lines @@
 
   cr.ui.FocusOutlineManager.forDocument(document);
   initializeSplitter();
   bmm.addBookmarkModelListeners();
   dnd.init(selectItemsAfterUserAction);
   bmm.tree.reload();
 }
 
 initializeBookmarkManager();
 })();