Get rid of new Function

[mourner]

It was not necessary, because the created function was never called and immediately stringified afterwards for the blob.

Enables using webworkify in environments where CSP allows creating blobs but bans eval.

cc @anandthakker

[anandthakker]

Interesting -- I wonder why it used Function previously; is there any chance that the stringification varies by platform/browser? (cc @substack )

[mourner]

@anandthakker it shouldn't vary since it's standardized. And I don't think webworkify relies on a string -> function -> string conversion in any meaningful way.

[anandthakker]

cool, I'm sold

[mourner]

One thing that may be affected is browserify's sources object (5-th argument), which will be augmented. It probably doesn't matter though because generated modules are never used anywhere else. If they do, we can change the behavior so that sources remains untouched.

[anandthakker]

Tested this locally and it looks good. Merging.