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

Side by Side 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/autofill/android/personal_data_manager_android.h" 5 #include "chrome/browser/autofill/android/personal_data_manager_android.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <algorithm> 8 #include <algorithm>
9 #include <memory> 9 #include <memory>
10 #include <utility> 10 #include <utility>
(...skipping 514 matching lines...) Expand 10 before | Expand all | Expand 10 after
525 } else { 525 } else {
526 card.set_guid(guid); 526 card.set_guid(guid);
527 personal_data_manager_->UpdateCreditCard(card); 527 personal_data_manager_->UpdateCreditCard(card);
528 } 528 }
529 return ConvertUTF8ToJavaString(env, card.guid()); 529 return ConvertUTF8ToJavaString(env, card.guid());
530 } 530 }
531 531
532 void PersonalDataManagerAndroid::UpdateServerCardBillingAddress( 532 void PersonalDataManagerAndroid::UpdateServerCardBillingAddress(
533 JNIEnv* env, 533 JNIEnv* env,
534 const JavaParamRef<jobject>& unused_obj, 534 const JavaParamRef<jobject>& unused_obj,
535 const JavaParamRef<jstring>& jcard_server_id, 535 const JavaParamRef<jobject>& jcard) {
536 const JavaParamRef<jstring>& jbilling_address_id) { 536 CreditCard card;
537 CreditCard card("", kSettingsOrigin); 537 PopulateNativeCreditCardFromJava(jcard, env, &card);
538 card.set_record_type(CreditCard::MASKED_SERVER_CARD); 538
539 card.set_server_id(ConvertJavaStringToUTF8(env, jcard_server_id)); 539 personal_data_manager_->UpdateServerCardMetadata(card);
540 card.set_billing_address_id(ConvertJavaStringToUTF8(env,
541 jbilling_address_id));
542 personal_data_manager_->UpdateServerCardBillingAddress(card);
543 } 540 }
544 541
545 ScopedJavaLocalRef<jstring> PersonalDataManagerAndroid::GetBasicCardPaymentType( 542 ScopedJavaLocalRef<jstring> PersonalDataManagerAndroid::GetBasicCardPaymentType(
546 JNIEnv* env, 543 JNIEnv* env,
547 const JavaParamRef<jobject>& unused_obj, 544 const JavaParamRef<jobject>& unused_obj,
548 const JavaParamRef<jstring>& jcard_number, 545 const JavaParamRef<jstring>& jcard_number,
549 const jboolean jempty_if_invalid) { 546 const jboolean jempty_if_invalid) {
550 base::string16 card_number = ConvertJavaStringToUTF16(env, jcard_number); 547 base::string16 card_number = ConvertJavaStringToUTF16(env, jcard_number);
551 548
552 if (static_cast<bool>(jempty_if_invalid) && 549 if (static_cast<bool>(jempty_if_invalid) &&
(...skipping 374 matching lines...) Expand 10 before | Expand all | Expand 10 after
927 base::android::ConvertJavaStringToUTF16(env, jcountry_name))); 924 base::android::ConvertJavaStringToUTF16(env, jcountry_name)));
928 } 925 }
929 926
930 static jlong Init(JNIEnv* env, const JavaParamRef<jobject>& obj) { 927 static jlong Init(JNIEnv* env, const JavaParamRef<jobject>& obj) {
931 PersonalDataManagerAndroid* personal_data_manager_android = 928 PersonalDataManagerAndroid* personal_data_manager_android =
932 new PersonalDataManagerAndroid(env, obj); 929 new PersonalDataManagerAndroid(env, obj);
933 return reinterpret_cast<intptr_t>(personal_data_manager_android); 930 return reinterpret_cast<intptr_t>(personal_data_manager_android);
934 } 931 }
935 932
936 } // namespace autofill 933 } // namespace autofill
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698