Upgrade bundling to uglify-es

dy · dy · commit 328af5fdd544 · 2017-11-28T18:02:47.000-05:00
Conflicts:
	package.json
diff --git a/package.json b/package.json
@@ -56,6 +56,8 @@
     "3d-view": "^2.0.0",
     "@plotly/d3-sankey": "^0.5.0",
     "alpha-shape": "^1.0.0",
+    "bubleify": "^1.0.0",
+    "canvas-fit": "^1.5.0",
     "color-rgba": "^1.1.1",
     "convex-hull": "^1.0.3",
     "country-regex": "^1.1.0",
@@ -86,6 +88,7 @@
     "has-hover": "^1.0.1",
     "mapbox-gl": "^0.22.0",
     "matrix-camera-controller": "^2.1.3",
+    "minify-stream": "^1.1.0",
     "mouse-change": "^1.4.0",
     "mouse-event-offset": "^3.0.2",
     "mouse-wheel": "^1.0.2",
@@ -140,7 +143,6 @@
     "read-last-lines": "^1.1.0",
     "requirejs": "^2.3.1",
     "through2": "^2.0.3",
-    "uglify-js": "^2.8.12",
     "watchify": "^3.9.0",
     "xml2js": "^0.4.16"
   }
diff --git a/tasks/bundle.js b/tasks/bundle.js
@@ -33,6 +33,7 @@ _bundle(constants.pathToPlotlyIndex, constants.pathToPlotlyDist, {
     pathToMinBundle: constants.pathToPlotlyDistMin
 });
 
+
 // Browserify the geo assets
 _bundle(constants.pathToPlotlyGeoAssetsSrc, constants.pathToPlotlyGeoAssetsDist, {
     standalone: 'PlotlyGeoAssets'
@@ -53,3 +54,4 @@ constants.partialBundlePaths.forEach(function(pathObj) {
         pathToMinBundle: pathObj.distMin
     });
 });
+
diff --git a/tasks/util/browserify_wrapper.js b/tasks/util/browserify_wrapper.js
@@ -2,11 +2,11 @@ var fs = require('fs');
 var path = require('path');
 
 var browserify = require('browserify');
-var UglifyJS = require('uglify-js');
+var bubleify = require('bubleify');
+var minify = require('minify-stream');
 
 var constants = require('./constants');
 var compressAttributes = require('./compress_attributes');
-var patchMinified = require('./patch_minified');
 var strictD3 = require('./strict_d3');
 
 /** Convenience browserify wrapper
@@ -46,30 +46,30 @@ module.exports = function _bundle(pathToIndex, pathToBundle, opts) {
     }
 
     var b = browserify(pathToIndex, browserifyOpts);
-    var bundleWriteStream = fs.createWriteStream(pathToBundle);
 
-    bundleWriteStream.on('finish', function() {
-        logger(pathToBundle);
-        if(opts.then) {
-            opts.then();
-        }
-    });
+    b.transform(bubleify, constants.bubleifyOptions);
 
-    b.bundle(function(err, buf) {
+    var bundleStream = b.bundle(function(err, buf) {
         if(err) throw err;
+    })
 
-        if(outputMinified) {
-            var minifiedCode = UglifyJS.minify(buf.toString(), constants.uglifyOptions).code;
-            minifiedCode = patchMinified(minifiedCode);
-
-            fs.writeFile(pathToMinBundle, minifiedCode, function(err) {
-                if(err) throw err;
-
+    if (outputMinified) {
+        bundleStream
+            .pipe(minify(constants.uglifyOptions))
+            .pipe(fs.createWriteStream(pathToMinBundle))
+            .on('finish', function() {
                 logger(pathToMinBundle);
             });
-        }
-    })
-    .pipe(bundleWriteStream);
+    }
+
+    bundleStream
+        .pipe(fs.createWriteStream(pathToBundle))
+        .on('finish', function() {
+            logger(pathToBundle);
+            if(opts.then) {
+                opts.then();
+            }
+        });
 };
 
 function logger(pathToOutput) {
diff --git a/tasks/util/constants.js b/tasks/util/constants.js
@@ -84,16 +84,29 @@ module.exports = {
     testContainerHome: '/var/www/streambed/image_server/plotly.js',
 
     uglifyOptions: {
-        fromString: true,
         mangle: true,
         compress: {
-            warnings: false,
-            screw_ie8: true
+            warnings: false
         },
         output: {
             beautify: false,
             ascii_only: true
-        }
+        },
+        sourceMap: false
+    },
+
+    bubleifyOptions: {
+        target: {
+            chrome: 48,
+            firefox: 44,
+            edge: 12
+        },
+        transforms: {
+            arrow: true,
+            defaultParameter: false,
+            dangerousForOf: true,
+        },
+        sourceMap: false
     },
 
     licenseDist: [
diff --git a/tasks/util/patch_minified.js b/tasks/util/patch_minified.js
