Close the password bubble on the icon click.

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

Index: chrome/browser/ui/views/passwords/manage_passwords_icon_view.cc
diff --git a/chrome/browser/ui/views/passwords/manage_passwords_icon_view.cc b/chrome/browser/ui/views/passwords/manage_passwords_icon_view.cc
index 42457d001978e73f58b838eedd5bac30851dd331..7a9179dcf299fabf4438445ef5bc32e27d3b6674 100644
--- a/chrome/browser/ui/views/passwords/manage_passwords_icon_view.cc
+++ b/chrome/browser/ui/views/passwords/manage_passwords_icon_view.cc
@@ -68,3 +68,7 @@ bool ManagePasswordsIconView::IsBubbleShowing() const {
 void ManagePasswordsIconView::OnExecuting(
     BubbleIconView::ExecuteSource source) {
 }
+
+void ManagePasswordsIconView::OnMousePressSuppressed() {

[Peter Kasting, 2014/07/22 18:55:18]

Instead of adding a new virtual method with the somewhat confusing name of
OnMousePressSuppressed(), why not just override OnMousePressed() in this class,
have it call the superclass method to set the |suppress_mouse_released_action_|
variable correctly, then call CloseBubble()?

[vasilii, 2014/07/23 09:22:53]

Done.

+  ManagePasswordsBubbleView::CloseBubble();
+}