Align PaymentAppManifest and PaymentAppOption dictionaries with the spec |
||||
Issue descriptionWe have two payment-related dictionaries in Blink that are different from the spec in interesting ways: https://cs.chromium.org/chromium/src/third_party/WebKit/Source/modules/payments/PaymentAppManifest.idl https://cs.chromium.org/chromium/src/third_party/WebKit/Source/modules/payments/PaymentAppOption.idl PaymentAppManifest doesn't exist in the spec anymore, and this is PaymentAppOption: https://w3c.github.io/webpayments-payment-apps-api/#payment-options It's PaymentAppOptions with a trailing 's' in the spec, but the names of dictionaries aren't observable, so that's not a big deal. Important differences: * We have a "DOMString? icon" member but the spec has "sequence<ImageObjects> icons". * We have an id member, but the spec does not. This bug can remain open until Blink's IDL for these dictionaries is a strict subset of the spec, or matches it exactly.
,
Mar 9 2017
foolip@, Yes, recently the shape of APIs was changed. (For example, PaymentAppManifest was already removed from the spec.) The work is not trivial because it requires structure change. Anyway, I'm working on the issue right now. So, I'll take this issue. Thanks :)
,
Mar 9 2017
,
Mar 9 2017
Thanks Jinho, hard work is hard :)
,
May 21 2017
|
||||
►
Sign in to add a comment |
||||
Comment 1 by foolip@chromium.org
, Mar 9 2017