Web platform test of PaymentRequestUpdateEvent constructor (secure context) fails

RESOLVED FIXED in Firefox 56

Status

()

defect
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: ben.tian, Assigned: ben.tian)

Tracking

(Blocks 1 bug)

unspecified
mozilla56
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox56 fixed)

Details

Attachments

(1 attachment)

Web platform test of PaymentRequestUpdateEvent constructor (secure context) fails:
https://w3c-test.org/payment-request/payment-request-update-event-constructor.https.html

[PaymentRequestUpdateEvent can be constructed with an EventInitDict, even if not trusted]
assert_false: constructed in script, so not be trusted expected false got true

--
Note bug 1379925 patch is required to run the test now, otherwise test crashes when constructing PaymentRequestUpdateEvent.
Attachment #8886014 - Attachment description: Bug 1380552 - Do not override method |IsTrusted| in PaymentRequestUpdateEvent → Patch 1 (v1): Do not override method |IsTrusted| in PaymentRequestUpdateEvent
Attachment #8886014 - Flags: feedback?(alchen) → feedback+
Comment on attachment 8886014 [details] [diff] [review]
[final] Patch 1: Do not override method |IsTrusted| in PaymentRequestUpdateEvent, f=alchen, r=baku

May need to rebase according to bug 1379925 comment 4. Cancel r? first.
Attachment #8886014 - Flags: review?(amarchesini)
Comment on attachment 8886014 [details] [diff] [review]
[final] Patch 1: Do not override method |IsTrusted| in PaymentRequestUpdateEvent, f=alchen, r=baku

No need to revise. Restore r?.
Attachment #8886014 - Flags: review?(amarchesini)
No longer depends on: 1379925
Comment on attachment 8886014 [details] [diff] [review]
[final] Patch 1: Do not override method |IsTrusted| in PaymentRequestUpdateEvent, f=alchen, r=baku

Review of attachment 8886014 [details] [diff] [review]:
-----------------------------------------------------------------

Be sure that SetTrusted(true) has been called before dispatching the event.
Attachment #8886014 - Flags: review?(amarchesini) → review+
(In reply to Andrea Marchesini [:baku] from comment #4)
> Be sure that SetTrusted(true) has been called before dispatching the event.

Yes it's called in
http://searchfox.org/mozilla-central/rev/cef8389c687203085dc6b52de2fbd0260d7495bf/dom/payments/PaymentRequest.cpp#558
Attachment #8886014 - Attachment description: Patch 1 (v1): Do not override method |IsTrusted| in PaymentRequestUpdateEvent → [final] Patch 1: Do not override method |IsTrusted| in PaymentRequestUpdateEvent, f=alchen, r=baku
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/770646a7519a
Do not override method |IsTrusted| in PaymentRequestUpdateEvent. r=baku
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/770646a7519a
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.