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

Unified Diff: components/autofill/content/browser/content_autofill_driver_unittest.cc

Issue 184103016: Autofill: Refactoring to support fetching password after a username is selected (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address gcasto's comments. Created 6 years, 9 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: components/autofill/content/browser/content_autofill_driver_unittest.cc
diff --git a/components/autofill/content/browser/content_autofill_driver_unittest.cc b/components/autofill/content/browser/content_autofill_driver_unittest.cc
index fc0b140ee472413011cb1ea6f6b44a617178b31d..665df3b8fbe13e10ac0c0dad01e2c1e9f51bfc9d 100644
--- a/components/autofill/content/browser/content_autofill_driver_unittest.cc
+++ b/components/autofill/content/browser/content_autofill_driver_unittest.cc
@@ -166,11 +166,6 @@ class ContentAutofillDriverTest : public ChromeRenderViewHostTestHarness {
&autofill_param))
return false;
break;
- case AutofillMsg_AcceptPasswordAutofillSuggestion::ID:
- if (!AutofillMsg_AcceptPasswordAutofillSuggestion::Read(
- message, &autofill_param))
- return false;
- break;
default:
NOTREACHED();
}
@@ -291,15 +286,6 @@ TEST_F(ContentAutofillDriverTest, AcceptDataListSuggestion) {
EXPECT_EQ(input_value, output_value);
}
-TEST_F(ContentAutofillDriverTest, AcceptPasswordAutofillSuggestion) {
- base::string16 input_value(base::ASCIIToUTF16("barbaz"));
- base::string16 output_value;
- driver_->RendererShouldAcceptPasswordAutofillSuggestion(input_value);
- EXPECT_TRUE(GetString16FromMessageWithID(
- AutofillMsg_AcceptPasswordAutofillSuggestion::ID, &output_value));
- EXPECT_EQ(input_value, output_value);
-}
-
TEST_F(ContentAutofillDriverTest, ClearFilledFormSentToRenderer) {
driver_->RendererShouldClearFilledForm();
EXPECT_TRUE(HasMessageMatchingID(AutofillMsg_ClearForm::ID));

Powered by Google App Engine
This is Rietveld 408576698