Password bubble: Introduce a command to open the bubble.

chrome/browser/ui/views/passwords/manage_passwords_icon_view.h

Index: chrome/browser/ui/views/passwords/manage_passwords_icon_view.h
diff --git a/chrome/browser/ui/views/passwords/manage_passwords_icon_view.h b/chrome/browser/ui/views/passwords/manage_passwords_icon_view.h
index a215a73d6eb487085a716327e03e6832c9df2cc8..139bb6253e1445fd921d8afa50dd97e9551d5d34 100644
--- a/chrome/browser/ui/views/passwords/manage_passwords_icon_view.h
+++ b/chrome/browser/ui/views/passwords/manage_passwords_icon_view.h
@@ -9,6 +9,7 @@
 #include "chrome/browser/ui/views/location_bar/location_bar_view.h"
 #include "ui/views/controls/image_view.h"
 
+class CommandUpdater;
 class ManagePasswordsBubbleUIController;
 
 // View for the password icon in the Omnibox.
@@ -20,7 +21,8 @@ class ManagePasswordsIconView : public views::ImageView {
   // constructor. Instead, a LocationBarView::Delegate is passed here so that it
   // can be queried for the current WebContents as needed.
   explicit ManagePasswordsIconView(

[vabr (Chromium), 2014/04/22 14:32:06]

You can drop the "explicit".

-      LocationBarView::Delegate* location_bar_delegate);
+      LocationBarView::Delegate* location_bar_delegate,
+      CommandUpdater* command_updater);
   virtual ~ManagePasswordsIconView();
 
   // Updates the image and its tooltip appropriately, hiding or showing the icon
@@ -45,6 +47,10 @@ class ManagePasswordsIconView : public views::ImageView {
   // The delegate used to get the currently visible WebContents.
   LocationBarView::Delegate* location_bar_delegate_;
 
+  // The updater used to deliver commands to the browser; we'll use this to
+  // pop open the bubble when necessary.
+  CommandUpdater* command_updater_;
+
   DISALLOW_COPY_AND_ASSIGN(ManagePasswordsIconView);
 };