Closed Bug 1497699 Opened 2 years ago Closed 2 years ago

Drop support for PaymentItem.type when removed from spec

Categories

(Core :: DOM: Web Payments, defect, P1)

defect

Tracking

()

RESOLVED FIXED
mozilla64
Tracking Status
firefox64 --- fixed

People

(Reporter: marcosc, Assigned: marcosc)

References

(Blocks 1 open bug, )

Details

(Keywords: dev-doc-complete)

Attachments

(1 file)

Basically want to revert bug 1432079.
Being removed from the spec. Front-end no longer needs this.
Assignee: nobody → mcaceres
Severity: normal → trivial
Status: NEW → ASSIGNED
Priority: -- → P1
Comment on attachment 9015720 [details]
Bug 1497699 - Drop support for PaymentItem.type. r=edenchuang,baku

Need IDL ok from Baku (trying to get this fixed so we don't need to be bugging you for IDL reviews).
Attachment #9015720 - Flags: review?(amarchesini)
Attachment #9015720 - Flags: review?(amarchesini) → review+
Keywords: checkin-needed
Pushed by ebalazs@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/56d027190d7b
Drop support for PaymentItem.type. r=edenchuang,baku
Keywords: checkin-needed
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/13546 for changes under testing/web-platform/tests
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.
https://hg.mozilla.org/mozilla-central/rev/56d027190d7b
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
Flags: qe-verify-
Added PaymentItem dictionary to documentation, with this field marked as obsolete:

https://developer.mozilla.org/en-US/docs/Web/API/PaymentItem

Submitted PR to BCD repo to add this dictionary and include this new data:

https://github.com/mdn/browser-compat-data/pull/3134
Removed type entirely after MattN pointed out nobody ever shipped it before it was removed, and only Firefox ever implemented it at all.
You need to log in before you can comment on or make changes to this bug.