New issue
Advanced search Search tips

Issue 654773 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Normalize billing address before sending to merchant website.

Project Member Reported by se...@chromium.org, Oct 11 2016

Issue description

^^^
 
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 25 2016

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

commit 8081c02908534b12968e52de312c2b7ca7b19456
Author: sebsg <sebsg@chromium.org>
Date: Tue Oct 25 02:17:38 2016

[Payments] Normalize billing address before sending to the merchant.

BUG= 654773 

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

[modify] https://crrev.com/8081c02908534b12968e52de312c2b7ca7b19456/chrome/android/java/src/org/chromium/chrome/browser/autofill/PersonalDataManager.java
[modify] https://crrev.com/8081c02908534b12968e52de312c2b7ca7b19456/chrome/android/java/src/org/chromium/chrome/browser/payments/AutofillPaymentInstrument.java
[modify] https://crrev.com/8081c02908534b12968e52de312c2b7ca7b19456/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentInstrument.java
[modify] https://crrev.com/8081c02908534b12968e52de312c2b7ca7b19456/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestImpl.java
[add] https://crrev.com/8081c02908534b12968e52de312c2b7ca7b19456/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentResponseHelper.java
[modify] https://crrev.com/8081c02908534b12968e52de312c2b7ca7b19456/chrome/android/java_sources.gni
[modify] https://crrev.com/8081c02908534b12968e52de312c2b7ca7b19456/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestUseStatsTest.java
[modify] https://crrev.com/8081c02908534b12968e52de312c2b7ca7b19456/chrome/browser/autofill/android/personal_data_manager_android.cc
[modify] https://crrev.com/8081c02908534b12968e52de312c2b7ca7b19456/chrome/browser/autofill/android/personal_data_manager_android.h
[modify] https://crrev.com/8081c02908534b12968e52de312c2b7ca7b19456/components/autofill/core/browser/payments/full_card_request.cc

Comment 2 by se...@chromium.org, Oct 27 2016

Status: Fixed (was: Started)

Comment 3 by se...@chromium.org, Oct 27 2016

Labels: -M-55 M-56
Steps to reproduce:
1) Add an address with "California" state through Autofill settings.
2) Add card "4111 1111 1111 1111" with that address as billing address in Autofill settings.
3) Pay using this card on https://googlechrome.github.io/samples/paymentrequest/credit-cards/

Expected:
The "billingAddress"/"region" should be "CA", not "California". (This happens because the address is normalized on the fly when user clicks [PAY].)
Thanks for providing verification steps.

Verified in - M56-56.0.2906.0
Components: -UI>Browser>Autofill>Payments UI>Browser>Payments

Sign in to add a comment