Use object literal shorthand in js_optimizer. NFC

test/js_optimizer/JSDCE-defaultArg-output.js

@@ -2,7 +2,7 @@ var usedAsDefaultArg = 42;
 
 var usedAsDefaultArg2 = [ 1, 2 ];
 
-function g({notUsed: notUsed}, a, b = usedAsDefaultArg, [c, d] = usedAsDefaultArg2) {
+function g({notUsed}, a, b = usedAsDefaultArg, [c, d] = usedAsDefaultArg2) {
   return a + b + notUsed + 1;
 }
 

test/js_optimizer/JSDCE-objectPattern-output.js

@@ -3,7 +3,7 @@ let d = 40;
 let z = 50;
 
 globalThis.f = function([, r]) {
-  let {a: a, b: b} = r;
+  let {a, b} = r;
   let {z: c} = r;
   let [, i, {foo: p, bar: q}] = r;
   return g(a, b, c, d, z);

test/js_optimizer/applyImportAndExportNameChanges2-output.js

@@ -246,9 +246,9 @@ env["__memory_base"] = STATIC_BASE;
 env["__table_base"] = 0;
 
 var imports = {
-  env: env,
+  env,
   global: {
-    NaN: NaN,
+    NaN,
     Infinity: Infinity
   },
   "global.Math": Math,

test/js_optimizer/constructor-output.js

@@ -40,17 +40,17 @@ function u(a) {
     };
   }
   return m({
-    Int8Array: Int8Array,
-    Int16Array: Int16Array,
-    Int32Array: Int32Array,
-    Uint8Array: Uint8Array,
-    Uint16Array: Uint16Array,
-    Uint32Array: Uint32Array,
-    Float32Array: Float32Array,
-    Float64Array: Float64Array,
-    NaN: NaN,
+    Int8Array,
+    Int16Array,
+    Int32Array,
+    Uint8Array,
+    Uint16Array,
+    Uint32Array,
+    Float32Array,
+    Float64Array,
+    NaN,
     Infinity: Infinity,
-    Math: Math
+    Math
   }, q, r.buffer);
 }
 // EMSCRIPTEN_END_ASM

test/js_optimizer/test-object-literals-output.js

@@ -0,0 +1,14 @@
+var foo = 1;
+
+var baz = {};
+
+var bar = {
+  foo,
+  baz
+};
+
+var bar2 = {
+  foo,
+  baz,
+  bar: foo
+};

test/js_optimizer/test-object-literals.js

@@ -0,0 +1,15 @@
+var foo = 1;
+var baz = {};
+
+// Verify that shorthand object literals are preserved.
+var bar = {
+  foo,
+  baz,
+};
+
+// Objects that could use shorthand are also converted where possible.
+var bar2 = {
+  foo: foo,
+  "baz": baz,
+  bar: foo,
+};

test/js_optimizer/wasm2js-output.js

@@ -430,17 +430,17 @@ function X() {}
     };
   }
   return D({
-    Int8Array: Int8Array,
-    Int16Array: Int16Array,
-    Int32Array: Int32Array,
-    Uint8Array: Uint8Array,
-    Uint16Array: Uint16Array,
-    Uint32Array: Uint32Array,
-    Float32Array: Float32Array,
-    Float64Array: Float64Array,
-    NaN: NaN,
+    Int8Array,
+    Int16Array,
+    Int32Array,
+    Uint8Array,
+    Uint16Array,
+    Uint32Array,
+    Float32Array,
+    Float64Array,
+    NaN,
     Infinity: Infinity,
-    Math: Math
+    Math
   }, H, I.buffer);
 }
 // EMSCRIPTEN_END_ASM

test/test_other.py

@@ -2889,6 +2889,7 @@ def test_extern_prepost(self):
     'unsignPointers': ('test-unsignPointers.js', ['unsignPointers', '--closure-friendly']),
     'asanify': ('test-asanify.js', ['asanify']),
     'safeHeap': ('test-safeHeap.js', ['safeHeap']),
+    'object_literals': ('test-object-literals.js', []),
     'LittleEndianHeap': ('test-LittleEndianHeap.js', ['littleEndianHeap']),
   })
   @crossplatform

tools/acorn-optimizer.mjs

@@ -2107,6 +2107,7 @@ if (!noPrint) {
     keep_quoted_props: closureFriendly, // for closure
     wrap_func_args: false, // don't add extra braces
     comments: true, // for closure as well
+    shorthand: true, // Use object literal shorthand notation
   });
 
   output += '\n';