Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(468)

Unified Diff: components/payments/content/payment_request_dialog.h

Issue 2779283002: [Web Payments] Implement the CVC Unmask dialog. (Closed)
Patch Set: Rebase + add browser test Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/payments/content/payment_request_dialog.h
diff --git a/components/payments/content/payment_request_dialog.h b/components/payments/content/payment_request_dialog.h
index 8859167c9f06ef360a6513bbc571c830c2f75da2..259bdbaac940d338789783be4e55774e635ead1c 100644
--- a/components/payments/content/payment_request_dialog.h
+++ b/components/payments/content/payment_request_dialog.h
@@ -5,6 +5,9 @@
#ifndef COMPONENTS_PAYMENTS_CONTENT_PAYMENT_REQUEST_DIALOG_H_
#define COMPONENTS_PAYMENTS_CONTENT_PAYMENT_REQUEST_DIALOG_H_
+#include "base/memory/weak_ptr.h"
+#include "components/autofill/core/browser/payments/full_card_request.h"
Mathieu 2017/04/04 19:05:47 do we need this (vs fwd declare) because of the we
anthonyvd 2017/04/05 02:12:37 Yeah we need this because the fwd declare would be
+
namespace payments {
// Used to interact with a cross-platform Payment Request dialog.
@@ -17,6 +20,11 @@ class PaymentRequestDialog {
virtual void CloseDialog() = 0;
virtual void ShowErrorMessage() = 0;
+
+ virtual void ShowCvcUnmaskPrompt(
Mathieu 2017/04/04 19:05:47 include comment about non trivial function
anthonyvd 2017/04/05 02:12:37 Done.
+ const autofill::CreditCard& credit_card,
+ base::WeakPtr<autofill::payments::FullCardRequest::ResultDelegate>
+ result_delegate) = 0;
};
} // namespace payments

Powered by Google App Engine
This is Rietveld 408576698