lighthouse 12.4.0-dev.20250317 → 12.4.0-dev.20250319

package/core/audits/audit.js

@@ -491,7 +491,8 @@ class Audit {
     const trace = artifacts.traces[Audit.DEFAULT_PASS];
     const devtoolsLog = artifacts.devtoolsLogs[Audit.DEFAULT_PASS];
     const gatherContext = artifacts.GatherContext;
-    return {trace, devtoolsLog, gatherContext, settings: context.settings, URL: artifacts.URL};
+    const {URL, SourceMaps} = artifacts;
+    return {trace, devtoolsLog, gatherContext, settings: context.settings, URL, SourceMaps};
   }
 }
 

package/core/audits/bootup-time.js

@@ -49,7 +49,7 @@ class BootupTime extends Audit {
       description: str_(UIStrings.description),
       scoreDisplayMode: Audit.SCORING_MODES.METRIC_SAVINGS,
       guidanceLevel: 1,
-      requiredArtifacts: ['traces', 'devtoolsLogs', 'URL', 'GatherContext'],
+      requiredArtifacts: ['traces', 'devtoolsLogs', 'URL', 'GatherContext', 'SourceMaps'],
     };
   }
 

package/core/audits/byte-efficiency/efficient-animated-content.js

@@ -38,7 +38,7 @@ class EfficientAnimatedContent extends ByteEfficiencyAudit {
       description: str_(UIStrings.description),
       scoreDisplayMode: ByteEfficiencyAudit.SCORING_MODES.METRIC_SAVINGS,
       guidanceLevel: 3,
-      requiredArtifacts: ['devtoolsLogs', 'traces', 'GatherContext', 'URL'],
+      requiredArtifacts: ['devtoolsLogs', 'traces', 'GatherContext', 'URL', 'SourceMaps'],
     };
   }
 

package/core/audits/byte-efficiency/legacy-javascript.d.ts

@@ -1,14 +1,4 @@
 export default LegacyJavascript;
-export type Pattern = {
-    name: string;
-    expression: string;
-    estimateBytes?: (content: string) => number;
-};
-export type PatternMatchResult = {
-    name: string;
-    line: number;
-    column: number;
-};
 export type ByteEfficiencyProduct = import("./byte-efficiency-audit.js").ByteEfficiencyProduct;
 export type Item = LH.Audit.ByteEfficiencyItem & {
     subItems: {
@@ -21,40 +11,6 @@ export type SubItem = {
     location: LH.Audit.Details.SourceLocationValue;
 };
 declare class LegacyJavascript extends ByteEfficiencyAudit {
-    /**
-     * @param {string?} object
-     * @param {string} property
-     * @param {string} coreJs3Module
-     */
-    static buildPolyfillExpression(object: string | null, property: string, coreJs3Module: string): string;
-    static getPolyfillModuleData(): import("../../scripts/legacy-javascript/create-polyfill-module-data.js").PolyfillModuleData;
-    static getCoreJsPolyfillData(): {
-        name: string;
-        coreJs3Module: string;
-    }[];
-    /**
-     * @return {Pattern[]}
-     */
-    static getPolyfillPatterns(): Pattern[];
-    /**
-     * @return {Pattern[]}
-     */
-    static getTransformPatterns(): Pattern[];
-    /**
-     * Returns a collection of match results grouped by script url.
-     *
-     * @param {CodePatternMatcher} matcher
-     * @param {LH.Artifacts['Scripts']} scripts
-     * @param {LH.Artifacts.Bundle[]} bundles
-     * @return {Map<LH.Artifacts.Script, PatternMatchResult[]>}
-     */
-    static detectAcrossScripts(matcher: CodePatternMatcher, scripts: LH.Artifacts["Scripts"], bundles: LH.Artifacts.Bundle[]): Map<LH.Artifacts.Script, PatternMatchResult[]>;
-    /**
-     * @param {LH.Artifacts.Script} script
-     * @param {PatternMatchResult[]} matches
-     * @return {number}
-     */
-    static estimateWastedBytes(script: LH.Artifacts.Script, matches: PatternMatchResult[]): number;
     /**
      * @param {LH.Artifacts} artifacts
      * @param {Array<LH.Artifacts.NetworkRequest>} networkRecords
@@ -69,22 +25,4 @@ export namespace UIStrings {
     let detectedCoreJs2Warning: string;
 }
 import { ByteEfficiencyAudit } from './byte-efficiency-audit.js';
-/**
- * Takes a list of patterns (consisting of a name identifier and a RegExp expression string)
- * and via `match` returns match results with line / column information for a given code input.
- * Only returns the first match per pattern given.
- */
-declare class CodePatternMatcher {
-    /**
-     * @param {Pattern[]} patterns
-     */
-    constructor(patterns: Pattern[]);
-    re: RegExp;
-    patterns: Pattern[];
-    /**
-     * @param {string} code
-     * @return {PatternMatchResult[]}
-     */
-    match(code: string): PatternMatchResult[];
-}
 //# sourceMappingURL=legacy-javascript.d.ts.map

package/core/audits/byte-efficiency/legacy-javascript.js

@@ -11,33 +11,17 @@
  * ./core/scripts/legacy-javascript - verification tool.
  */
 
-/** @typedef {{name: string, expression: string, estimateBytes?: (content: string) => number}} Pattern */
-/** @typedef {{name: string, line: number, column: number}} PatternMatchResult */
 /** @typedef {import('./byte-efficiency-audit.js').ByteEfficiencyProduct} ByteEfficiencyProduct */
 /** @typedef {LH.Audit.ByteEfficiencyItem & {subItems: {type: 'subitems', items: SubItem[]}}} Item */
 /** @typedef {{signal: string, location: LH.Audit.Details.SourceLocationValue}} SubItem */
 
-import fs from 'fs';
-
 import {Audit} from '../audit.js';
 import {ByteEfficiencyAudit} from './byte-efficiency-audit.js';
 import {EntityClassification} from '../../computed/entity-classification.js';
 import {JSBundles} from '../../computed/js-bundles.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 import {estimateCompressionRatioForContent} from '../../lib/script-helpers.js';
-import {LH_ROOT} from '../../../shared/root.js';
-
-const polyfillModuleDataJson = fs.readFileSync(
-  `${LH_ROOT}/core/audits/byte-efficiency/polyfill-module-data.json`, 'utf-8');
-
-/** @type {import('../../scripts/legacy-javascript/create-polyfill-module-data.js').PolyfillModuleData} */
-const polyfillModuleData = JSON.parse(polyfillModuleDataJson);
-
-const graphJson = fs.readFileSync(
-  `${LH_ROOT}/core/audits/byte-efficiency/polyfill-graph-data.json`, 'utf-8');
-
-/** @type {import('../../scripts/legacy-javascript/create-polyfill-size-estimation.js').PolyfillSizeEstimator} */
-const graph = JSON.parse(graphJson);
+import {detectLegacyJavaScript} from '../../lib/legacy-javascript/legacy-javascript.js';
 
 const UIStrings = {
   /** Title of a Lighthouse audit that tells the user about legacy polyfills and transforms used on the page. This is displayed in a list of audit titles that Lighthouse generates. */
@@ -53,68 +37,6 @@ const UIStrings = {
 
 const str_ = i18n.createIcuMessageFn(import.meta.url, UIStrings);
 
-/**
- * Takes a list of patterns (consisting of a name identifier and a RegExp expression string)
- * and via `match` returns match results with line / column information for a given code input.
- * Only returns the first match per pattern given.
- */
-class CodePatternMatcher {
-  /**
-   * @param {Pattern[]} patterns
-   */
-  constructor(patterns) {
-    const patternsExpression = patterns.map(pattern => `(${pattern.expression})`).join('|');
-    this.re = new RegExp(`(^\r\n|\r|\n)|${patternsExpression}`, 'g');
-    this.patterns = patterns;
-  }
-
-  /**
-   * @param {string} code
-   * @return {PatternMatchResult[]}
-   */
-  match(code) {
-    // Reset RegExp state.
-    this.re.lastIndex = 0;
-
-    const seen = new Set();
-    /** @type {PatternMatchResult[]} */
-    const matches = [];
-    /** @type {RegExpExecArray | null} */
-    let result;
-    let line = 0;
-    let lineBeginsAtIndex = 0;
-    // Each pattern maps to one subgroup in the generated regex. For each iteration of RegExp.exec,
-    // only one subgroup will be defined. Exec until no more matches.
-    while ((result = this.re.exec(code)) !== null) {
-      // Discard first value in `result` - it's just the entire match.
-      const captureGroups = result.slice(1);
-      // isNewline - truthy if matching a newline, used to track the line number.
-      // `patternExpressionMatches` maps to each possible pattern in `this.patterns`.
-      // Only one of [isNewline, ...patternExpressionMatches] is ever truthy.
-      const [isNewline, ...patternExpressionMatches] = captureGroups;
-      if (isNewline) {
-        line++;
-        lineBeginsAtIndex = result.index + 1;
-        continue;
-      }
-      const pattern = this.patterns[patternExpressionMatches.findIndex(Boolean)];
-
-      if (seen.has(pattern)) {
-        continue;
-      }
-      seen.add(pattern);
-
-      matches.push({
-        name: pattern.name,
-        line,
-        column: result.index - lineBeginsAtIndex,
-      });
-    }
-
-    return matches;
-  }
-}
-
 class LegacyJavascript extends ByteEfficiencyAudit {
   /**
    * @return {LH.Audit.Meta}
@@ -131,271 +53,6 @@ class LegacyJavascript extends ByteEfficiencyAudit {
     };
   }
 
-  /**
-   * @param {string?} object
-   * @param {string} property
-   * @param {string} coreJs3Module
-   */
-  static buildPolyfillExpression(object, property, coreJs3Module) {
-    const qt = (/** @type {string} */ token) =>
-      `['"]${token}['"]`; // don't worry about matching string delims
-    const kebabCaseToCamelCase = (/** @type {string} */ str) =>
-      str.replace(/(-\w)/g, m => m[1].toUpperCase());
-
-    let expression = '';
-
-    if (object) {
-      // String.prototype.startsWith =
-      expression += `${object}\\.${property}\\s?=[^=]`;
-    } else {
-      // Promise =
-      // window.Promise =// Promise =Z
-      // but not: SomePromise =
-      expression += `(?:window\\.|[\\s;]+)${property}\\s?=[^=]`;
-    }
-
-    // String.prototype['startsWith'] =
-    if (object) {
-      expression += `|${object}\\[${qt(property)}\\]\\s?=[^=]`;
-    }
-
-    // Object.defineProperty(String.prototype, 'startsWith'
-    expression += `|defineProperty\\(${object || 'window'},\\s?${qt(property)}`;
-
-    // es-shims
-    // no(Object,{entries:r},{entries:function
-    if (object) {
-      expression += `|\\(${object},\\s*{${property}:.*},\\s*{${property}`;
-    }
-
-    // core-js
-    if (object) {
-      const objectWithoutPrototype = object.replace('.prototype', '');
-      // e(e.S,"Object",{values
-      // Minified + mangled pattern found in CDN babel-polyfill.
-      // see https://cdnjs.cloudflare.com/ajax/libs/babel-polyfill/7.2.5/polyfill.min.js
-      // TODO: perhaps this is the wrong place to check for a CDN polyfill. Remove?
-      // expression += `|;e\\([^,]+,${qt(objectWithoutPrototype)},{${property}:`;
-
-      // core-js@3 minified pattern.
-      // {target:"Array",proto:true},{fill:fill
-      // {target:"Array",proto:true,forced:!HAS_SPECIES_SUPPORT||!USES_TO_LENGTH},{filter:
-      expression += `|{target:${qt(objectWithoutPrototype)}[^;]*},{${property}:`;
-    } else {
-      // Detect polyfills for new classes: Map, Set, WeakSet, etc.
-      // TODO: so far, no class polyfills are enabled for detection.
-      // See `modulesToSkip` in create-polyfill-module-data.js
-
-      // collection("Map",
-      // expression += `|collection\\(${qt(property)},`;
-    }
-
-    // Un-minified code may have module names.
-    // core-js/modules/es.object.is-frozen
-    expression += `|core-js/modules/${coreJs3Module}(?:\\.js)?"`;
-    // rollup unminified output for commonjs modules.
-    // ex: es.reflect.own-keys -> function requireEs_reflect_ownKeys ()
-    let rollupSlug = kebabCaseToCamelCase(coreJs3Module).replaceAll('.', '_');
-    rollupSlug = rollupSlug[0].toUpperCase() + rollupSlug.slice(1);
-    expression += `|require${rollupSlug} \\(`;
-
-    return expression;
-  }
-
-  static getPolyfillModuleData() {
-    return polyfillModuleData;
-  }
-
-  static getCoreJsPolyfillData() {
-    return this.getPolyfillModuleData().filter(d => d.corejs).map(d => {
-      return {
-        name: d.name,
-        coreJs3Module: d.modules[0],
-      };
-    });
-  }
-
-  /**
-   * @return {Pattern[]}
-   */
-  static getPolyfillPatterns() {
-    /** @type {Pattern[]} */
-    const patterns = [];
-
-    for (const {name, coreJs3Module} of this.getCoreJsPolyfillData()) {
-      const parts = name.split('.');
-      const object = parts.length > 1 ? parts.slice(0, parts.length - 1).join('.') : null;
-      const property = parts[parts.length - 1];
-      patterns.push({
-        name,
-        expression: this.buildPolyfillExpression(object, property, coreJs3Module),
-      });
-    }
-
-    return patterns;
-  }
-
-  /**
-   * @return {Pattern[]}
-   */
-  static getTransformPatterns() {
-    /**
-     * @param {string} content
-     * @param {RegExp|string} pattern
-     * @return {number}
-     */
-    const count = (content, pattern) => {
-      // Split is slightly faster than match.
-      if (typeof pattern === 'string') {
-        return content.split(pattern).length - 1;
-      }
-
-      return (content.match(pattern) ?? []).length;
-    };
-
-    // For expression: prefer a string that is found in the transform runtime support code (those won't ever be minified).
-
-    return [
-      // @babel/plugin-transform-classes
-      //
-      // input:
-      //
-      // class MyTestClass {
-      //   log() {
-      //     console.log(1);
-      //   }
-      // };
-      //
-      // output:
-      //
-      // function _classCallCheck(a, n) { if (!(a instanceof n)) throw new TypeError("Cannot call a class as a function"); }
-      // function _defineProperties(e, r) { for (var t = 0; t < r.length; t++) { var o = r[t]; o.enumerable = o.enumerable || !1, o.configurable = !0, "value" in o && (o.writable = !0), Object.defineProperty(e, _toPropertyKey(o.key), o); } }
-      // function _createClass(e, r, t) { return r && _defineProperties(e.prototype, r), t && _defineProperties(e, t), Object.defineProperty(e, "prototype", { writable: !1 }), e; }
-      // function _toPropertyKey(t) { var i = _toPrimitive(t, "string"); return "symbol" == typeof i ? i : i + ""; }
-      // function _toPrimitive(t, r) { if ("object" != typeof t || !t) return t; var e = t[Symbol.toPrimitive]; if (void 0 !== e) { var i = e.call(t, r || "default"); if ("object" != typeof i) return i; throw new TypeError("@@toPrimitive must return a primitive value."); } return ("string" === r ? String : Number)(t); }
-      // let MyTestClass = function () {
-      //   function MyTestClass() {
-      //     _classCallCheck(this, MyTestClass);
-      //   }
-      //   return _createClass(MyTestClass, [{
-      //     key: "log",
-      //     value: function log() {
-      //       console.log(1);
-      //     }
-      //   }]);
-      // }();
-      {
-        name: '@babel/plugin-transform-classes',
-        expression: 'Cannot call a class as a function',
-        estimateBytes: content => {
-          return 1000 + (count(content, '_classCallCheck') - 1) * '_classCallCheck()'.length;
-        },
-      },
-      {
-        name: '@babel/plugin-transform-regenerator',
-        expression: 'Generator is already running|regeneratorRuntime',
-        // Example of this transform: https://gist.github.com/connorjclark/af8bccfff377ac44efc104a79bc75da2
-        // `regeneratorRuntime.awrap` is generated for every usage of `await`, and adds ~80 bytes each.
-        estimateBytes: content => {
-          return count(content, /regeneratorRuntime\(?\)?\.a?wrap/g) * 80;
-        },
-      },
-      {
-        name: '@babel/plugin-transform-spread',
-        expression: 'Invalid attempt to spread non-iterable instance',
-        estimateBytes: content => {
-          const per = '_toConsumableArray()'.length;
-          return 1169 + count(content, /\.apply\(void 0,\s?_toConsumableArray/g) * per;
-        },
-      },
-    ];
-  }
-
-  /**
-   * Returns a collection of match results grouped by script url.
-   *
-   * @param {CodePatternMatcher} matcher
-   * @param {LH.Artifacts['Scripts']} scripts
-   * @param {LH.Artifacts.Bundle[]} bundles
-   * @return {Map<LH.Artifacts.Script, PatternMatchResult[]>}
-   */
-  static detectAcrossScripts(matcher, scripts, bundles) {
-    /** @type {Map<LH.Artifacts.Script, PatternMatchResult[]>} */
-    const scriptToMatchResults = new Map();
-    const polyfillData = this.getPolyfillModuleData();
-
-    for (const script of Object.values(scripts)) {
-      if (!script.content) continue;
-
-      // Start with pattern matching against the downloaded script.
-      const matches = matcher.match(script.content);
-
-      // If it's a bundle with source maps, add in the polyfill modules by name too.
-      const bundle = bundles.find(b => b.script.scriptId === script.scriptId);
-      if (bundle) {
-        for (const {name, modules} of polyfillData) {
-          // Skip if the pattern matching found a match for this polyfill.
-          if (matches.some(m => m.name === name)) continue;
-
-          const source = bundle.rawMap.sources.find(source => modules.some(module => {
-            return source.endsWith(`/${module}.js`) || source.includes(`node_modules/${module}/`);
-          }));
-          if (!source) continue;
-
-          const mapping = bundle.map.mappings().find(m => m.sourceURL === source);
-          if (mapping) {
-            matches.push({name, line: mapping.lineNumber, column: mapping.columnNumber});
-          } else {
-            matches.push({name, line: 0, column: 0});
-          }
-        }
-      }
-
-      if (!matches.length) continue;
-      scriptToMatchResults.set(script, matches);
-    }
-
-    return scriptToMatchResults;
-  }
-
-  /**
-   * @param {LH.Artifacts.Script} script
-   * @param {PatternMatchResult[]} matches
-   * @return {number}
-   */
-  static estimateWastedBytes(script, matches) {
-    // Split up results based on polyfill / transform. Only transforms start with @.
-    const polyfillResults = matches.filter(m => !m.name.startsWith('@'));
-    const transformResults = matches.filter(m => m.name.startsWith('@'));
-
-    let estimatedWastedBytesFromPolyfills = 0;
-    const modulesSeen = new Set();
-    for (const result of polyfillResults) {
-      const modules = graph.dependencies[result.name];
-      if (!modules) continue; // Shouldn't happen.
-      for (const module of modules) {
-        modulesSeen.add(module);
-      }
-    }
-
-    estimatedWastedBytesFromPolyfills += [...modulesSeen].reduce((acc, moduleIndex) => {
-      return acc + graph.moduleSizes[moduleIndex];
-    }, 0);
-    estimatedWastedBytesFromPolyfills = Math.min(estimatedWastedBytesFromPolyfills, graph.maxSize);
-
-    let estimatedWastedBytesFromTransforms = 0;
-
-    for (const result of transformResults) {
-      const pattern = this.getTransformPatterns().find(p => p.name === result.name);
-      if (!pattern || !pattern.estimateBytes || !script.content) continue;
-      estimatedWastedBytesFromTransforms += pattern.estimateBytes(script.content);
-    }
-
-    const estimatedWastedBytes =
-      estimatedWastedBytesFromPolyfills + estimatedWastedBytesFromTransforms;
-    return estimatedWastedBytes;
-  }
-
   /**
    * @param {LH.Artifacts} artifacts
    * @param {Array<LH.Artifacts.NetworkRequest>} networkRecords
@@ -412,20 +69,18 @@ class LegacyJavascript extends ByteEfficiencyAudit {
     /** @type {Item[]} */
     const items = [];
 
-    const matcher = new CodePatternMatcher([
-      ...this.getPolyfillPatterns(),
-      ...this.getTransformPatterns(),
-    ]);
-
     /** @type {Map<string, number>} */
     const compressionRatioByUrl = new Map();
 
-    const scriptToMatchResults =
-      this.detectAcrossScripts(matcher, artifacts.Scripts, bundles);
-    for (const [script, matches] of scriptToMatchResults.entries()) {
+    for (const script of artifacts.Scripts) {
+      const bundle = bundles.find(bundle => bundle.script.scriptId === script.scriptId);
+      const {matches, estimatedByteSavings} =
+        detectLegacyJavaScript(script.content ?? '', bundle?.map ?? null);
+      if (matches.length === 0) continue;
+
       const compressionRatio = estimateCompressionRatioForContent(
         compressionRatioByUrl, script.url, artifacts, networkRecords);
-      const wastedBytes = Math.round(this.estimateWastedBytes(script, matches) * compressionRatio);
+      const wastedBytes = Math.round(estimatedByteSavings * compressionRatio);
       /** @type {typeof items[number]} */
       const item = {
         url: script.url,
@@ -438,10 +93,7 @@ class LegacyJavascript extends ByteEfficiencyAudit {
         totalBytes: 0,
       };
 
-      const bundle = bundles.find(bundle => bundle.script.scriptId === script.scriptId);
-      const matchesSorted =
-        matches.sort((a, b) => a.name > b.name ? 1 : a.name === b.name ? 0 : -1);
-      for (const match of matchesSorted) {
+      for (const match of matches) {
         const {name, line, column} = match;
         /** @type {SubItem} */
         const subItem = {

package/core/audits/byte-efficiency/modern-image-formats.js

@@ -37,7 +37,7 @@ class ModernImageFormats extends ByteEfficiencyAudit {
       scoreDisplayMode: ByteEfficiencyAudit.SCORING_MODES.METRIC_SAVINGS,
       guidanceLevel: 3,
       requiredArtifacts: ['OptimizedImages', 'devtoolsLogs', 'traces', 'URL', 'GatherContext',
-        'ImageElements'],
+        'ImageElements', 'SourceMaps'],
     };
   }
 

package/core/audits/byte-efficiency/offscreen-images.js

@@ -52,7 +52,7 @@ class OffscreenImages extends ByteEfficiencyAudit {
       supportedModes: ['navigation'],
       guidanceLevel: 2,
       requiredArtifacts: ['ImageElements', 'ViewportDimensions', 'GatherContext', 'devtoolsLogs',
-        'traces', 'URL'],
+        'traces', 'URL', 'SourceMaps'],
     };
   }
 
@@ -163,12 +163,12 @@ class OffscreenImages extends ByteEfficiencyAudit {
    * @return {Promise<import('./byte-efficiency-audit.js').ByteEfficiencyProduct>}
    */
   static async audit_(artifacts, networkRecords, context) {
+    const {URL, SourceMaps} = artifacts;
     const images = artifacts.ImageElements;
     const viewportDimensions = artifacts.ViewportDimensions;
     const gatherContext = artifacts.GatherContext;
     const trace = artifacts.traces[ByteEfficiencyAudit.DEFAULT_PASS];
     const devtoolsLog = artifacts.devtoolsLogs[ByteEfficiencyAudit.DEFAULT_PASS];
-    const URL = artifacts.URL;
 
     /** @type {string[]} */
     const warnings = [];
@@ -199,7 +199,7 @@ class OffscreenImages extends ByteEfficiencyAudit {
     const unfilteredResults = Array.from(resultsMap.values());
     // get the interactive time or fallback to getting the end of trace time
     try {
-      const metricComputationData = {trace, devtoolsLog, gatherContext, settings, URL};
+      const metricComputationData = {trace, devtoolsLog, gatherContext, settings, URL, SourceMaps};
       const interactive = await Interactive.request(metricComputationData, context);
 
       // use interactive to generate items

package/core/audits/byte-efficiency/render-blocking-resources.js

@@ -112,7 +112,8 @@ class RenderBlockingResources extends Audit {
       // TODO: look into adding an `optionalArtifacts` property that captures the non-required nature
       // of CSSUsage
       requiredArtifacts:
-        ['URL', 'traces', 'devtoolsLogs', 'Stylesheets', 'CSSUsage', 'GatherContext', 'Stacks'],
+        // eslint-disable-next-line max-len
+        ['URL', 'traces', 'devtoolsLogs', 'Stylesheets', 'CSSUsage', 'GatherContext', 'Stacks', 'SourceMaps'],
     };
   }
 
@@ -126,10 +127,11 @@ class RenderBlockingResources extends Audit {
     const gatherContext = artifacts.GatherContext;
     const trace = artifacts.traces[Audit.DEFAULT_PASS];
     const devtoolsLog = artifacts.devtoolsLogs[Audit.DEFAULT_PASS];
+    const SourceMaps = artifacts.SourceMaps;
     const simulatorData = {devtoolsLog, settings: context.settings};
     const simulator = await LoadSimulator.request(simulatorData, context);
     const wastedCssBytes = await RenderBlockingResources.computeWastedCSSBytes(artifacts, context);
-    const navInsights = await NavigationInsights.request({trace, settings}, context);
+    const navInsights = await NavigationInsights.request({trace, settings, SourceMaps}, context);
 
     const renderBlocking = navInsights.model.RenderBlocking;
     if (renderBlocking instanceof Error) throw renderBlocking;
@@ -141,7 +143,7 @@ class RenderBlockingResources extends Audit {
     };
 
     const metricComputationData = {trace, devtoolsLog, gatherContext, simulator,
-      settings: metricSettings, URL: artifacts.URL};
+      settings: metricSettings, URL: artifacts.URL, SourceMaps: artifacts.SourceMaps};
 
     // Cast to just `LanternMetric` since we explicitly set `throttlingMethod: 'simulate'`.
     const fcpSimulation = /** @type {LH.Artifacts.LanternMetric} */

package/core/audits/byte-efficiency/unminified-css.js

@@ -37,7 +37,8 @@ class UnminifiedCSS extends ByteEfficiencyAudit {
       description: str_(UIStrings.description),
       scoreDisplayMode: ByteEfficiencyAudit.SCORING_MODES.METRIC_SAVINGS,
       guidanceLevel: 3,
-      requiredArtifacts: ['Stylesheets', 'devtoolsLogs', 'traces', 'URL', 'GatherContext'],
+      requiredArtifacts: ['Stylesheets', 'devtoolsLogs', 'traces', 'URL', 'GatherContext',
+        'SourceMaps'],
     };
   }
 

package/core/audits/byte-efficiency/unminified-javascript.js

@@ -44,7 +44,8 @@ class UnminifiedJavaScript extends ByteEfficiencyAudit {
       description: str_(UIStrings.description),
       scoreDisplayMode: ByteEfficiencyAudit.SCORING_MODES.METRIC_SAVINGS,
       guidanceLevel: 3,
-      requiredArtifacts: ['Scripts', 'devtoolsLogs', 'traces', 'GatherContext', 'URL'],
+      requiredArtifacts: ['Scripts', 'devtoolsLogs', 'traces', 'GatherContext', 'URL',
+        'SourceMaps'],
     };
   }
 

package/core/audits/byte-efficiency/unused-css-rules.js

@@ -35,7 +35,7 @@ class UnusedCSSRules extends ByteEfficiencyAudit {
       scoreDisplayMode: ByteEfficiencyAudit.SCORING_MODES.METRIC_SAVINGS,
       guidanceLevel: 1,
       requiredArtifacts:
-        ['Stylesheets', 'CSSUsage', 'URL', 'devtoolsLogs', 'traces', 'GatherContext'],
+        ['Stylesheets', 'CSSUsage', 'URL', 'devtoolsLogs', 'traces', 'GatherContext', 'SourceMaps'],
     };
   }
 

package/core/audits/byte-efficiency/unused-javascript.js

@@ -69,7 +69,7 @@ class UnusedJavaScript extends ByteEfficiencyAudit {
       scoreDisplayMode: ByteEfficiencyAudit.SCORING_MODES.METRIC_SAVINGS,
       guidanceLevel: 1,
       requiredArtifacts: ['JsUsage', 'Scripts', 'SourceMaps', 'GatherContext',
-        'devtoolsLogs', 'traces', 'URL'],
+        'devtoolsLogs', 'traces', 'URL', 'SourceMaps'],
     };
   }
 

package/core/audits/byte-efficiency/uses-long-cache-ttl.js

@@ -46,7 +46,7 @@ class CacheHeaders extends Audit {
       description: str_(UIStrings.description),
       scoreDisplayMode: Audit.SCORING_MODES.METRIC_SAVINGS,
       guidanceLevel: 3,
-      requiredArtifacts: ['devtoolsLogs'],
+      requiredArtifacts: ['devtoolsLogs', 'SourceMaps'],
     };
   }
 

package/core/audits/byte-efficiency/uses-optimized-images.js

@@ -37,7 +37,7 @@ class UsesOptimizedImages extends ByteEfficiencyAudit {
       scoreDisplayMode: ByteEfficiencyAudit.SCORING_MODES.METRIC_SAVINGS,
       guidanceLevel: 2,
       requiredArtifacts: ['OptimizedImages', 'ImageElements', 'GatherContext', 'devtoolsLogs',
-        'traces', 'URL'],
+        'traces', 'URL', 'SourceMaps'],
     };
   }
 

package/core/audits/byte-efficiency/uses-responsive-images.js

@@ -48,7 +48,7 @@ class UsesResponsiveImages extends ByteEfficiencyAudit {
       scoreDisplayMode: ByteEfficiencyAudit.SCORING_MODES.METRIC_SAVINGS,
       guidanceLevel: 2,
       requiredArtifacts: ['ImageElements', 'ViewportDimensions', 'GatherContext',
-        'devtoolsLogs', 'traces', 'URL'],
+        'devtoolsLogs', 'traces', 'URL', 'SourceMaps'],
     };
   }