workaround webkit parsing error

apply `webkit` to jetstream tests
mishoo · Jun 5, 2017 · 2b6b61e · 2b6b61e
1 parent 540220b
commit 2b6b61e
Show file tree

Hide file tree

Showing 4 changed files with 37 additions and 2 deletions.
diff --git a/lib/output.js b/lib/output.js
@@ -70,6 +70,7 @@ function OutputStream(options) {
         semicolons       : true,
         shebang          : true,
         source_map       : null,
+        webkit           : false,
         width            : 80,
         wrap_iife        : false,
     }, true);
@@ -597,6 +598,13 @@ function OutputStream(options) {
             return true;
         }
 
+        if (output.option('webkit')) {
+            var p = output.parent();
+            if (p instanceof AST_PropAccess && p.expression === this) {
+                return true;
+            }
+        }
+
         if (output.option('wrap_iife')) {
             var p = output.parent();
             return p instanceof AST_Call && p.expression === this;

diff --git a/test/compress/functions.js b/test/compress/functions.js
@@ -267,3 +267,29 @@ issue_203: {
     }
     expect_stdout: "42"
 }
+
+no_webkit: {
+    beautify = {
+        webkit: false,
+    }
+    input: {
+        console.log(function() {
+            1 + 1;
+        }.a = 1);
+    }
+    expect_exact: "console.log(function(){1+1}.a=1);"
+    expect_stdout: "1"
+}
+
+webkit: {
+    beautify = {
+        webkit: true,
+    }
+    input: {
+        console.log(function() {
+            1 + 1;
+        }.a = 1);
+    }
+    expect_exact: "console.log((function(){1+1}).a=1);"
+    expect_stdout: "1"
+}
diff --git a/test/jetstream.js b/test/jetstream.js
@@ -12,14 +12,14 @@ if (typeof phantom == "undefined") {
     });
     var args = process.argv.slice(2);
     if (!args.length) {
-        args.push("-mc");
+        args.push("-mcb", "beautify=false,webkit");
     }
     args.push("--timings");
     var child_process = require("child_process");
     try {
         require("phantomjs-prebuilt");
     } catch(e) {
-        child_process.execSync("npm install phantomjs-prebuilt@2.1.14");
+        child_process.execSync("npm install phantomjs-prebuilt@2.1.14 --no-save");
     }
     var http = require("http");
     var server = http.createServer(function(request, response) {

diff --git a/test/mocha/release.js b/test/mocha/release.js
@@ -48,6 +48,7 @@ describe("test/jetstream.js", function() {
         it("Should pass with options " + options, function(done) {
             var args = options.split(/ /);
             args.unshift("test/jetstream.js");
+            args.push("-b", "beautify=false,webkit");
             run(process.argv[0], args, done);
         });
     });