Hide the password bubble when the underlying ManagePasswordsUIController changes its state.

chrome/browser/ui/passwords/manage_passwords_icon.h

Index: chrome/browser/ui/passwords/manage_passwords_icon.h
diff --git a/chrome/browser/ui/passwords/manage_passwords_icon.h b/chrome/browser/ui/passwords/manage_passwords_icon.h
index 165374075da3b098742a3f6df8defcd96e47333f..e9e87150a3c0e66dcf1812c59cea70645f65fca1 100644
--- a/chrome/browser/ui/passwords/manage_passwords_icon.h
+++ b/chrome/browser/ui/passwords/manage_passwords_icon.h
@@ -28,10 +28,14 @@ class ManagePasswordsIcon {
   ManagePasswordsIcon();
   ~ManagePasswordsIcon();
 
-  // Called from SetState() iff the icon's state has changed in order to do
-  // whatever platform-specific UI work is necessary given the new state.
+  // Called from SetState() and SetActive() in order to do whatever
+  // platform-specific UI work is necessary.
   virtual void UpdateVisibleUI() = 0;
 
+  // Called from SetState() iff the icon's state has changed in order to close

[vabr (Chromium), 2015/03/25 11:43:24]

nit: Is the "in order to close the current bubble" part necessary? Could leaving
it out actually make the comment on the purpose of this hook clearer?

[vasilii, 2015/03/25 11:48:55]

Done.

+  // the current bubble.
+  virtual void OnChangingState() = 0;
+
  private:
   // Updates the resource IDs in response to state changes.
   void UpdateIDs();