[test] Measure gzip sizes without metadata. NFC

RReverser · RReverser · commit b69fe9921e8a · 2025-07-09T02:11:14.000+01:00
Writing to a temporary gzip file, measuring and then deleting is both inefficient, but also results in a slightly larger reported size because it then also writes extra metadata like the original filename.

That metadata is not something we care about measuring, as more often than not compression on the Web happens on the fly anyway.
diff --git a/test/code_size/audio_worklet_wasm.json b/test/code_size/audio_worklet_wasm.json
@@ -1,10 +1,10 @@
 {
   "a.html": 519,
-  "a.html.gz": 364,
+  "a.html.gz": 357,
   "a.js": 3853,
-  "a.js.gz": 2050,
+  "a.js.gz": 2045,
   "a.wasm": 1288,
-  "a.wasm.gz": 867,
+  "a.wasm.gz": 860,
   "total": 5660,
-  "total_gz": 3281
+  "total_gz": 3262
 }
diff --git a/test/code_size/embind_hello_wasm.json b/test/code_size/embind_hello_wasm.json
@@ -1,10 +1,10 @@
 {
   "a.html": 552,
-  "a.html.gz": 380,
+  "a.html.gz": 373,
   "a.js": 7266,
-  "a.js.gz": 3321,
+  "a.js.gz": 3316,
   "a.wasm": 7294,
-  "a.wasm.gz": 3346,
+  "a.wasm.gz": 3339,
   "total": 15112,
-  "total_gz": 7047
+  "total_gz": 7028
 }
diff --git a/test/code_size/embind_val_wasm.json b/test/code_size/embind_val_wasm.json
@@ -1,10 +1,10 @@
 {
   "a.html": 552,
-  "a.html.gz": 380,
+  "a.html.gz": 373,
   "a.js": 5367,
-  "a.js.gz": 2540,
+  "a.js.gz": 2535,
   "a.wasm": 7468,
-  "a.wasm.gz": 3468,
+  "a.wasm.gz": 3461,
   "total": 13387,
-  "total_gz": 6388
+  "total_gz": 6369
 }
diff --git a/test/code_size/hello_wasm_worker_wasm.json b/test/code_size/hello_wasm_worker_wasm.json
@@ -1,10 +1,10 @@
 {
   "a.html": 519,
-  "a.html.gz": 364,
+  "a.html.gz": 357,
   "a.js": 830,
-  "a.js.gz": 530,
+  "a.js.gz": 525,
   "a.wasm": 1885,
-  "a.wasm.gz": 1079,
+  "a.wasm.gz": 1072,
   "total": 3234,
-  "total_gz": 1973
+  "total_gz": 1954
 }
diff --git a/test/code_size/hello_webgl2_wasm.json b/test/code_size/hello_webgl2_wasm.json
@@ -1,10 +1,10 @@
 {
   "a.html": 454,
-  "a.html.gz": 328,
+  "a.html.gz": 321,
   "a.js": 4386,
-  "a.js.gz": 2252,
+  "a.js.gz": 2247,
   "a.wasm": 8278,
-  "a.wasm.gz": 5627,
+  "a.wasm.gz": 5620,
   "total": 13118,
-  "total_gz": 8207
+  "total_gz": 8188
 }
diff --git a/test/code_size/hello_webgl2_wasm2js.json b/test/code_size/hello_webgl2_wasm2js.json
@@ -1,8 +1,8 @@
 {
   "a.html": 346,
-  "a.html.gz": 262,
+  "a.html.gz": 255,
   "a.js": 18103,
-  "a.js.gz": 9771,
+  "a.js.gz": 9766,
   "total": 18449,
-  "total_gz": 10033
+  "total_gz": 10021
 }
diff --git a/test/code_size/hello_webgl_wasm.json b/test/code_size/hello_webgl_wasm.json
@@ -1,10 +1,10 @@
 {
   "a.html": 454,
-  "a.html.gz": 328,
+  "a.html.gz": 321,
   "a.js": 3924,
-  "a.js.gz": 2092,
+  "a.js.gz": 2087,
   "a.wasm": 8278,
-  "a.wasm.gz": 5627,
+  "a.wasm.gz": 5620,
   "total": 12656,
-  "total_gz": 8047
+  "total_gz": 8028
 }
diff --git a/test/code_size/hello_webgl_wasm2js.json b/test/code_size/hello_webgl_wasm2js.json
@@ -1,8 +1,8 @@
 {
   "a.html": 346,
-  "a.html.gz": 262,
+  "a.html.gz": 255,
   "a.js": 17630,
-  "a.js.gz": 9603,
+  "a.js.gz": 9598,
   "total": 17976,
-  "total_gz": 9865
+  "total_gz": 9853
 }
diff --git a/test/code_size/hello_world_wasm.json b/test/code_size/hello_world_wasm.json
@@ -1,10 +1,10 @@
 {
   "a.html": 552,
-  "a.html.gz": 380,
+  "a.html.gz": 373,
   "a.js": 287,
-  "a.js.gz": 247,
+  "a.js.gz": 242,
   "a.wasm": 95,
-  "a.wasm.gz": 108,
+  "a.wasm.gz": 101,
   "total": 934,
-  "total_gz": 735
+  "total_gz": 716
 }
diff --git a/test/code_size/hello_world_wasm2js.json b/test/code_size/hello_world_wasm2js.json
@@ -1,8 +1,8 @@
 {
   "a.html": 323,
-  "a.html.gz": 253,
+  "a.html.gz": 246,
   "a.js": 965,
-  "a.js.gz": 593,
+  "a.js.gz": 588,
   "total": 1288,
-  "total_gz": 846
+  "total_gz": 834
 }
diff --git a/test/code_size/math_wasm.json b/test/code_size/math_wasm.json
@@ -1,10 +1,10 @@
 {
   "a.html": 552,
-  "a.html.gz": 380,
+  "a.html.gz": 373,
   "a.js": 110,
-  "a.js.gz": 125,
+  "a.js.gz": 120,
   "a.wasm": 2678,
-  "a.wasm.gz": 1655,
+  "a.wasm.gz": 1648,
   "total": 3340,
-  "total_gz": 2160
+  "total_gz": 2141
 }
diff --git a/test/code_size/random_printf_wasm.json b/test/code_size/random_printf_wasm.json
@@ -1,6 +1,6 @@
 {
   "a.html": 12511,
-  "a.html.gz": 6855,
+  "a.html.gz": 6848,
   "total": 12511,
-  "total_gz": 6855
+  "total_gz": 6848
 }
diff --git a/test/code_size/random_printf_wasm2js.json b/test/code_size/random_printf_wasm2js.json
@@ -1,6 +1,6 @@
 {
   "a.html": 17224,
-  "a.html.gz": 7558,
+  "a.html.gz": 7551,
   "total": 17224,
-  "total_gz": 7558
+  "total_gz": 7551
 }
diff --git a/test/other/codesize/test_codesize_cxx_ctors1.gzsize b/test/other/codesize/test_codesize_cxx_ctors1.gzsize
@@ -1 +1 @@
-8171
+8162
diff --git a/test/other/codesize/test_codesize_cxx_ctors2.gzsize b/test/other/codesize/test_codesize_cxx_ctors2.gzsize
@@ -1 +1 @@
-8160
+8151
diff --git a/test/other/codesize/test_codesize_cxx_except.gzsize b/test/other/codesize/test_codesize_cxx_except.gzsize
@@ -1 +1 @@
-9159
+9150
diff --git a/test/other/codesize/test_codesize_cxx_except_wasm.gzsize b/test/other/codesize/test_codesize_cxx_except_wasm.gzsize
@@ -1 +1 @@
-8119
+8110
diff --git a/test/other/codesize/test_codesize_cxx_except_wasm_legacy.gzsize b/test/other/codesize/test_codesize_cxx_except_wasm_legacy.gzsize
@@ -1 +1 @@
-8119
+8110
diff --git a/test/other/codesize/test_codesize_cxx_lto.gzsize b/test/other/codesize/test_codesize_cxx_lto.gzsize
@@ -1 +1 @@
-8023
+8014
diff --git a/test/other/codesize/test_codesize_cxx_mangle.gzsize b/test/other/codesize/test_codesize_cxx_mangle.gzsize
@@ -1 +1 @@
-9176
+9167
diff --git a/test/other/codesize/test_codesize_cxx_noexcept.gzsize b/test/other/codesize/test_codesize_cxx_noexcept.gzsize
@@ -1 +1 @@
-8171
+8162
diff --git a/test/other/codesize/test_codesize_cxx_wasmfs.gzsize b/test/other/codesize/test_codesize_cxx_wasmfs.gzsize
@@ -1 +1 @@
-3355
+3346
diff --git a/test/other/codesize/test_codesize_files_js_fs.gzsize b/test/other/codesize/test_codesize_files_js_fs.gzsize
@@ -1 +1 @@
-7485
+7476
diff --git a/test/other/codesize/test_codesize_files_wasmfs.gzsize b/test/other/codesize/test_codesize_files_wasmfs.gzsize
@@ -1 +1 @@
-2606
+2597
diff --git a/test/other/codesize/test_codesize_hello_O0.gzsize b/test/other/codesize/test_codesize_hello_O0.gzsize
@@ -1 +1 @@
-8319
+8310
diff --git a/test/other/codesize/test_codesize_hello_O1.gzsize b/test/other/codesize/test_codesize_hello_O1.gzsize
@@ -1 +1 @@
-2496
+2487
diff --git a/test/other/codesize/test_codesize_hello_O2.gzsize b/test/other/codesize/test_codesize_hello_O2.gzsize
@@ -1 +1 @@
-2157
+2148
diff --git a/test/other/codesize/test_codesize_hello_O3.gzsize b/test/other/codesize/test_codesize_hello_O3.gzsize
@@ -1 +1 @@
-2119
+2110
diff --git a/test/other/codesize/test_codesize_hello_Os.gzsize b/test/other/codesize/test_codesize_hello_Os.gzsize
@@ -1 +1 @@
-2119
+2110
diff --git a/test/other/codesize/test_codesize_hello_Oz.gzsize b/test/other/codesize/test_codesize_hello_Oz.gzsize
@@ -1 +1 @@
-1921
+1912
diff --git a/test/other/codesize/test_codesize_hello_dylink.gzsize b/test/other/codesize/test_codesize_hello_dylink.gzsize
@@ -1 +1 @@
-11482
+11473
diff --git a/test/other/codesize/test_codesize_hello_export_nothing.gzsize b/test/other/codesize/test_codesize_hello_export_nothing.gzsize
@@ -1 +1 @@
-1508
+1499
diff --git a/test/other/codesize/test_codesize_hello_single_file.gzsize b/test/other/codesize/test_codesize_hello_single_file.gzsize
@@ -1 +1 @@
-3632
+3623
diff --git a/test/other/codesize/test_codesize_hello_wasmfs.gzsize b/test/other/codesize/test_codesize_hello_wasmfs.gzsize
@@ -1 +1 @@
-2119
+2110
diff --git a/test/other/codesize/test_codesize_libcxxabi_message_O3.gzsize b/test/other/codesize/test_codesize_libcxxabi_message_O3.gzsize
@@ -1 +1 @@
-1696
+1687
diff --git a/test/other/codesize/test_codesize_libcxxabi_message_O3_standalone.gzsize b/test/other/codesize/test_codesize_libcxxabi_message_O3_standalone.gzsize
@@ -1 +1 @@
-1727
+1718
diff --git a/test/other/codesize/test_codesize_mem_O3.gzsize b/test/other/codesize/test_codesize_mem_O3.gzsize
@@ -1 +1 @@
-2134
+2125
diff --git a/test/other/codesize/test_codesize_mem_O3_grow.gzsize b/test/other/codesize/test_codesize_mem_O3_grow.gzsize
@@ -1 +1 @@
-2283
+2274
diff --git a/test/other/codesize/test_codesize_mem_O3_grow_standalone.gzsize b/test/other/codesize/test_codesize_mem_O3_grow_standalone.gzsize
@@ -1 +1 @@
-1997
+1988
diff --git a/test/other/codesize/test_codesize_mem_O3_standalone.gzsize b/test/other/codesize/test_codesize_mem_O3_standalone.gzsize
@@ -1 +1 @@
-1963
+1954
diff --git a/test/other/codesize/test_codesize_mem_O3_standalone_lib.gzsize b/test/other/codesize/test_codesize_mem_O3_standalone_lib.gzsize
@@ -1 +1 @@
-1719
+1710
diff --git a/test/other/codesize/test_codesize_mem_O3_standalone_narg.gzsize b/test/other/codesize/test_codesize_mem_O3_standalone_narg.gzsize
@@ -1 +1 @@
-1727
+1718
diff --git a/test/other/codesize/test_codesize_mem_O3_standalone_narg_flto.gzsize b/test/other/codesize/test_codesize_mem_O3_standalone_narg_flto.gzsize
@@ -1 +1 @@
-1727
+1718
diff --git a/test/other/codesize/test_codesize_minimal_64.gzsize b/test/other/codesize/test_codesize_minimal_64.gzsize
@@ -1 +1 @@
-1267
+1258
diff --git a/test/other/codesize/test_codesize_minimal_O0.gzsize b/test/other/codesize/test_codesize_minimal_O0.gzsize
@@ -1 +1 @@
-6614
+6605
diff --git a/test/other/codesize/test_codesize_minimal_O1.gzsize b/test/other/codesize/test_codesize_minimal_O1.gzsize
@@ -1 +1 @@
-1329
+1320
diff --git a/test/other/codesize/test_codesize_minimal_O2.gzsize b/test/other/codesize/test_codesize_minimal_O2.gzsize
@@ -1 +1 @@
-1197
+1188
diff --git a/test/other/codesize/test_codesize_minimal_O3.gzsize b/test/other/codesize/test_codesize_minimal_O3.gzsize
@@ -1 +1 @@
-1163
+1154
diff --git a/test/other/codesize/test_codesize_minimal_Os.gzsize b/test/other/codesize/test_codesize_minimal_Os.gzsize
@@ -1 +1 @@
-1163
+1154
diff --git a/test/other/codesize/test_codesize_minimal_Os_mr.gzsize b/test/other/codesize/test_codesize_minimal_Os_mr.gzsize
@@ -1 +1 @@
-307
+298
diff --git a/test/other/codesize/test_codesize_minimal_Oz-ctors.gzsize b/test/other/codesize/test_codesize_minimal_Oz-ctors.gzsize
@@ -1 +1 @@
-1147
+1138
diff --git a/test/other/codesize/test_codesize_minimal_Oz.gzsize b/test/other/codesize/test_codesize_minimal_Oz.gzsize
@@ -1 +1 @@
-1163
+1154
diff --git a/test/other/codesize/test_codesize_minimal_esm.gzsize b/test/other/codesize/test_codesize_minimal_esm.gzsize
@@ -1 +1 @@
-1249
+1240
diff --git a/test/other/codesize/test_codesize_minimal_pthreads.gzsize b/test/other/codesize/test_codesize_minimal_pthreads.gzsize
@@ -1 +1 @@
-3744
+3735
diff --git a/test/other/codesize/test_codesize_minimal_pthreads_memgrowth.gzsize b/test/other/codesize/test_codesize_minimal_pthreads_memgrowth.gzsize
@@ -1 +1 @@
-3950
+3941
diff --git a/test/other/codesize/test_codesize_minimal_wasmfs.gzsize b/test/other/codesize/test_codesize_minimal_wasmfs.gzsize
@@ -1 +1 @@
-1163
+1154
diff --git a/test/test_other.py b/test/test_other.py
@@ -226,13 +226,10 @@ def llvm_nm(file):
   return symbols
 
 
-def get_file_gzipped_size(f):
-  f_gz = f + '.gz'
-  with gzip.open(f_gz, 'wb') as gzf:
-    gzf.write(read_binary(f))
-  size = os.path.getsize(f_gz)
-  delete_file(f_gz)
-  return size
+def get_file_and_gz_sizes(f):
+  contents = read_binary(f)
+  gz_contents = gzip.compress(contents)
+  return len(contents), len(gz_contents)
 
 
 def deminify_syms(names, minification_map):
@@ -9322,8 +9319,7 @@ def run_codesize_test(self, filename, cflags, check_funcs=True, check_full_js=Fa
 
     self.run_process(build_cmd + ['--profiling-funcs', '--closure=1'])
 
-    js_size = os.path.getsize('a.out.js')
-    gz_size = get_file_gzipped_size('a.out.js')
+    js_size, gz_size = get_file_and_gz_sizes('a.out.js')
     js_size_file = expected_basename + '.jssize'
     gz_size_file = expected_basename + '.gzsize'
     self.check_expected_size_in_file('js', js_size_file, js_size)
@@ -11962,8 +11958,7 @@ def print_percent(actual, expected):
       f_gz = f + '.gz'
       expected_size = expected_results[f] if f in expected_results else float('inf')
       expected_size_gz = expected_results[f_gz] if f_gz in expected_results else float('inf')
-      size = os.path.getsize(f)
-      size_gz = get_file_gzipped_size(f)
+      size, size_gz = get_file_and_gz_sizes(f)
 
       obtained_results[f] = size
       obtained_results[f_gz] = size_gz

Original file line number	Diff line number	Diff line change
`@@ -1,8 +1,8 @@`
`1`	`1`	`{`
`2`	`2`	`"a.html": 346,`
`3`		`- "a.html.gz": 262,`
	`3`	`+ "a.html.gz": 255,`
`4`	`4`	`"a.js": 18103,`
`5`		`- "a.js.gz": 9771,`
	`5`	`+ "a.js.gz": 9766,`
`6`	`6`	`"total": 18449,`
`7`		`- "total_gz": 10033`
	`7`	`+ "total_gz": 10021`
`8`	`8`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,8 +1,8 @@`
`1`	`1`	`{`
`2`	`2`	`"a.html": 323,`
`3`		`- "a.html.gz": 253,`
	`3`	`+ "a.html.gz": 246,`
`4`	`4`	`"a.js": 965,`
`5`		`- "a.js.gz": 593,`
	`5`	`+ "a.js.gz": 588,`
`6`	`6`	`"total": 1288,`
`7`		`- "total_gz": 846`
	`7`	`+ "total_gz": 834`
`8`	`8`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"a.html": 12511,`
`3`		`- "a.html.gz": 6855,`
	`3`	`+ "a.html.gz": 6848,`
`4`	`4`	`"total": 12511,`
`5`		`- "total_gz": 6855`
	`5`	`+ "total_gz": 6848`
`6`	`6`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"a.html": 17224,`
`3`		`- "a.html.gz": 7558,`
	`3`	`+ "a.html.gz": 7551,`
`4`	`4`	`"total": 17224,`
`5`		`- "total_gz": 7558`
	`5`	`+ "total_gz": 7551`
`6`	`6`	`}`