[Web Payments] Implement the CVC Unmask dialog.

components/payments/core/autofill_payment_instrument.h

 // 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.
 
 #ifndef COMPONENTS_PAYMENTS_CORE_AUTOFILL_PAYMENT_INSTRUMENT_H_
 #define COMPONENTS_PAYMENTS_CORE_AUTOFILL_PAYMENT_INSTRUMENT_H_
 
 #include <set>
 #include <string>
 #include <vector>
 
 #include "base/macros.h"
 #include "components/autofill/core/browser/credit_card.h"
+#include "components/autofill/core/browser/payments/full_card_request.h"
 #include "components/payments/core/payment_instrument.h"
 
 namespace autofill {
 class AutofillProfile;
 }
 
 namespace payments {
 
 // Represents an Autofill/Payments credit card form of payment in Payment
 // Request.
-class AutofillPaymentInstrument : public PaymentInstrument {
+class AutofillPaymentInstrument
+    : public PaymentInstrument,
+      public autofill::payments::FullCardRequest::ResultDelegate {
  public:
+  class FullCardRequestDelegate {

[Mathieu, 2017/04/05 00:02:23]

I think I still see a world where this is not needed? What if
AutofillPaymentInstrument is constructed with a PaymentRequestDelegate* (which
would be moved to /core)? PaymentRequestState would pass it along and would be
itself constructed with it (similar to how you have it now).

[anthonyvd, 2017/04/05 02:12:37]

Yep, you're right. Done.

+   public:
+    virtual ~FullCardRequestDelegate() {}
+    virtual void DoFullCardRequest(
+        const autofill::CreditCard& credit_card,
+        base::WeakPtr<autofill::payments::FullCardRequest::ResultDelegate>
+            result_delegate) = 0;
+  };
+
   // |billing_profiles| is owned by the caller and should outlive this object.
+  // |full_card_request_delegate| must outlive this object.
   AutofillPaymentInstrument(
       const std::string& method_name,
       const autofill::CreditCard& card,
       const std::vector<autofill::AutofillProfile*>& billing_profiles,
-      const std::string& app_locale);
+      const std::string& app_locale,
+      FullCardRequestDelegate* full_card_request_delegate);
   ~AutofillPaymentInstrument() override;
 
   // PaymentInstrument:
   void InvokePaymentApp(PaymentInstrument::Delegate* delegate) override;
   bool IsValid() override;
 
+  // autofill::payments::FullCardRequest::ResultDelegate:
+  void OnFullCardRequestSucceeded(const autofill::CreditCard& card,
+                                  const base::string16& cvc) override;
+  void OnFullCardRequestFailed() override;
+
  private:
   // A copy of the card is owned by this object.
-  const autofill::CreditCard credit_card_;
+  autofill::CreditCard credit_card_;
   // Not owned by this object, should outlive this.
   const std::vector<autofill::AutofillProfile*>& billing_profiles_;
 
   const std::string app_locale_;
 
+  PaymentInstrument::Delegate* delegate_;
+  FullCardRequestDelegate* full_card_request_delegate_;
+
+  base::WeakPtrFactory<AutofillPaymentInstrument> weak_ptr_factory_;
+
   DISALLOW_COPY_AND_ASSIGN(AutofillPaymentInstrument);
 };
 
 }  // namespace payments
 
 #endif  // COMPONENTS_PAYMENTS_CORE_AUTOFILL_PAYMENT_INSTRUMENT_H_