-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Test that MediaQueryList extends EventTarget #20401
Test that MediaQueryList extends EventTarget #20401
Conversation
@@ -48,7 +48,7 @@ | |||
triggerMQLEvent(mql); | |||
await waitForChangesReported(); | |||
|
|||
assert_true(_event instanceof MediaQueryListEvent); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This assertion was incorrect because mql
is from another Realm.
@@ -159,10 +159,10 @@ | |||
|
|||
triggerMQLEvent(mql); | |||
await waitForChangesReported(); | |||
assert_array_equals(calls, ["addListener", "addEventListener"], "triggerMQLEvent"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please see #20423.
Firefox fails some subtests in MediaQueryList-extends-EventTarget-interop.html , is there a bug filed for that? |
* Make sure "handleEvent" property is looked up * Extract `uncaught_error_test` helper WebKit issue: https://bugs.webkit.org/show_bug.cgi?id=203288 Follow-up of #15105 and #20401.
WebKit issue: MediaQueryList should extend EventTarget.
Related: #19841, #18390, #18389.
//cc @zcorpan, @jdm