Bookmark pop-up doesn't open if Ctrl+D is set as keyboard shortcut for added extensions.

chrome/browser/command_updater.h

 // 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.
 
 #ifndef CHROME_BROWSER_COMMAND_UPDATER_H_
 #define CHROME_BROWSER_COMMAND_UPDATER_H_
 
 #include "base/basictypes.h"
 #include "base/containers/hash_tables.h"
 #include "ui/base/window_open_disposition.h"
@@ skipping 44 matching lines @@
 
   // Removes |observer| for all commands on which it's registered.
   void RemoveCommandObserver(CommandObserver* observer);
 
   // Notify all observers of a particular command that the command has been
   // enabled or disabled. If the command does not exist, it is created and
   // initialized to |state|. This function is very lightweight if the command
   // state has not changed.
   void UpdateCommandEnabled(int id, bool state);
 
+  void set_bookmark_icon_selected(bool selection) {

[Avi (use Gerrit), 2014/11/28 14:56:07]

You need to have some comments explaining the meaning of this set of accessors
and of the variable below.

+    bookmark_icon_selection_ = selection;
+  }
+
+  bool get_bookmark_icon_selected() { return bookmark_icon_selection_; }
+
  private:
   // A piece of data about a command - whether or not it is enabled, and a list
   // of objects that observe the enabled state of this command.
   class Command;
 
   // Get a Command node for a given command ID, creating an entry if it doesn't
   // exist if desired.
   Command* GetCommand(int id, bool create);
 
   // The delegate is responsible for executing commands.
   CommandUpdaterDelegate* delegate_;
 
   // This is a map of command IDs to states and observer lists
   typedef base::hash_map<int, Command*> CommandMap;
   CommandMap commands_;
 
+  bool bookmark_icon_selection_;
+
   DISALLOW_COPY_AND_ASSIGN(CommandUpdater);
 };
 
 #endif  // CHROME_BROWSER_COMMAND_UPDATER_H_