[Web Payments] Implement the CVC Unmask dialog.

components/payments/content/payment_request_state.cc

 // Copyright 2016 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/content/payment_request_state.h"
 
 #include <set>
 
 #include "base/strings/utf_string_conversions.h"
 #include "components/autofill/core/browser/autofill_data_util.h"
 #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"
 
 namespace payments {
 
 PaymentRequestState::PaymentRequestState(
     PaymentRequestSpec* spec,
     Delegate* delegate,
     const std::string& app_locale,
@@ skipping 150 matching lines @@
     if (!supported_card_networks.count(basic_card_network))
       continue;
 
     // TODO(crbug.com/701952): Should use the method name preferred by the
     // merchant (either "basic-card" or the basic card network e.g. "visa").
 
     // Copy the credit cards as part of AutofillPaymentInstrument so they are
     // 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));
   }
 }
 
 void PaymentRequestState::SetDefaultProfileSelections() {
   if (!shipping_profiles().empty())
     selected_shipping_profile_ = shipping_profiles()[0];
 
   if (!contact_profiles().empty())
     selected_contact_profile_ = contact_profiles()[0];
@@ skipping 77 matching lines @@
       spec_->details().shipping_options.rend(),
       [](const payments::mojom::PaymentShippingOptionPtr& element) {
         return element->selected;
       });
   if (selected_shipping_option_it != spec_->details().shipping_options.rend()) {
     selected_shipping_option_ = selected_shipping_option_it->get();
   }
 }
 
 }  // namespace payments