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

IE10 can't create WebWorkers from blob:-URLs #12

Closed
Sebmaster opened this issue Feb 25, 2013 · 5 comments
Closed

IE10 can't create WebWorkers from blob:-URLs #12

Sebmaster opened this issue Feb 25, 2013 · 5 comments

Comments

@Sebmaster
Copy link
Collaborator

IE10 throws a SecurityError when trying to instantiate a Worker from a blob URL.

According to this (http://stackoverflow.com/questions/10343913/how-to-create-a-web-worker-from-a-string) SO question, you have to use a wrapper script which basically calls eval on the input string.

@Sebmaster
Copy link
Collaborator Author

Even though the pull request in #13 fixes the behaviour for now, it might be worth thinking about providing an extra eval.js-Worker to allow WebWorkers in IE10.

What's your stance on that @adambom, shall I have a go at that too, or close this one for now?

@adambom
Copy link
Collaborator

adambom commented Feb 25, 2013

I don't always use eval, but when I do it's because IE forces me to.

I think we need to do this. I want to try and support all the browser quirks, if possible. I think it's useful to have a library that normalizes web workers across all the browsers. We may discover that this exposes security flaws, but we'll cross that bridge when we come to it.

IE10 support is especially important, I think, because it's the emerging standard for building Metro desktop apps. Developers are going to need a clean and easy to use framework for multi-core processing.

@Sebmaster
Copy link
Collaborator Author

Okay then, since this requires me to work in the RemoteRef constructor again, are there any changes you want to have in #11 or #13, or are they merge-ready?

@adambom
Copy link
Collaborator

adambom commented Feb 25, 2013

Ready to go. They're merged in.

@Sebmaster
Copy link
Collaborator Author

For now fixed by e063e2e.

Getting WebWorkers to work in IE10: #16.

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

No branches or pull requests

2 participants