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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/payments/CardEditor.java

Issue 2515853002: [Payments] Fix new billing address label. (Closed)
Patch Set: Created 4 years, 1 month 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/android/java/src/org/chromium/chrome/browser/payments/CardEditor.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/payments/CardEditor.java b/chrome/android/java/src/org/chromium/chrome/browser/payments/CardEditor.java
index 5c377534ca55b24ea99d30f6bc093cc9c3f39f17..4cfb34ebb222246d929a5bf3b6a48070504c5b93 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/payments/CardEditor.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/payments/CardEditor.java
@@ -551,6 +551,13 @@ public class CardEditor extends EditorBase<AutofillPaymentInstrument>
// User has cancelled the address editor.
mBillingAddressField.setValue(null);
} else {
+ // Get the billing address label for the profile.
+ billingAddress.getProfile().setLabel(
+ PersonalDataManager.getInstance()
+ .getBillingAddressLabelForPaymentRequest(
+ billingAddress.getProfile()));
+ billingAddress.completeAddress(billingAddress.getProfile());
please use gerrit instead 2016/11/18 21:56:22 There seems to be no need to call "completeAddress
sebsg 2016/11/21 16:07:09 We have to re-assign the label for the AutofillAdd
+
// User has added a new complete address. Add it to the top of the
// dropdown.
mProfilesForBillingAddress.add(billingAddress.getProfile());

Powered by Google App Engine
This is Rietveld 408576698