[Payments] Upload card UI now has a CVC prompt

chrome/browser/ui/autofill/chrome_autofill_client.h

Index: chrome/browser/ui/autofill/chrome_autofill_client.h
diff --git a/chrome/browser/ui/autofill/chrome_autofill_client.h b/chrome/browser/ui/autofill/chrome_autofill_client.h
index 083522eec61494e1ee82d8841033a0826412f359..7cff42cf39027ece2ec7f0afe0e9c6e29368e16d 100644
--- a/chrome/browser/ui/autofill/chrome_autofill_client.h
+++ b/chrome/browser/ui/autofill/chrome_autofill_client.h
@@ -45,6 +45,7 @@ class ChromeAutofillClient
   IdentityProvider* GetIdentityProvider() override;
   rappor::RapporServiceImpl* GetRapporServiceImpl() override;
   ukm::UkmService* GetUkmService() override;
+  SaveCardBubbleController* GetSaveCardBubbleController() override;
   void ShowAutofillSettings() override;
   void ShowUnmaskPrompt(const CreditCard& card,
                         UnmaskCardReason reason,
@@ -55,6 +56,7 @@ class ChromeAutofillClient
   void ConfirmSaveCreditCardToCloud(
       const CreditCard& card,
       std::unique_ptr<base::DictionaryValue> legal_message,
+      const bool upload_cvc_should_be_requested,

[csashi, 2017/03/31 21:48:15]

I think you can ditch the const.

[csashi, 2017/03/31 21:48:15]

s/upload_cvc_should_be_request/request_cvc_if_missing (also reads the same as
your flag).

[Jared Saul, 2017/04/01 04:18:01]

Done.

[Jared Saul, 2017/04/01 04:18:02]

I agree that it makes it match the flag better, but code-wise I'm not sure this
should change.  This value is literally "yes you should request CVC" or "no you
should not".  The "if_missing" check happens before this point.

       const base::Closure& callback) override;
   void ConfirmCreditCardFillAssist(const CreditCard& card,
                                    const base::Closure& callback) override;