| Index: components/autofill/core/browser/autofill_manager_unittest.cc
|
| diff --git a/components/autofill/core/browser/autofill_manager_unittest.cc b/components/autofill/core/browser/autofill_manager_unittest.cc
|
| index 177bb9ba20e0e27463523b9ddd449c14274003b7..0e9e81339f1c08207cc9ca0a2e875bb3c88b7ff2 100644
|
| --- a/components/autofill/core/browser/autofill_manager_unittest.cc
|
| +++ b/components/autofill/core/browser/autofill_manager_unittest.cc
|
| @@ -68,8 +68,8 @@ class MockAutofillClient : public TestAutofillClient {
|
|
|
| ~MockAutofillClient() override {}
|
|
|
| - MOCK_METHOD1(ConfirmSaveCreditCardLocally,
|
| - void(const base::Closure& callback));
|
| + MOCK_METHOD2(ConfirmSaveCreditCardLocally,
|
| + void(const CreditCard& card, const base::Closure& callback));
|
|
|
| private:
|
| DISALLOW_COPY_AND_ASSIGN(MockAutofillClient);
|
| @@ -878,7 +878,7 @@ class AutofillManagerTest : public testing::Test {
|
| form.fields[1].value = ASCIIToUTF16("4111111111111111");
|
| form.fields[2].value = ASCIIToUTF16("11");
|
| form.fields[3].value = ASCIIToUTF16("2017");
|
| - EXPECT_CALL(autofill_client_, ConfirmSaveCreditCardLocally(_)).Times(1);
|
| + EXPECT_CALL(autofill_client_, ConfirmSaveCreditCardLocally(_, _));
|
| FormSubmitted(form);
|
| }
|
|
|
| @@ -895,7 +895,7 @@ class AutofillManagerTest : public testing::Test {
|
| "2017");
|
| card->SetTypeForMaskedCard(kVisaCard);
|
|
|
| - EXPECT_CALL(autofill_client_, ConfirmSaveCreditCardLocally(_)).Times(0);
|
| + EXPECT_CALL(autofill_client_, ConfirmSaveCreditCardLocally(_, _)).Times(0);
|
| EXPECT_CALL(*autofill_driver_, SendFormDataToRenderer(_, _, _))
|
| .Times(AtLeast(1));
|
| autofill_manager_->FillOrPreviewCreditCardForm(
|
| @@ -2596,7 +2596,7 @@ TEST_F(AutofillManagerTest, FormSubmittedAutocompleteEnabled) {
|
| MockAutocompleteHistoryManager* m = static_cast<
|
| MockAutocompleteHistoryManager*>(
|
| autofill_manager_->autocomplete_history_manager_.get());
|
| - EXPECT_CALL(*m, OnWillSubmitForm(_)).Times(1);
|
| + EXPECT_CALL(*m, OnWillSubmitForm(_));
|
| FormSubmitted(form);
|
| }
|
|
|
| @@ -2623,7 +2623,7 @@ TEST_F(AutofillManagerTest, AutocompleteSuggestions_SomeWhenAutofillDisabled) {
|
| MockAutocompleteHistoryManager*>(
|
| autofill_manager_->autocomplete_history_manager_.get());
|
| EXPECT_CALL(*m,
|
| - OnGetAutocompleteSuggestions(_, _, _, _)).Times(1);
|
| + OnGetAutocompleteSuggestions(_, _, _, _));
|
|
|
| GetAutofillSuggestions(form, field);
|
| }
|
| @@ -2710,7 +2710,7 @@ TEST_F(AutofillManagerTest, AutocompleteSuggestions_SomeWhenAutofillEmpty) {
|
| MockAutocompleteHistoryManager*>(
|
| autofill_manager_->autocomplete_history_manager_.get());
|
| EXPECT_CALL(*m,
|
| - OnGetAutocompleteSuggestions(_, _, _, _)).Times(1);
|
| + OnGetAutocompleteSuggestions(_, _, _, _));
|
|
|
| GetAutofillSuggestions(form, field);
|
| }
|
| @@ -3047,7 +3047,7 @@ TEST_F(AutofillManagerTest, CreditCardSavedWhenAutocompleteOff) {
|
| form.fields[1].value = ASCIIToUTF16("4111111111111111");
|
| form.fields[2].value = ASCIIToUTF16("11");
|
| form.fields[3].value = ASCIIToUTF16("2017");
|
| - EXPECT_CALL(autofill_client_, ConfirmSaveCreditCardLocally(_)).Times(1);
|
| + EXPECT_CALL(autofill_client_, ConfirmSaveCreditCardLocally(_, _));
|
| FormSubmitted(form);
|
| }
|
|
|
| @@ -3066,7 +3066,7 @@ TEST_F(AutofillManagerTest, InvalidCreditCardNumberIsNotSaved) {
|
| form.fields[1].value = ASCIIToUTF16(card);
|
| form.fields[2].value = ASCIIToUTF16("11");
|
| form.fields[3].value = ASCIIToUTF16("2017");
|
| - EXPECT_CALL(autofill_client_, ConfirmSaveCreditCardLocally(_)).Times(0);
|
| + EXPECT_CALL(autofill_client_, ConfirmSaveCreditCardLocally(_, _)).Times(0);
|
| FormSubmitted(form);
|
| }
|
|
|
| @@ -3665,7 +3665,7 @@ TEST_F(AutofillManagerTest, DontUploadCreditCardIfFeatureNotEnabled) {
|
| base::HistogramTester histogram_tester;
|
|
|
| // The save prompt should be shown instead of doing an upload.
|
| - EXPECT_CALL(autofill_client_, ConfirmSaveCreditCardLocally(_)).Times(1);
|
| + EXPECT_CALL(autofill_client_, ConfirmSaveCreditCardLocally(_, _));
|
| FormSubmitted(credit_card_form);
|
| EXPECT_FALSE(autofill_manager_->credit_card_was_uploaded());
|
|
|
| @@ -3699,7 +3699,7 @@ TEST_F(AutofillManagerTest, DontUploadCreditCardIfCvcUnavailable) {
|
| base::HistogramTester histogram_tester;
|
|
|
| // Neither a local save nor an upload should happen in this case.
|
| - EXPECT_CALL(autofill_client_, ConfirmSaveCreditCardLocally(_)).Times(0);
|
| + EXPECT_CALL(autofill_client_, ConfirmSaveCreditCardLocally(_, _)).Times(0);
|
| FormSubmitted(credit_card_form);
|
| EXPECT_FALSE(autofill_manager_->credit_card_was_uploaded());
|
|
|
| @@ -3736,7 +3736,7 @@ TEST_F(AutofillManagerTest, DontUploadCreditCardIfNoMatchingProfileAvailable) {
|
| base::HistogramTester histogram_tester;
|
|
|
| // Neither a local save nor an upload should happen in this case.
|
| - EXPECT_CALL(autofill_client_, ConfirmSaveCreditCardLocally(_)).Times(0);
|
| + EXPECT_CALL(autofill_client_, ConfirmSaveCreditCardLocally(_, _)).Times(0);
|
| FormSubmitted(credit_card_form);
|
| EXPECT_FALSE(autofill_manager_->credit_card_was_uploaded());
|
|
|
| @@ -3776,7 +3776,7 @@ TEST_F(AutofillManagerTest, DontUploadCreditCardIfUploadDetailsFails) {
|
| base::HistogramTester histogram_tester;
|
|
|
| // The save prompt should be shown instead of doing an upload.
|
| - EXPECT_CALL(autofill_client_, ConfirmSaveCreditCardLocally(_)).Times(1);
|
| + EXPECT_CALL(autofill_client_, ConfirmSaveCreditCardLocally(_, _));
|
| FormSubmitted(credit_card_form);
|
| EXPECT_FALSE(autofill_manager_->credit_card_was_uploaded());
|
|
|
|
|