codeninjasllc/codecombat
1
0
Fork 0
mirror of https://github.com/codeninjasllc/codecombat.git synced 2024-11-23 23:58:02 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix #3519; restricting importScripts has always been buggy and we probably don't need it

Browse source
This commit is contained in:
Nick Winter 2016-04-17 17:32:37 -07:00
parent 15d762e93e
commit d739fbef9b
1 changed files with 0 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
app/assets/javascripts/workers/worker_world.js
View file

@ -84,12 +84,6 @@ var ensureLanguagesImportedFromUserCodeMap = function (userCodeMap) {
var restricted = ["XMLHttpRequest", "Worker"];
if (!self.navigator || !(self.navigator.userAgent.indexOf('MSIE') > 0) &&
!self.navigator.userAgent.match(/Trident.*rv\:11\./) &&
!self.navigator.userAgent.match(/Edge/)) {
// Can't restrict 'importScripts' in IE11, skip for all IE versions
restricted.push("importScripts");
}
for(var i = 0; i < restricted.length; ++i) {
// We could do way more from this: http://stackoverflow.com/questions/10653809/making-webworkers-a-safe-environment
Object.defineProperty(self, restricted[i], {