[Web Payments] Implement the CVC Unmask dialog.

components/payments/content/payment_request_state.cc

Index: components/payments/content/payment_request_state.cc
diff --git a/components/payments/content/payment_request_state.cc b/components/payments/content/payment_request_state.cc
index 785517d5db21a1e8cb380394b80a636fdbebfe5f..3418eb4b63822faaf08c52a71e363f99326679e6 100644
--- a/components/payments/content/payment_request_state.cc
+++ b/components/payments/content/payment_request_state.cc
@@ -11,6 +11,7 @@
 #include "components/autofill/core/browser/autofill_profile.h"
 #include "components/autofill/core/browser/credit_card.h"
 #include "components/autofill/core/browser/personal_data_manager.h"
+#include "components/payments/content/payment_request_delegate.h"

[please use gerrit instead, 2017/03/30 13:14:09]

I don't think this is necessary, because you don't have the
PaymentRequestDelegate data type anywhere in the file.
GetPaymentRequestDelegate() does not count, I believe.

[anthonyvd, 2017/03/30 15:43:41]

Yeah it's required for the compiler to know that PaymentRequestDelegate inherits
from FullCardRequestDelegate (which is the type that the
AutofillPaymentInstrument ctor accepts).

 #include "components/payments/content/payment_request_spec.h"
 #include "components/payments/content/payment_response_helper.h"
 #include "components/payments/core/autofill_payment_instrument.h"
@@ -181,7 +182,8 @@ void PaymentRequestState::PopulateProfileCache() {
     // indirectly owned by this object.
     std::unique_ptr<PaymentInstrument> instrument =
         base::MakeUnique<AutofillPaymentInstrument>(
-            basic_card_network, *card, shipping_profiles_, app_locale_);
+            basic_card_network, *card, shipping_profiles_, app_locale_,
+            delegate_->GetPaymentRequestDelegate());
     available_instruments_.push_back(std::move(instrument));
   }
 }