diff options
author | rouslan <rouslan@chromium.org> | 2016-01-14 12:10:42 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-01-14 20:11:44 +0000 |
commit | acfb6cffa322cac5195cc49f573809f7426061e8 (patch) | |
tree | d35981b09bab2139653caea8e8a74422e4b5674b /android_webview/native | |
parent | 2cb6a94651373c776b4a4d75a9f916e76d4f5021 (diff) | |
download | chromium_src-acfb6cffa322cac5195cc49f573809f7426061e8.zip chromium_src-acfb6cffa322cac5195cc49f573809f7426061e8.tar.gz chromium_src-acfb6cffa322cac5195cc49f573809f7426061e8.tar.bz2 |
Add card details and legal message to Android save credit card infobar.
Based on the mocks at:
https://folio.googleplex.com/chrome-ux/mocks/304-payments-zero-integration/ZI_Upstream#%2F11-updated-mobile%20prompts%20-%20proposal.png
This patch also updates the AutofillClient interface to pass in the
CreditCard object into ConfirmSaveCard*() methods. This allows the UI to
show some card information to the user.
TBR=estade@chromium.org
BUG=535784
Review URL: https://codereview.chromium.org/1540423004
Cr-Commit-Position: refs/heads/master@{#369521}
Diffstat (limited to 'android_webview/native')
-rw-r--r-- | android_webview/native/aw_autofill_client.cc | 6 | ||||
-rw-r--r-- | android_webview/native/aw_autofill_client.h | 8 |
2 files changed, 9 insertions, 5 deletions
diff --git a/android_webview/native/aw_autofill_client.cc b/android_webview/native/aw_autofill_client.cc index 911571a..6d84526 100644 --- a/android_webview/native/aw_autofill_client.cc +++ b/android_webview/native/aw_autofill_client.cc @@ -222,13 +222,15 @@ void AwAutofillClient::OnUnmaskVerificationResult(PaymentsRpcResult result) { } void AwAutofillClient::ConfirmSaveCreditCardLocally( + const autofill::CreditCard& card, const base::Closure& callback) { NOTIMPLEMENTED(); } void AwAutofillClient::ConfirmSaveCreditCardToCloud( - const base::Closure& callback, - scoped_ptr<base::DictionaryValue> legal_message) { + const autofill::CreditCard& card, + scoped_ptr<base::DictionaryValue> legal_message, + const base::Closure& callback) { NOTIMPLEMENTED(); } diff --git a/android_webview/native/aw_autofill_client.h b/android_webview/native/aw_autofill_client.h index 290c669..52cd4b1 100644 --- a/android_webview/native/aw_autofill_client.h +++ b/android_webview/native/aw_autofill_client.h @@ -67,10 +67,12 @@ class AwAutofillClient : public autofill::AutofillClient, const autofill::CreditCard& card, base::WeakPtr<autofill::CardUnmaskDelegate> delegate) override; void OnUnmaskVerificationResult(PaymentsRpcResult result) override; - void ConfirmSaveCreditCardLocally(const base::Closure& callback) override; + void ConfirmSaveCreditCardLocally(const autofill::CreditCard& card, + const base::Closure& callback) override; void ConfirmSaveCreditCardToCloud( - const base::Closure& callback, - scoped_ptr<base::DictionaryValue> legal_message) override; + const autofill::CreditCard& card, + scoped_ptr<base::DictionaryValue> legal_message, + const base::Closure& callback) override; void LoadRiskData( const base::Callback<void(const std::string&)>& callback) override; bool HasCreditCardScanFeature() override; |