JoBoEVD/UglifyJS
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add more globals, whereof most defined after es5.1

Browse Source 
Also do not pollute env with mocks replacing standard globals
This commit is contained in:
Anthony Van de Gejuchte
2016-07-16 18:56:53 +02:00
committed by Richard van Velzen
parent 72a9d799b6
commit 88384cf351
2 changed files with 25 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
lib/propmangle.js
View File

@@ -45,15 +45,26 @@
function find_builtins() { function find_builtins() {
// Compatibility fix for es5.1 and earlier where Symbol isn't defined // Compatibility fix for some standard defined globals not defined on every js environment
if (!global.Symbol) { var new_globals = ["Symbol", "Map", "Promise", "Proxy", "Reflect", "Set", "WeakMap", "WeakSet"];
global.Symbol = new Function(); var objects = {};
}
new_globals.forEach(function (new_global) {
objects[new_global] = global[new_global] || new Function();
});
var a = []; var a = [];
[ Object, Array, Function, Number, [ Object, Array, Function, Number,
String, Boolean, Error, Math, String, Boolean, Error, Math,
Date, RegExp, Symbol Date, RegExp, objects.Symbol, ArrayBuffer,
DataView, decodeURI, decodeURIComponent,
encodeURI, encodeURIComponent, eval, EvalError,
Float32Array, Float64Array, Int8Array, Int16Array,
Int32Array, isFinite, isNaN, JSON, objects.Map, parseFloat,
parseInt, objects.Promise, objects.Proxy, RangeError, ReferenceError,
objects.Reflect, objects.Set, SyntaxError, TypeError, Uint8Array,
Uint8ClampedArray, Uint16Array, Uint32Array, URIError,
objects.WeakMap, objects.WeakSet
].forEach(function(ctor){ ].forEach(function(ctor){
Object.getOwnPropertyNames(ctor).map(add); Object.getOwnPropertyNames(ctor).map(add);
if (ctor.prototype) { if (ctor.prototype) {

10
test/mocha/builtins.js
View File

@@ -4,7 +4,7 @@ var assert = require("assert");
describe("builtins", function() { describe("builtins", function() {
it ("Should not mangle builtins", function() { it ("Should not mangle builtins", function() {
var test = "function foo(something){\n" + var test = "function foo(something){\n" +
" return [Object,Array,Function,Number,String,Boolean,Error,Math,Date,RegExp,Symbol,something];\n" + " return [Object,Array,Function,Number,String,Boolean,Error,Math,Date,RegExp,Symbol,Map,Promise,Proxy,Reflect,Set,WeakMap,WeakSet,Float32Array,something];\n" +
"};"; "};";
var result = UglifyJS.minify(test, {fromString: true, parse: {bare_returns: true}}).code; var result = UglifyJS.minify(test, {fromString: true, parse: {bare_returns: true}}).code;
@@ -22,5 +22,13 @@ describe("builtins", function() {
assert.notEqual(result.indexOf("Date"), -1); assert.notEqual(result.indexOf("Date"), -1);
assert.notEqual(result.indexOf("RegExp"), -1); assert.notEqual(result.indexOf("RegExp"), -1);
assert.notEqual(result.indexOf("Symbol"), -1); assert.notEqual(result.indexOf("Symbol"), -1);
assert.notEqual(result.indexOf("Promise"), -1);
assert.notEqual(result.indexOf("Proxy"), -1);
assert.notEqual(result.indexOf("Reflect"), -1);
assert.notEqual(result.indexOf("Set"), -1);
assert.notEqual(result.indexOf("WeakMap"), -1);
assert.notEqual(result.indexOf("WeakSet"), -1);
assert.notEqual(result.indexOf("Map"), -1);
assert.notEqual(result.indexOf("Float32Array"), -1);
}); });
}); });