Don't close the password bubble on implicit navigations.

chrome/browser/ui/passwords/manage_passwords_ui_controller.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_UI_CONTROLLER_H_
 #define CHROME_BROWSER_UI_PASSWORDS_MANAGE_PASSWORDS_UI_CONTROLLER_H_
 
 #include "base/macros.h"
 #include "chrome/browser/ui/passwords/manage_passwords_state.h"
 #include "chrome/browser/ui/passwords/passwords_client_ui_delegate.h"
@@ skipping 49 matching lines @@
                             const GURL& origin) override;
 
   // PasswordStore::Observer:
   void OnLoginsChanged(
       const password_manager::PasswordStoreChangeList& changes) override;
 
   // Set the state of the Omnibox icon, and possibly show the associated bubble
   // without user interaction.
   virtual void UpdateIconAndBubbleState(ManagePasswordsIconView* icon);
 
-  bool IsAutomaticallyOpeningBubble() const { return should_pop_up_bubble_; }
+  bool IsAutomaticallyOpeningBubble() const {
+    return bubble_status_ == SHOULD_POP_UP;
+  }
 
   // PasswordsModelDelegate:
   const GURL& GetOrigin() const override;
   password_manager::ui::State GetState() const override;
   const autofill::PasswordForm& GetPendingPassword() const override;
   bool IsPasswordOverridden() const override;
   const std::vector<const autofill::PasswordForm*>& GetCurrentForms()
       const override;
   const std::vector<const autofill::PasswordForm*>& GetFederatedForms()
       const override;
@@ skipping 45 matching lines @@
 
   // content::WebContentsObserver:
   void DidNavigateMainFrame(
       const content::LoadCommittedDetails& details,
       const content::FrameNavigateParams& params) override;
   void WasHidden() override;
 
  private:
   friend class content::WebContentsUserData<ManagePasswordsUIController>;
 
+  enum BubbleStatus {
+    NOT_SHOWN,
+    // The bubble is to be popped up in the next call to
+    // UpdateBubbleAndIconVisibility().
+    SHOULD_POP_UP,
+    SHOWN,
+  };
+
   // Shows the password bubble without user interaction.
   void ShowBubbleWithoutUserInteraction();
 
   // Closes the account chooser gracefully so the callback is called. Then sets
   // the state to MANAGE_STATE.
   void DestroyAccountChooser();
 
   // content::WebContentsObserver:
   void WebContentsDestroyed() override;
 
   // The wrapper around current state and data.
   ManagePasswordsState passwords_data_;
 
   // The controller for the blocking dialogs.
   scoped_ptr<PasswordDialogControllerImpl> dialog_controller_;
 
-  // Contains true if the bubble is to be popped up in the next call to
-  // UpdateBubbleAndIconVisibility().
-  bool should_pop_up_bubble_;
+  BubbleStatus bubble_status_;
 
   DISALLOW_COPY_AND_ASSIGN(ManagePasswordsUIController);
 };
 
 #endif  // CHROME_BROWSER_UI_PASSWORDS_MANAGE_PASSWORDS_UI_CONTROLLER_H_