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

Unified Diff: components/autofill/core/browser/autofill_metrics_unittest.cc

Issue 23882013: [rAc] Persist selection of newly added cards and addresses. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix tests, add a test Created 7 years, 3 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/core/browser/autofill_metrics_unittest.cc
diff --git a/components/autofill/core/browser/autofill_metrics_unittest.cc b/components/autofill/core/browser/autofill_metrics_unittest.cc
index 22b13185689794376f5910a9d4f95f1edd735df9..c12b2ff5f544f4a2be25cb50fb28f861ca30b9aa 100644
--- a/components/autofill/core/browser/autofill_metrics_unittest.cc
+++ b/components/autofill/core/browser/autofill_metrics_unittest.cc
@@ -125,7 +125,7 @@ class TestPersonalDataManager : public PersonalDataManager {
}
MOCK_METHOD1(SaveImportedCreditCard,
- void(const CreditCard& imported_credit_card));
+ std::string(const CreditCard& imported_credit_card));
private:
void CreateTestAutofillProfiles(ScopedVector<AutofillProfile>* profiles) {
@@ -319,8 +319,9 @@ scoped_ptr<ConfirmInfoBarDelegate> AutofillMetricsTest::CreateDelegate(
CreditCard credit_card;
return AutofillCCInfoBarDelegate::Create(
metric_logger,
- base::Bind(&TestPersonalDataManager::SaveImportedCreditCard,
- base::Unretained(personal_data_.get()), credit_card));
+ base::Bind(
+ base::IgnoreResult(&TestPersonalDataManager::SaveImportedCreditCard),
+ base::Unretained(personal_data_.get()), credit_card));
}
// Test that we log quality metrics appropriately.
« no previous file with comments | « components/autofill/core/browser/autofill_merge_unittest.cc ('k') | components/autofill/core/browser/personal_data_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698