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

Unified Diff: components/autofill/core/browser/autofill_manager.h

Issue 2829853008: Stores server card as a full server card when upload to server succeeds. (Closed)
Patch Set: Makes OnDidUploadCard public so TestAutofillManager in anonymous namespace can reference it. 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/autofill_manager.h
diff --git a/components/autofill/core/browser/autofill_manager.h b/components/autofill/core/browser/autofill_manager.h
index c8597275741e5c61a8aeeabfdd57a72a710d78b0..83915b57441687a54bf9be6268efc5af1f076f3c 100644
--- a/components/autofill/core/browser/autofill_manager.h
+++ b/components/autofill/core/browser/autofill_manager.h
@@ -272,6 +272,11 @@ class AutofillManager : public AutofillDownloadManager::Observer,
return form_interactions_ukm_logger_.get();
}
+ // payments::PaymentsClientDelegate:
+ // Exposed for testing.
+ void OnDidUploadCard(AutofillClient::PaymentsRpcResult result,
+ const std::string& server_id) override;
+
// Exposed for testing.
AutofillExternalDelegate* external_delegate() {
return external_delegate_;
@@ -301,7 +306,6 @@ class AutofillManager : public AutofillDownloadManager::Observer,
AutofillClient::PaymentsRpcResult result,
const base::string16& context_token,
std::unique_ptr<base::DictionaryValue> legal_message) override;
- void OnDidUploadCard(AutofillClient::PaymentsRpcResult result) override;
// payments::FullCardRequest::ResultDelegate:
void OnFullCardRequestSucceeded(const CreditCard& card,

Powered by Google App Engine
This is Rietveld 408576698