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
Add idlharness.js test for Fullscreen #5903
Conversation
Adding a whole other spec's IDL as untested isn't commonly done, so this ran into multiple problems in idlharness.js, which were fixed: * test_primary_interface_of and test_interface_of were missing untested checks * the readonly (no setter) checks didn't know about [LenientSetter] There's still no coverage for [LenientSetter] actually working.
Firefox (nightly)Testing web-platform-tests at revision 1bca831 All results1 test ran/fullscreen/interfaces.html
|
Chrome (unstable)Testing web-platform-tests at revision 1bca831 All results1 test ran/fullscreen/interfaces.html
|
@@ -1684,6 +1691,9 @@ IdlInterface.prototype.test_interface_of = function(desc, obj, exception, expect | |||
for (var i = 0; i < this.members.length; i++) | |||
{ | |||
var member = this.members[i]; | |||
if (member.untested) { |
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.
Yes, the style in this file is inconsistent, so I tried to be locally consistent.
I'm not quite familiar with IDL tests... so probably @annevk should review this. |
I don't know the IDL harness code. I was actually asking on IRC who can so I'm surprised @jgraham assigns me! Having said that, if this runs I'd be okay with landing it... |
@annevk says not to expect speedy responses from @Ms2ger or @ayg, so I'll just go ahead and merge this. It will affect other tests using testharness.js, but should in those cases too only be skipping tests that should have been skipped all along. I checked XMLHttpRequest/interfaces.html and html/dom/interfaces.html before and after and as expected there's a reduction in the number of tests. |
Adding a whole other spec's IDL as untested isn't commonly done, so this
ran into multiple problems in idlharness.js, which were fixed:
There's still no coverage for [LenientSetter] actually working.
This change is