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

Unified Diff: chrome/browser/autofill/android/personal_data_manager_android.cc

Issue 2625413002: [Payments] Update metadata instead of billing address for server cards. (Closed)
Patch Set: Updated comment Created 3 years, 11 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: chrome/browser/autofill/android/personal_data_manager_android.cc
diff --git a/chrome/browser/autofill/android/personal_data_manager_android.cc b/chrome/browser/autofill/android/personal_data_manager_android.cc
index 3a5ad199fa1ab9d30acf74e136fa25c8d6858177..782f25cf70a10e5f5a1fbd10752e11eb2ba90bc4 100644
--- a/chrome/browser/autofill/android/personal_data_manager_android.cc
+++ b/chrome/browser/autofill/android/personal_data_manager_android.cc
@@ -532,14 +532,11 @@ ScopedJavaLocalRef<jstring> PersonalDataManagerAndroid::SetCreditCard(
void PersonalDataManagerAndroid::UpdateServerCardBillingAddress(
JNIEnv* env,
const JavaParamRef<jobject>& unused_obj,
- const JavaParamRef<jstring>& jcard_server_id,
- const JavaParamRef<jstring>& jbilling_address_id) {
- CreditCard card("", kSettingsOrigin);
- card.set_record_type(CreditCard::MASKED_SERVER_CARD);
- card.set_server_id(ConvertJavaStringToUTF8(env, jcard_server_id));
- card.set_billing_address_id(ConvertJavaStringToUTF8(env,
- jbilling_address_id));
- personal_data_manager_->UpdateServerCardBillingAddress(card);
+ const JavaParamRef<jobject>& jcard) {
+ CreditCard card;
+ PopulateNativeCreditCardFromJava(jcard, env, &card);
+
+ personal_data_manager_->UpdateServerCardMetadata(card);
}
ScopedJavaLocalRef<jstring> PersonalDataManagerAndroid::GetBasicCardPaymentType(

Powered by Google App Engine
This is Rietveld 408576698