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

Unified Diff: components/autofill/core/browser/payments/payments_client.h

Issue 2829853008: Stores server card as a full server card when upload to server succeeds. (Closed)
Patch Set: Renames AddServerCreditCard to AddFullServerCreditCard. Adds DCHECK to verify that we are in a tran… 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/autofill/core/browser/payments/payments_client.h
diff --git a/components/autofill/core/browser/payments/payments_client.h b/components/autofill/core/browser/payments/payments_client.h
index 40fb7d1590c748cbeb52094e2ad59d6aa172922d..9e2549dcfd0665335f90415afbe956193577c87b 100644
--- a/components/autofill/core/browser/payments/payments_client.h
+++ b/components/autofill/core/browser/payments/payments_client.h
@@ -47,7 +47,10 @@ class PaymentsClientDelegate {
std::unique_ptr<base::DictionaryValue> legal_message) = 0;
// Returns the result of an upload request.
- virtual void OnDidUploadCard(AutofillClient::PaymentsRpcResult result) = 0;
+ // If |result| == |AutofillClient::SUCCESS|, |server_id| may, optionally,
+ // contain the opaque identifier for the card on the server.
+ virtual void OnDidUploadCard(AutofillClient::PaymentsRpcResult result,
+ const std::string& server_id) = 0;
};
// PaymentsClient issues Payments RPCs and manages responses and failure

Powered by Google App Engine
This is Rietveld 408576698