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).

[Bernhard Bauer, 2015/04/15 15:08:56]

"Pass the delete and undo commands through".

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

Done.

+      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.
+      // If search box is not active and the global undo command has no visible

[Bernhard Bauer, 2015/04/15 15:08:56]

You don't need to mention the part about the search field. Also, the undo
command always has no visible UI, so that's not an "if". Just use the old
phrasing without the "otherwise".

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

Done.

+      // UI, always enable it, and just make it a no-op if undo is not possible.
       e.canExecute = e.currentTarget.activeElement !== $('term');
       break;
 
     default:
       canExecuteForList(e);
       if (!e.defaultPrevented)
         canExecuteForTree(e);
       break;
   }
 }
@@ skipping 951 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();
 })();