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

Remove ServiceWorker+MessagePort UA detection #7185

Open
wpt-issue-mover opened this issue Aug 31, 2017 · 0 comments
Open

Remove ServiceWorker+MessagePort UA detection #7185

wpt-issue-mover opened this issue Aug 31, 2017 · 0 comments

Comments

@wpt-issue-mover
Copy link

Originally posted as w3c/testharness.js#247 by @sfuqua on 10 Mar 2017, 21:16 UTC:

#239 introduces some changes to service worker testability. As part of that change, a UA sniff for Microsoft Edge is introduced due to the browser currently not supporting MessagePort in a ServiceWorker context - this is difficult to feature detect properly since the feature check needs to run in the worker context, not the client page's context.

This issue tracks removing the UA check either completely (once it is no longer needed) or with some sort of channel between the worker and the client page to communicate feature status.

The relevant code is at the top of the create_remote_worker function introduced in the referenced PR.

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

No branches or pull requests

3 participants