Bookmarked URL gets open in New tab, on pressing enter key for 'Folders' dropdown.

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 488 matching lines @@
           bmm.list.dataModel && bmm.list.dataModel.length > 1 &&
           !isUnmodifiable(bmm.tree.getBookmarkNodeById(bmm.list.parentId));
       break;
 
     case 'undo-command':
       // 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');
       break;
 
+    case 'open-in-same-window-command':
+      // open-in-same-window-command executes for list only.
+      e.canExecute = e.currentTarget.activeElement == bmm.list;

[Bernhard Bauer, 2015/04/16 09:01:19]

There is also a second check below in canExecuteForList, which we would never
reach if we set canExecute to true here. I think we could just move this down
and combine the checks? 

+      break;
+
     default:
       canExecuteForList(e);
       if (!e.defaultPrevented)
         canExecuteForTree(e);
       break;
   }
 }
 
 /**
  * Helper function for handling canExecute for the list and the tree.
@@ skipping 998 matching lines @@
 
   cr.ui.FocusOutlineManager.forDocument(document);
   initializeSplitter();
   bmm.addBookmarkModelListeners();
   dnd.init(selectItemsAfterUserAction);
   bmm.tree.reload();
 }
 
 initializeBookmarkManager();
 })();