Skip to content
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

page visibility idlharness test is broken #5116

Closed
bzbarsky opened this issue Mar 10, 2017 · 2 comments · Fixed by #5204
Closed

page visibility idlharness test is broken #5116

bzbarsky opened this issue Mar 10, 2017 · 2 comments · Fixed by #5204

Comments

@bzbarsky
Copy link
Contributor

If you implement onvisibilitychange, it will happily tell you "Unrecognized type EventHandler" and fail the test.

Looks like @plehegar forgot to add a definition of EventHandler in 47ebaea

@plehegar
Copy link
Member

@bzbarsky I wasn't able to confirm that the fix removes the failure. If you can check and confirm the fix, that would be nice.

@bzbarsky
Copy link
Contributor Author

Once the obvisibilitychange fix in https://bugzilla.mozilla.org/show_bug.cgi?id=1333912 lands I should be able to do that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants