New issue
Advanced search Search tips

Issue 666048 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug



Sign in to add a comment

New Billing Address label is wrong

Project Member Reported by se...@chromium.org, Nov 16 2016

Issue description

It uses the company name instead of the person's name.

The reason is that is calls GetAddressLabelForPaymentRequest which is used for shipping address where we show the name separately.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 22 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/cc022347d5d434c2caddfa24568b772d66081dd1

commit cc022347d5d434c2caddfa24568b772d66081dd1
Author: sebsg <sebsg@chromium.org>
Date: Tue Nov 22 18:59:36 2016

[Payments] Fix new billing address label.

BUG= 666048 

Review-Url: https://codereview.chromium.org/2515853002
Cr-Commit-Position: refs/heads/master@{#433933}

[modify] https://crrev.com/cc022347d5d434c2caddfa24568b772d66081dd1/chrome/android/java/src/org/chromium/chrome/browser/autofill/PersonalDataManager.java
[modify] https://crrev.com/cc022347d5d434c2caddfa24568b772d66081dd1/chrome/android/java/src/org/chromium/chrome/browser/payments/AddressEditor.java
[modify] https://crrev.com/cc022347d5d434c2caddfa24568b772d66081dd1/chrome/android/java/src/org/chromium/chrome/browser/payments/AutofillAddress.java
[modify] https://crrev.com/cc022347d5d434c2caddfa24568b772d66081dd1/chrome/android/java/src/org/chromium/chrome/browser/payments/CardEditor.java
[modify] https://crrev.com/cc022347d5d434c2caddfa24568b772d66081dd1/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestImpl.java
[modify] https://crrev.com/cc022347d5d434c2caddfa24568b772d66081dd1/chrome/android/java/src/org/chromium/chrome/browser/payments/ui/PaymentOption.java
[modify] https://crrev.com/cc022347d5d434c2caddfa24568b772d66081dd1/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestBillingAddressTest.java
[modify] https://crrev.com/cc022347d5d434c2caddfa24568b772d66081dd1/chrome/browser/autofill/android/personal_data_manager_android.cc
[modify] https://crrev.com/cc022347d5d434c2caddfa24568b772d66081dd1/chrome/browser/autofill/android/personal_data_manager_android.h

Comment 2 by se...@chromium.org, Dec 1 2016

Cc: rouslan@chromium.org
Status: Fixed (was: Assigned)
Components: -UI>Browser>Autofill>Payments UI>Browser>Payments

Sign in to add a comment