New issue
Advanced search Search tips

Issue 665190 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Launch-OWP
Launch-Accessibility: ----
Launch-Exp-Leadership: ----
Launch-Leadership: ----
Launch-Legal: ----
Launch-M-Approved: ----
Launch-M-Target: 57-Stable
Launch-Privacy: ----
Launch-Security: ----
Launch-Test: ----
Launch-UI: ----
Rollout-Type: ----



Sign in to add a comment

Implement updated basic card spec

Project Member Reported by rouslan@chromium.org, Nov 14 2016

Issue description

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/+/e71bf006592c9ebba8c459848308b27e3efa63b4

commit e71bf006592c9ebba8c459848308b27e3efa63b4
Author: rouslan <rouslan@chromium.org>
Date: Tue Nov 22 01:29:21 2016

Consider payment method specific data when filtering payment apps.

Chrome filters payment apps by payment method, e.g., "visa",
"mastercard", or "https://android.com/pay". With the upcoming
implementation of the new "basic-card" payment method, the filtering
needs to be based on the method specific data. The following snippet,
for example, specifies the "visa" and "debit" filters inside of the
method specific data.

new PaymentRequest([{
                     supportedMethods: ['basic-card'],
                     data: {
                       supportedNetworks: ['visa'],
                       supportedTypes: ['debit']
                     }
                   }],
                   shoppingCartContents);

This patch adds ability to filter out payment apps based on method
specific data as well as the list of payment methods that they support.

BUG= 665190 

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

[modify] https://crrev.com/e71bf006592c9ebba8c459848308b27e3efa63b4/chrome/android/java/src/org/chromium/chrome/browser/payments/AutofillPaymentApp.java
[modify] https://crrev.com/e71bf006592c9ebba8c459848308b27e3efa63b4/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentApp.java
[modify] https://crrev.com/e71bf006592c9ebba8c459848308b27e3efa63b4/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestImpl.java
[modify] https://crrev.com/e71bf006592c9ebba8c459848308b27e3efa63b4/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestTestBase.java

Project Member

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

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/clank/internal/apps/+/04718e1c9d8a359ac22dafdd08c874eb94a08842

commit 04718e1c9d8a359ac22dafdd08c874eb94a08842
Author: Rouslan Solomakhin <rouslan@google.com>
Date: Tue Nov 22 02:04:21 2016

Labels: -Pri-1 -Type-Feature M-57 OS-Android Pri-2 Type-Launch-OWP
Labels: Launch-M-Target-57-Stable
Project Member

Comment 5 by bugdroid1@chromium.org, Jan 9 2017

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

commit b1be35c4c222d904ae02ea5c5dfdf1972d031f1e
Author: rouslan <rouslan@chromium.org>
Date: Mon Jan 09 17:22:36 2017

Implement the new basic card specification.

The basic card specification has been updated to include card types
(credit, debit, prepaid), which are distinct from card networks (amex,
discover, mastercard, visa, etc). This patch adds support for the new
format without removing support for the old format. Parsing of the JSON
data that encodes the parameters happens in the renderer for safety.

Old format (still supported):
new PaymentRequest([{
                     supportedMethods: ['visa']
                   }],
                   shoppingCartContents);

New format (in this patch):
new PaymentRequest([{
                     supportedMethods: ['basic-card'],
                     data: {
                       supportedNetworks: ['visa'],
                       supportedTypes: ['debit']
                     }
                   }],
                   shoppingCartContents);

Basic card spec:
https://w3c.github.io/webpayments-methods-card/

Intent to implement and ship:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/IYRjdUKxCoM/8B-jp4g9AgAJ

BUG= 665190 

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

[modify] https://crrev.com/b1be35c4c222d904ae02ea5c5dfdf1972d031f1e/chrome/android/java/src/org/chromium/chrome/browser/payments/AutofillPaymentApp.java
[modify] https://crrev.com/b1be35c4c222d904ae02ea5c5dfdf1972d031f1e/chrome/android/java/src/org/chromium/chrome/browser/payments/AutofillPaymentInstrument.java
[modify] https://crrev.com/b1be35c4c222d904ae02ea5c5dfdf1972d031f1e/chrome/android/java/src/org/chromium/chrome/browser/payments/CardEditor.java
[modify] https://crrev.com/b1be35c4c222d904ae02ea5c5dfdf1972d031f1e/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentApp.java
[modify] https://crrev.com/b1be35c4c222d904ae02ea5c5dfdf1972d031f1e/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestImpl.java
[modify] https://crrev.com/b1be35c4c222d904ae02ea5c5dfdf1972d031f1e/chrome/android/java_sources.gni
[add] https://crrev.com/b1be35c4c222d904ae02ea5c5dfdf1972d031f1e/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestBasicCardTest.java
[add] https://crrev.com/b1be35c4c222d904ae02ea5c5dfdf1972d031f1e/chrome/test/data/android/payments/basic_card.js
[add] https://crrev.com/b1be35c4c222d904ae02ea5c5dfdf1972d031f1e/chrome/test/data/android/payments/payment_request_basic_card_test.html
[modify] https://crrev.com/b1be35c4c222d904ae02ea5c5dfdf1972d031f1e/components/payments/payment_request.mojom
[modify] https://crrev.com/b1be35c4c222d904ae02ea5c5dfdf1972d031f1e/third_party/WebKit/LayoutTests/payments/payment-request-interface.html
[modify] https://crrev.com/b1be35c4c222d904ae02ea5c5dfdf1972d031f1e/third_party/WebKit/Source/modules/modules_idl_files.gni
[add] https://crrev.com/b1be35c4c222d904ae02ea5c5dfdf1972d031f1e/third_party/WebKit/Source/modules/payments/BasicCardRequest.idl
[modify] https://crrev.com/b1be35c4c222d904ae02ea5c5dfdf1972d031f1e/third_party/WebKit/Source/modules/payments/PaymentRequest.cpp
[modify] https://crrev.com/b1be35c4c222d904ae02ea5c5dfdf1972d031f1e/third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in

Status: Fixed (was: Started)
Project Member

Comment 9 by bugdroid1@chromium.org, Aug 9

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

commit 9558d815524be81ef126226d4ad420b9f9987c21
Author: Eric Willigers <ericwilligers@chromium.org>
Date: Thu Aug 09 02:07:44 2018

Retire PaymentRequestBasicCard flag

Support for basic card payment method specification shipped in M57.
https://codereview.chromium.org/2501593003
https://www.chromestatus.com/feature/5408502604365824

BUG= 665190 

Change-Id: I29780c35a6ccf5c9d5a94b21e38d8605ba8b77f9
Reviewed-on: https://chromium-review.googlesource.com/1166744
Reviewed-by: Rouslan Solomakhin <rouslan@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Eric Willigers <ericwilligers@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581754}
[modify] https://crrev.com/9558d815524be81ef126226d4ad420b9f9987c21/third_party/blink/renderer/modules/payments/payment_request.cc
[modify] https://crrev.com/9558d815524be81ef126226d4ad420b9f9987c21/third_party/blink/renderer/platform/runtime_enabled_features.json5

Sign in to add a comment