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

chrome/browser/ui/passwords/manage_passwords_icon.h

 // Copyright 2014 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_UI_PASSWORDS_MANAGE_PASSWORDS_ICON_H_
 #define CHROME_BROWSER_UI_PASSWORDS_MANAGE_PASSWORDS_ICON_H_
 
 #include "base/basictypes.h"
 #include "components/password_manager/core/common/password_manager_ui.h"
 
@@ skipping 10 matching lines @@
   bool active() const { return active_; }
 
  protected:
   // The ID of the icon and text resources that are currently displayed.
   int icon_id_;
   int tooltip_text_id_;
 
   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();
 
   password_manager::ui::State state_;
   bool active_;
 
   DISALLOW_COPY_AND_ASSIGN(ManagePasswordsIcon);
 };
 
 #endif  // CHROME_BROWSER_UI_PASSWORDS_MANAGE_PASSWORDS_ICON_H_