Rouslan Solomakhin would like Danyao Wang to review this change.
[Payment Request] Remove PaymentAddress.languageCode.
Intent to deprecate:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/ma2J2RumrmM/nmhnTmB5CAAJ
(Consensus on the thread is that removal is OK.)
Chrome status dashboard entry:
https://www.chromestatus.com/feature/4992562146312192
Bug: 877521
Change-Id: I5dc1cce6d313f2b4138b784899cff00078ee2abe
---
M chrome/android/java/src/org/chromium/chrome/browser/payments/AutofillAddress.java
M chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestImpl.java
M chrome/browser/ui/views/payments/payment_request_payment_response_browsertest.cc
M components/payments/content/payment_request_state_unittest.cc
M components/payments/content/payment_response_helper_unittest.cc
M components/payments/core/payment_address.cc
M components/payments/core/payment_address_unittest.cc
M components/payments/core/payment_request_data_util.cc
M components/payments/core/payment_request_data_util_unittest.cc
M components/payments/core/payments_validators.cc
M components/payments/core/payments_validators.h
M components/payments/core/payments_validators_unittest.cc
M components/payments/mojom/payment_request_data.mojom
M ios/chrome/browser/payments/payment_request_util_unittest.mm
M ios/chrome/browser/payments/payment_response_helper_unittest.mm
M ios/chrome/browser/ui/payments/payment_request_payment_response_egtest.mm
M ios/chrome/browser/web/resources/payment_request.js
M third_party/blink/renderer/modules/payments/address_errors.idl
M third_party/blink/renderer/modules/payments/on_payment_response_test.cc
M third_party/blink/renderer/modules/payments/payment_address.cc
M third_party/blink/renderer/modules/payments/payment_address.h
M third_party/blink/renderer/modules/payments/payment_address.idl
M third_party/blink/renderer/modules/payments/payment_address_test.cc
M third_party/blink/renderer/modules/payments/payment_request.cc
M third_party/blink/renderer/modules/payments/payment_response_test.cc
M third_party/blink/renderer/modules/payments/payments_validators.cc
M third_party/blink/renderer/modules/payments/payments_validators.h
M third_party/blink/renderer/modules/payments/payments_validators_test.cc
28 files changed, 22 insertions(+), 445 deletions(-)
To view, visit change 1464225. To unsubscribe, or for help writing mail filters, visit settings.
Danyao, PTAL.
LGTM!
Patch set 5:Code-Review +1
Tom, security owners PTAL:
components/payments/mojom/payment_request_data.mojom
Moe, owners PTAL ios/*
OWNERS stamp.
Patch set 6:Code-Review +1
LGTM
To view, visit change 1464225. To unsubscribe, or for help writing mail filters, visit settings.
Patch set 6:Commit-Queue +2
Try jobs failed on following builders:
chromium_presubmit on luci.chromium.try (JOB_FAILED, https://ci.chromium.org/b/8921444824897146240)
Chris: Blink API owners PTAL.
Commit Bot merged this change.
[Payment Request] Remove PaymentAddress.languageCode.
Intent to deprecate:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/ma2J2RumrmM/nmhnTmB5CAAJ
(Consensus on the thread is that removal is OK.)
Chrome status dashboard entry:
https://www.chromestatus.com/feature/4992562146312192
Bug: 877521
Change-Id: I5dc1cce6d313f2b4138b784899cff00078ee2abe
Reviewed-on: https://chromium-review.googlesource.com/c/1464225
Reviewed-by: Tom Sepez <tse...@chromium.org>
Reviewed-by: Moe Ahmadi <mah...@chromium.org>
Reviewed-by: Chris Harrelson <chri...@chromium.org>
Reviewed-by: Danyao Wang <dan...@chromium.org>
Commit-Queue: Rouslan Solomakhin <rou...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#632757}
D third_party/blink/web_tests/external/wpt/payment-request/historical.https-expected.txt
M third_party/blink/web_tests/virtual/stable/webexposed/global-interface-listing-expected.txt
M third_party/blink/web_tests/webexposed/global-interface-listing-expected.txt
31 files changed, 22 insertions(+), 459 deletions(-)