[Payments] Upload card UI now has a CVC prompt

components/autofill/core/browser/ui/save_card_bubble_controller.h

Index: components/autofill/core/browser/ui/save_card_bubble_controller.h
diff --git a/chrome/browser/ui/autofill/save_card_bubble_controller.h b/components/autofill/core/browser/ui/save_card_bubble_controller.h
similarity index 61%
rename from chrome/browser/ui/autofill/save_card_bubble_controller.h
rename to components/autofill/core/browser/ui/save_card_bubble_controller.h
index 6f74eb3f09414ffee90bbd3e3c8f7bf04c2c129f..e2a56c39a1008ce21b833642e7fbd6c952913991 100644
--- a/chrome/browser/ui/autofill/save_card_bubble_controller.h
+++ b/components/autofill/core/browser/ui/save_card_bubble_controller.h
@@ -2,8 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef CHROME_BROWSER_UI_AUTOFILL_SAVE_CARD_BUBBLE_CONTROLLER_H_
-#define CHROME_BROWSER_UI_AUTOFILL_SAVE_CARD_BUBBLE_CONTROLLER_H_
+#ifndef COMPONENTS_AUTOFILL_CORE_BROWSER_UI_SAVE_CARD_BUBBLE_CONTROLLER_H_
+#define COMPONENTS_AUTOFILL_CORE_BROWSER_UI_SAVE_CARD_BUBBLE_CONTROLLER_H_
 
 #include <memory>
 #include <vector>
@@ -31,6 +31,21 @@ class SaveCardBubbleController {
   // Returns the card that will be uploaded if the user accepts.
   virtual const CreditCard GetCard() const = 0;
 
+  // Returns the CVC image icon.
+  virtual int GetCvcImageRid() const = 0;
+
+  // Returns whether the dialog should include a field requesting the card's CVC
+  // from the user.
+  virtual bool ShouldRequestCvcFromUser() const = 0;
+
+  // Takes in the CVC provided by the user in the save card bubble (if it was
+  // not detected on the page), to be populated in the final request.

[csashi, 2017/04/02 22:36:15]

You can remove the "(if it was not detected on the page)" from this comment.
This function is not responsible for the logic on whether CVC was detected on
the page or not.

[Jared Saul, 2017/04/03 18:52:59]

Done.

+  virtual void SetCvcEnteredByUser(const base::string16& cvc) = 0;
+
+  // Returns the CVC provided by the user in the save card bubble (if it was not
+  // detected on the page).

[csashi, 2017/04/02 22:36:15]

Same as above.

[Jared Saul, 2017/04/03 18:52:59]

Done.

+  virtual base::string16 GetCvcEnteredByUser() const = 0;
+
   // Interaction.
   virtual void OnSaveButton() = 0;
   virtual void OnCancelButton() = 0;
@@ -43,6 +58,9 @@ class SaveCardBubbleController {
   // Returns empty vector if no legal message should be shown.
   virtual const LegalMessageLines& GetLegalMessageLines() const = 0;
 
+  // Utilities.
+  virtual bool InputCvcIsValid(const base::string16& input_text) const = 0;
+
  protected:
   SaveCardBubbleController() {}
   virtual ~SaveCardBubbleController() {}
@@ -52,4 +70,4 @@ class SaveCardBubbleController {
 
 }  // namespace autofill
 
-#endif  // CHROME_BROWSER_UI_AUTOFILL_SAVE_CARD_BUBBLE_CONTROLLER_H_
+#endif  // COMPONENTS_AUTOFILL_CORE_BROWSER_UI_SAVE_CARD_BUBBLE_CONTROLLER_H_