Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2149)

Unified Diff: chrome/browser/password_manager/password_manager_test_base.cc

Issue 2679983003: Remove unnecessary line in PasswordManagerBrowserTestBase. (Closed)
Patch Set: Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/password_manager/password_manager_test_base.cc
diff --git a/chrome/browser/password_manager/password_manager_test_base.cc b/chrome/browser/password_manager/password_manager_test_base.cc
index e8d0327ccaf2ca249f59990508be0f253846439a..d86aa8538dbf5d3d8c76e466a0517a46f20529bb 100644
--- a/chrome/browser/password_manager/password_manager_test_base.cc
+++ b/chrome/browser/password_manager/password_manager_test_base.cc
@@ -95,10 +95,6 @@ bool BubbleObserver::IsShowingUpdatePrompt() const {
void BubbleObserver::Dismiss() const {
passwords_ui_controller_->OnBubbleHidden();
- // Navigate away to reset the state to inactive.
- static_cast<content::WebContentsObserver*>(passwords_ui_controller_)
- ->DidNavigateMainFrame(content::LoadCommittedDetails(),
- content::FrameNavigateParams());
ASSERT_EQ(password_manager::ui::INACTIVE_STATE,
passwords_ui_controller_->GetState());
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698