[Payments] Record use stats after payment request completion.

components/payments/core/autofill_payment_instrument.cc

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/payments/core/autofill_payment_instrument.h"
 
 #include <memory>
 
 #include "base/json/json_writer.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 85 matching lines @@
 bool AutofillPaymentInstrument::IsValidForCanMakePayment() {
   autofill::CreditCardCompletionStatus status =
       autofill::GetCompletionStatusForCard(credit_card_, app_locale_,
                                            billing_profiles_);
   // Card has to have a cardholder name and number for the purposes of
   // CanMakePayment. An expired card is still valid at this stage.
   return !(status & autofill::CREDIT_CARD_NO_CARDHOLDER ||
            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) {
   DCHECK(delegate_);
   credit_card_ = card;
   cvc_ = cvc;
   is_waiting_for_card_unmask_ = false;
 
   if (!is_waiting_for_billing_address_normalization_)
     GenerateBasicCardResponse();
@@ skipping 35 matching lines @@
           .ToDictionaryValue();
   std::string stringified_details;
   base::JSONWriter::Write(*response_value, &stringified_details);
   delegate_->OnInstrumentDetailsReady(method_name(), stringified_details);
 
   delegate_ = nullptr;
   cvc_ = base::UTF8ToUTF16("");
 }
 
 }  // namespace payments