Closed
Bug 1380553
Opened 8 years ago
Closed 8 years ago
Web platform test of PaymentRequestUpdateEvent updateWith method fails
Categories
(Core :: DOM: Web Payments, defect)
Core
DOM: Web Payments
Tracking
()
RESOLVED
DUPLICATE
of bug 1379925
People
(Reporter: ben.tian, Assigned: ben.tian)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
Web platform test of PaymentRequestUpdateEvent updateWith method fails:
https://w3c-test.org/payment-request/payment-request-update-event-updatewith-method.https.html
2 failures:
[Calling .updateWith() with an undispatched untrusted event throws "InvalidStateError"]
assert_throws: untrusted event of type "just a test" must throw "InvalidStateError" function "() => { ev.updateWith(Promise.resolve()); }" did not throw
[Calling .updateWith() with a dispatched, untrusted event, throws "InvalidStateError"]
assert_throws: untrusted event of type "just a test" must throw "InvalidStateError" function "() => { ev.updateWith(Promise.resolve()) }" did not throw
---
Note bug 1379925 patch is required to run the test now, otherwise test crashes when constructing PaymentRequestUpdateEvent.
| Assignee | ||
Updated•8 years ago
|
Assignee: nobody → btian
| Assignee | ||
Comment 1•8 years ago
|
||
Attachment #8886058 -
Flags: review?(amarchesini)
Attachment #8886058 -
Flags: feedback?(alchen)
Updated•8 years ago
|
Attachment #8886058 -
Flags: feedback?(alchen) → feedback+
| Assignee | ||
Comment 2•8 years ago
|
||
Comment on attachment 8886058 [details] [diff] [review]
Patch 1 (v1): Throw InvalidStateError in method PaymentRequestUpdateEvent.updateWith if event is not trusted
May need to rebase according to bug 1379925 comment 4. Cancel r? first.
Attachment #8886058 -
Flags: review?(amarchesini)
| Assignee | ||
Comment 3•8 years ago
|
||
Patch in bug 1379925 comment 6 covers fix of this bug.
| Assignee | ||
Comment 4•8 years ago
|
||
Resolve DUPLICATE to bug 1379925 per comment 3.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•