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

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

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.cc
diff --git a/components/autofill/core/browser/payments/payments_client.cc b/components/autofill/core/browser/payments/payments_client.cc
index 44e85c2e8d9f5e4ce4a160c3ae32666d3836af3a..75a735c40475483b61232b42773ab6de936f2b45 100644
--- a/components/autofill/core/browser/payments/payments_client.cc
+++ b/components/autofill/core/browser/payments/payments_client.cc
@@ -341,17 +341,19 @@ class UploadCardRequest : public PaymentsRequest {
}
void ParseResponse(std::unique_ptr<base::DictionaryValue> response) override {
+ response->GetString("credit_card_id", &server_id_);
}
bool IsResponseComplete() override { return true; }
void RespondToDelegate(PaymentsClientDelegate* delegate,
AutofillClient::PaymentsRpcResult result) override {
- delegate->OnDidUploadCard(result);
+ delegate->OnDidUploadCard(result, server_id_);
}
private:
PaymentsClient::UploadRequestDetails request_details_;
+ std::string server_id_;
};
} // namespace
« no previous file with comments | « components/autofill/core/browser/payments/payments_client.h ('k') | components/autofill/core/browser/personal_data_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698