[Payments] Record use stats after payment request completion.

components/payments/core/autofill_payment_instrument.cc

Index: components/payments/core/autofill_payment_instrument.cc
diff --git a/components/payments/core/autofill_payment_instrument.cc b/components/payments/core/autofill_payment_instrument.cc
index 08502b4e832477bd4c4697e276240ca7fa536ee9..053ba4298f3b1438e033ca6c5d5230204d2043f2 100644
--- a/components/payments/core/autofill_payment_instrument.cc
+++ b/components/payments/core/autofill_payment_instrument.cc
@@ -103,6 +103,18 @@ bool AutofillPaymentInstrument::IsValidForCanMakePayment() {
            status & autofill::CREDIT_CARD_NO_NUMBER);
 }
 
+void AutofillPaymentInstrument::RecordUse() {
+  // Record the use of the credit card.
+  payment_request_delegate_->GetPersonalDataManager()->RecordUseOf(
+      credit_card_);
+
+  // Record the use of the billing address.

[Mathieu, 2017/05/11 01:25:27]

Yeah I'm not sure about that one, because we use address stats to rank addresses
in the shipping selector and in Autofill. Using a card that happens to have a
specific billing address doesn't mean you like that address as a shipping
address... 

Your earlier point about not deleting billing addresses after some time has
passed, but at that time we can simply check that the address is not used as a
billing address before deleting it. 

Can we remove this statement?

[sebsg, 2017/05/11 17:49:18]

Done.

+  DCHECK(credit_card_.billing_address_id() != "");
+  payment_request_delegate_->GetPersonalDataManager()->RecordUseOf(
+      *(payment_request_delegate_->GetPersonalDataManager()->GetProfileByGUID(
+          credit_card_.billing_address_id())));
+}
+
 void AutofillPaymentInstrument::OnFullCardRequestSucceeded(
     const autofill::CreditCard& card,
     const base::string16& cvc) {