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

Unified Diff: chrome/renderer/autofill/password_generation_manager_browsertest.cc

Issue 10168017: Only enable password generation if password manager and autofill are both (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove autofill requirement Created 8 years, 8 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/renderer/autofill/password_generation_manager_browsertest.cc
diff --git a/chrome/renderer/autofill/password_generation_manager_browsertest.cc b/chrome/renderer/autofill/password_generation_manager_browsertest.cc
index 1119bd489d381c6df8f05db1ec6df4c71c7fc843..69074b0dba50de96fd6665542a32854be707dfdc 100644
--- a/chrome/renderer/autofill/password_generation_manager_browsertest.cc
+++ b/chrome/renderer/autofill/password_generation_manager_browsertest.cc
@@ -113,7 +113,7 @@ TEST_F(PasswordGenerationManagerTest, DetectionTest) {
EXPECT_EQ(0u, generation_manager_->messages().size());
// Pretend like sync was enabled.
- AutofillMsg_PasswordSyncEnabled msg(0, true);
+ AutofillMsg_PasswordGenerationEnabled msg(0, true);
generation_manager_->OnMessageReceived(msg);
// Now we will send a message once the first password feld is focused.
@@ -135,7 +135,7 @@ TEST_F(PasswordGenerationManagerTest, DetectionTest) {
TEST_F(PasswordGenerationManagerTest, FillTest) {
// Make sure that we are enabled before loading HTML.
- AutofillMsg_PasswordSyncEnabled enabled_msg(0, true);
+ AutofillMsg_PasswordGenerationEnabled enabled_msg(0, true);
generation_manager_->OnMessageReceived(enabled_msg);
LoadHTML(kAccountCreationFormHTML);

Powered by Google App Engine
This is Rietveld 408576698