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

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

Issue 2915763003: [Password Manager] Show omnibox icon and anchored prompt once user start typing password (Closed)
Patch Set: Introduced |has_generated_password| argument Created 3 years, 4 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
Index: chrome/browser/password_manager/credential_manager_browsertest.cc
diff --git a/chrome/browser/password_manager/credential_manager_browsertest.cc b/chrome/browser/password_manager/credential_manager_browsertest.cc
index 3d7364597185a01c83ce41062db818d1f85669ec..14711ab1bc3a423ed99ef10c3997ac0502f5cb6e 100644
--- a/chrome/browser/password_manager/credential_manager_browsertest.cc
+++ b/chrome/browser/password_manager/credential_manager_browsertest.cc
@@ -158,7 +158,7 @@ class CredentialManagerBrowserTest : public PasswordManagerBrowserTestBase {
BubbleObserver prompt_observer(WebContents());
prompt_observer.WaitForSavePrompt();
- ASSERT_TRUE(prompt_observer.IsShowingSavePrompt());
+ ASSERT_TRUE(prompt_observer.WasSavePromptShown());
prompt_observer.AcceptSavePrompt();
WaitForPasswordStore();
@@ -329,8 +329,8 @@ IN_PROC_BROWSER_TEST_F(CredentialManagerBrowserTest,
// after the store replies.
WaitForPasswordStore();
BubbleObserver prompt_observer(WebContents());
- EXPECT_FALSE(prompt_observer.IsShowingSavePrompt());
- EXPECT_FALSE(prompt_observer.IsShowingUpdatePrompt());
+ EXPECT_FALSE(prompt_observer.WasSavePromptShown());
+ EXPECT_FALSE(prompt_observer.WasUpdatePromptShown());
// There should be an entry for both psl.example.com and www.example.com.
password_manager::TestPasswordStore::PasswordMap passwords =
@@ -421,7 +421,7 @@ IN_PROC_BROWSER_TEST_F(CredentialManagerBrowserTest,
BubbleObserver prompt_observer(WebContents());
// The autofill password manager shouldn't react to the successful login
// because it was suppressed when the site got the credential back.
- EXPECT_FALSE(prompt_observer.IsShowingSavePrompt());
+ EXPECT_FALSE(prompt_observer.WasSavePromptShown());
}
// Regression test for https://crbug.com/736357.
@@ -480,7 +480,7 @@ IN_PROC_BROWSER_TEST_F(CredentialManagerBrowserTest,
BubbleObserver prompt_observer(WebContents());
prompt_observer.WaitForSavePrompt();
- ASSERT_TRUE(prompt_observer.IsShowingSavePrompt());
+ ASSERT_TRUE(prompt_observer.WasSavePromptShown());
prompt_observer.AcceptSavePrompt();
WaitForPasswordStore();
@@ -546,7 +546,7 @@ IN_PROC_BROWSER_TEST_F(CredentialManagerBrowserTest, SaveViaAPIAndAutofill) {
WaitForPasswordStore();
BubbleObserver prompt_observer(WebContents());
- ASSERT_TRUE(prompt_observer.IsShowingSavePrompt());
+ ASSERT_TRUE(prompt_observer.WasSavePromptShown());
prompt_observer.AcceptSavePrompt();
WaitForPasswordStore();
@@ -604,8 +604,8 @@ IN_PROC_BROWSER_TEST_F(CredentialManagerBrowserTest, UpdateViaAPIAndAutofill) {
// after the store replies.
WaitForPasswordStore();
BubbleObserver prompt_observer(WebContents());
- EXPECT_FALSE(prompt_observer.IsShowingSavePrompt());
- EXPECT_FALSE(prompt_observer.IsShowingUpdatePrompt());
+ EXPECT_FALSE(prompt_observer.WasSavePromptShown());
+ EXPECT_FALSE(prompt_observer.WasUpdatePromptShown());
signin_form.skip_zero_click = false;
signin_form.times_used = 1;
signin_form.password_value = base::ASCIIToUTF16("API");

Powered by Google App Engine
This is Rietveld 408576698