browsertime 27.4.0 → 27.4.1

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Browsertime changelog (we do [semantic versioning](https://semver.org))
 
+## 27.4.1 - 2026-05-25
+
+### Fixed
+* `--chrome.coverage` no longer reports "0 B unused" for functions that executed at least once. The JS used-bytes calculation took the union of every V8 range with count > 0, but V8 returns nested ranges — an outer range over the whole function body and inner ranges over sub-blocks. If a function ran at all, its outer range already covered the whole body and the inner count == 0 ranges (the actual dead branches) got absorbed into the union and contributed nothing. On modern bundles where module-evaluation top-level code runs for almost every function, the per-script unused-% collapsed to roughly zero across the board, leaving sitespeed.io rendering a "0 B unused" column on pages that obviously ship plenty of dead code. The calculation now walks the ranges as nested intervals, painting the innermost range's count at every byte (the same definition Chrome DevTools' Coverage panel uses), so dead branches inside executed functions show up correctly. CSS rule-usage tracking returns flat, non-overlapping ranges and is unchanged [#2484](https://github.com/sitespeedio/browsertime/pull/2484).
+
 ## 27.4.0 - 2026-05-24
 
 ### Added

package/lib/chrome/coverage.js

@@ -2,7 +2,8 @@ import { getLogger } from '@sitespeed.io/log';
 
 const log = getLogger('browsertime.chrome.coverage');
 
-// Sum the union length of half-open [start, end) ranges.
+// Sum the union length of half-open [start, end) ranges. Used for the
+// flat, non-nested ranges that CSS rule-usage tracking returns.
 function unionLength(ranges) {
   if (ranges.length === 0) return 0;
   const sorted = ranges
@@ -24,6 +25,45 @@ function unionLength(ranges) {
   return total;
 }
 
+// Count used bytes for a script under detailed (block-level) V8
+// coverage. The CDP returns nested ranges: the outermost range is the
+// function body and inner ranges are sub-blocks. An inner range with
+// count == 0 inside a containing range with count > 0 represents dead
+// code in an otherwise-executed function — those bytes must read as
+// unused. A naive union of every range whose count > 0 is wrong: the
+// outer range alone covers the entire function, so inner count == 0
+// ranges get masked and zero-count branches disappear, making typical
+// modern bundles look as if every byte was executed. Walk every range
+// across every function from outermost to innermost (start ascending,
+// length descending), painting a per-byte coverage flag; inner ranges
+// overwrite outer ones, so the final flag at each byte reflects the
+// innermost range's count.
+export function usedJsBytes(scriptCoverage, totalBytes) {
+  const ranges = [];
+  for (const fn of scriptCoverage.functions) {
+    for (const r of fn.ranges) {
+      if (r.endOffset > r.startOffset) ranges.push(r);
+    }
+  }
+  if (ranges.length === 0) return 0;
+  ranges.sort((a, b) =>
+    a.startOffset === b.startOffset
+      ? b.endOffset - a.endOffset
+      : a.startOffset - b.startOffset
+  );
+  const used = new Uint8Array(totalBytes);
+  for (const r of ranges) {
+    const start = Math.max(0, r.startOffset);
+    const end = Math.min(totalBytes, r.endOffset);
+    if (end > start) used.fill(r.count > 0 ? 1 : 0, start, end);
+  }
+  let count = 0;
+  for (let i = 0; i < totalBytes; i++) {
+    if (used[i] === 1) count++;
+  }
+  return count;
+}
+
 function summarize(files) {
   let totalBytes = 0;
   let usedBytes = 0;
@@ -143,13 +183,7 @@ export class Coverage {
       const totalBytes = source.length;
       if (totalBytes === 0) continue;
 
-      const usedRanges = [];
-      for (const fn of script.functions) {
-        for (const r of fn.ranges) {
-          if (r.count > 0) usedRanges.push(r);
-        }
-      }
-      const usedBytes = unionLength(usedRanges);
+      const usedBytes = usedJsBytes(script, totalBytes);
       const unusedBytes = totalBytes - usedBytes;
       files.push({
         url: script.url,

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "browsertime",
   "description": "Get performance metrics from your web page using Browsertime.",
-  "version": "27.4.0",
+  "version": "27.4.1",
   "bin": "./bin/browsertime.js",
   "type": "module",
   "types": "./scripting.d.ts",