lighthouse 12.6.1 → 12.7.0-dev.20250628

package/core/config/filters.js

@@ -13,6 +13,7 @@ const baseArtifactKeySource = {
   fetchTime: '',
   LighthouseRunWarnings: '',
   BenchmarkIndex: '',
+  HostDPR: '',
   settings: '',
   Timing: '',
   URL: '',
@@ -263,6 +264,12 @@ function filterConfigByGatherMode(resolvedConfig, mode) {
  */
 function filterConfigByExplicitFilters(resolvedConfig, filters) {
   const {onlyAudits, onlyCategories, skipAudits} = filters;
+  if (onlyAudits && !onlyAudits.length) {
+    throw new Error(`onlyAudits cannot be an empty array.`);
+  }
+  if (onlyCategories && !onlyCategories.length) {
+    throw new Error(`onlyCategories cannot be an empty array.`);
+  }
 
   warnOnUnknownOnlyCategories(resolvedConfig.categories, onlyCategories);
 

package/core/config/validation.js

@@ -36,6 +36,10 @@ function isValidArtifactDependency(dependent, dependency) {
  * @param {string} pluginName
  */
 function assertValidPluginName(config, pluginName) {
+  const parts = pluginName.split('/');
+  if (parts.length === 2) {
+    pluginName = parts[1];
+  }
   if (!pluginName.startsWith('lighthouse-plugin-')) {
     throw new Error(`plugin name '${pluginName}' does not start with 'lighthouse-plugin-'`);
   }

package/core/gather/base-artifacts.js

@@ -9,6 +9,7 @@ import {isEqual} from 'lodash-es';
 
 import {
   getBrowserVersion, getBenchmarkIndex, getEnvironmentWarnings,
+  getDevicePixelRatio,
 } from './driver/environment.js';
 
 /**
@@ -20,6 +21,7 @@ import {
 async function getBaseArtifacts(resolvedConfig, driver, context) {
   const BenchmarkIndex = await getBenchmarkIndex(driver.executionContext);
   const {userAgent, product} = await getBrowserVersion(driver.defaultSession);
+  const HostDPR = await getDevicePixelRatio(driver.executionContext);
 
   return {
     // Meta artifacts.
@@ -29,6 +31,7 @@ async function getBaseArtifacts(resolvedConfig, driver, context) {
     settings: resolvedConfig.settings,
     // Environment artifacts that can always be computed.
     BenchmarkIndex,
+    HostDPR,
     HostUserAgent: userAgent,
     HostFormFactor: userAgent.includes('Android') || userAgent.includes('Mobile') ?
       'mobile' : 'desktop',

package/core/gather/driver/environment.d.ts

@@ -14,6 +14,12 @@ export function getBrowserVersion(session: LH.Gatherer.ProtocolSession): Promise
  * @return {Promise<number>}
  */
 export function getBenchmarkIndex(executionContext: LH.Gatherer.Driver["executionContext"]): Promise<number>;
+/**
+ * Computes the observed DPR.
+ * @param {LH.Gatherer.Driver['executionContext']} executionContext
+ * @return {Promise<number>}
+ */
+export function getDevicePixelRatio(executionContext: LH.Gatherer.Driver["executionContext"]): Promise<number>;
 /**
  * Returns a warning if the host device appeared to be underpowered according to BenchmarkIndex.
  *

package/core/gather/driver/environment.js

@@ -58,6 +58,22 @@ async function getBenchmarkIndex(executionContext) {
   return indexVal;
 }
 
+/**
+ * Computes the observed DPR.
+ * @param {LH.Gatherer.Driver['executionContext']} executionContext
+ * @return {Promise<number>}
+ */
+async function getDevicePixelRatio(executionContext) {
+  const status = {msg: 'Host device pixel ratio', id: 'lh:gather:getDevicePixelRatio'};
+  log.time(status);
+  // eslint-disable-next-line no-undef
+  const indexVal = await executionContext.evaluate(() => devicePixelRatio, {
+    args: [],
+  });
+  log.timeEnd(status);
+  return indexVal;
+}
+
 /**
  * Returns a warning if the host device appeared to be underpowered according to BenchmarkIndex.
  *
@@ -100,6 +116,7 @@ export {
   UIStrings,
   getBrowserVersion,
   getBenchmarkIndex,
+  getDevicePixelRatio,
   getSlowHostCpuWarning,
   getEnvironmentWarnings,
 };

package/core/gather/driver/execution-context.d.ts

@@ -58,11 +58,13 @@ export class ExecutionContext {
      */
     _evaluateInContext(expression: string, contextId: number | undefined, timeout: number): Promise<any>;
     /**
-     * Note: Prefer `evaluate` instead.
      * Evaluate an expression in the context of the current page. If useIsolation is true, the expression
      * will be evaluated in a content script that has access to the page's DOM but whose JavaScript state
      * is completely separate.
      * Returns a promise that resolves on the expression's value.
+     *
+     * @deprecated Use `evaluate` instead! It has a better API, and unlike `evaluateAsync` doesn't sometimes
+     * execute invalid code.
      * @param {string} expression
      * @param {{useIsolation?: boolean}=} options
      * @return {Promise<*>}

package/core/gather/driver/execution-context.js

@@ -151,11 +151,13 @@ class ExecutionContext {
   }
 
   /**
-   * Note: Prefer `evaluate` instead.
    * Evaluate an expression in the context of the current page. If useIsolation is true, the expression
    * will be evaluated in a content script that has access to the page's DOM but whose JavaScript state
    * is completely separate.
    * Returns a promise that resolves on the expression's value.
+   *
+   * @deprecated Use `evaluate` instead! It has a better API, and unlike `evaluateAsync` doesn't sometimes
+   * execute invalid code.
    * @param {string} expression
    * @param {{useIsolation?: boolean}=} options
    * @return {Promise<*>}

package/core/gather/driver/navigation.js

@@ -6,7 +6,7 @@
 
 import log from 'lighthouse-logger';
 
-import {waitForFullyLoaded, waitForFrameNavigated, waitForUserToContinue} from './wait-for-condition.js'; // eslint-disable-line max-len
+import {waitForFullyLoaded, waitForFrameNavigated, waitForUserToContinue} from './wait-for-condition.js';
 import * as constants from '../../config/constants.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 import UrlUtils from '../../lib/url-utils.js';
@@ -43,7 +43,6 @@ const DEFAULT_CPU_QUIET_THRESHOLD = 0;
 
 /** @param {NavigationOptions} options */
 function resolveWaitForFullyLoadedOptions(options) {
-  /* eslint-disable max-len */
   let {pauseAfterFcpMs, pauseAfterLoadMs, networkQuietThresholdMs, cpuQuietThresholdMs} = options;
   let maxWaitMs = options.maxWaitForLoad;
   let maxFCPMs = options.maxWaitForFcp;
@@ -56,7 +55,6 @@ function resolveWaitForFullyLoadedOptions(options) {
   if (typeof cpuQuietThresholdMs !== 'number') cpuQuietThresholdMs = DEFAULT_CPU_QUIET_THRESHOLD;
   if (typeof maxWaitMs !== 'number') maxWaitMs = constants.defaultSettings.maxWaitForLoad;
   if (typeof maxFCPMs !== 'number') maxFCPMs = constants.defaultSettings.maxWaitForFcp;
-  /* eslint-enable max-len */
 
   if (!options.waitUntil.includes('fcp')) maxFCPMs = undefined;
 

package/core/gather/driver/wait-for-condition.js

@@ -138,7 +138,6 @@ function waitForNetworkIdle(session, networkMonitor, networkQuietOptions) {
   /** @type {Promise<void>} */
   const promise = new Promise((resolve, reject) => {
     const onIdle = () => {
-      // eslint-disable-next-line no-use-before-define
       networkMonitor.once(busyEvent, onBusy);
       idleTimeout = setTimeout(() => {
         cancel();

package/core/gather/fetcher.js

@@ -10,8 +10,6 @@ import * as LH from '../../types/lh.js';
  * ignoring normal browser constraints such as CORS.
  */
 
-/* global fetch */
-
 /** @typedef {{content: string|null, status: number|null}} FetchResponse */
 
 class Fetcher {

package/core/gather/gatherers/accessibility.js

@@ -4,7 +4,7 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-/* global window, document, getNodeDetails */
+/* global getNodeDetails */
 
 import BaseGatherer from '../base-gatherer.js';
 import {axeSource} from '../../lib/axe.js';

package/core/gather/gatherers/anchor-elements.js

@@ -10,8 +10,6 @@ import BaseGatherer from '../base-gatherer.js';
 import {pageFunctions} from '../../lib/page-functions.js';
 import {resolveDevtoolsNodePathToObjectId} from '../driver/dom.js';
 
-/* eslint-env browser, node */
-
 /**
  * Function that is stringified and run in the page to collect anchor elements.
  * Additional complexity is introduced because anchors can be HTML or SVG elements.
@@ -38,10 +36,49 @@ function collectAnchorElements() {
     return onclick.slice(0, 1024);
   }
 
+  /**
+   * @param {HTMLElement|SVGElement} node
+   * @return {string|null}
+   */
+  function getLangOfInnerText(node) {
+    let curNodeLang = null;
+
+    // If we find multiple languages within this element, return null.
+    for (const child of node.querySelectorAll('*')) {
+      if (!child.textContent) continue;
+
+      const childLang = child.closest('[lang]')?.getAttribute('lang');
+      if (!childLang) continue;
+
+      if (!curNodeLang) {
+        curNodeLang = childLang;
+        continue;
+      }
+
+      if (curNodeLang.split('-')[0] !== childLang.split('-')[0]) {
+        return null;
+      }
+    }
+
+    return curNodeLang ?? node.closest('[lang]')?.getAttribute('lang') ?? null;
+  }
+
   /** @type {Array<HTMLAnchorElement|SVGAElement>} */
   // @ts-expect-error - put into scope via stringification
   const anchorElements = getElementsInDocument('a'); // eslint-disable-line no-undef
 
+  // Check, if document has only one lang attribute in opening html or in body tag. If so,
+  // there is no need to run the `getLangOfInnerText()` function with multiple
+  // possible DOM traversals
+  /** @type {Array<HTMLElement|SVGElement>} */
+  // @ts-expect-error - put into scope via stringification
+  const langElements = getElementsInDocument('[lang]'); // eslint-disable-line no-undef
+  const documentHasSingleLang = langElements.length === 1 &&
+    (langElements[0].nodeName === 'BODY' || langElements[0].nodeName === 'HTML');
+  const singleLang = documentHasSingleLang ? langElements[0].getAttribute('lang') : null;
+
+  // TODO: consider Content-Language.
+
   return anchorElements.map(node => {
     if (node instanceof HTMLAnchorElement) {
       return {
@@ -51,9 +88,11 @@ function collectAnchorElements() {
         role: node.getAttribute('role') || '',
         name: node.name,
         text: node.innerText, // we don't want to return hidden text, so use innerText
+        textLang: singleLang ?? getLangOfInnerText(node) ?? undefined,
         rel: node.rel,
         target: node.target,
         id: node.getAttribute('id') || '',
+        attributeNames: node.getAttributeNames(),
         // @ts-expect-error - getNodeDetails put into scope via stringification
         node: getNodeDetails(node),
       };
@@ -65,9 +104,11 @@ function collectAnchorElements() {
       onclick: getTruncatedOnclick(node),
       role: node.getAttribute('role') || '',
       text: node.textContent || '',
+      textLang: singleLang ?? getLangOfInnerText(node) ?? undefined,
       rel: '',
       target: node.target.baseVal || '',
       id: node.getAttribute('id') || '',
+      attributeNames: node.getAttributeNames(),
       // @ts-expect-error - getNodeDetails put into scope via stringification
       node: getNodeDetails(node),
     };
@@ -119,9 +160,27 @@ class AnchorElements extends BaseGatherer {
     const anchorsWithEventListeners = anchors.map(async anchor => {
       const listeners = await getEventListeners(session, anchor.node.devtoolsNodePath);
 
+      /** @type {Set<{type: string}>} */
+      const ancestorListeners = new Set();
+      const splitPath = anchor.node.devtoolsNodePath.split(',');
+      const ancestorListenerPromises = [];
+      while (splitPath.length >= 2) {
+        splitPath.length -= 2;
+        const path = splitPath.join(',');
+        const promise = getEventListeners(session, path).then(listeners => {
+          for (const listener of listeners) {
+            ancestorListeners.add(listener);
+          }
+        }).catch(() => {});
+        ancestorListenerPromises.push(promise);
+      }
+
+      await Promise.all(ancestorListenerPromises);
+
       return {
         ...anchor,
         listeners,
+        ancestorListeners: Array.from(ancestorListeners),
       };
     });
 

package/core/gather/gatherers/cache-contents.js

@@ -4,8 +4,6 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-/* global caches */
-
 import BaseGatherer from '../base-gatherer.js';
 
 /**

package/core/gather/gatherers/css-usage.js

@@ -30,7 +30,9 @@ class CSSUsage extends BaseGatherer {
 
     // Force style to recompute.
     // Doesn't appear to be necessary in newer versions of Chrome.
-    await executionContext.evaluateAsync('getComputedStyle(document.body)');
+    await executionContext.evaluate(() => window.getComputedStyle(document.body), {
+      args: [],
+    });
 
     const {ruleUsage} = await session.sendCommand('CSS.stopRuleUsageTracking');
     await session.sendCommand('CSS.disable');

package/core/gather/gatherers/dobetterweb/doctype.js

@@ -6,8 +6,6 @@
 
 import BaseGatherer from '../../base-gatherer.js';
 
-/* global document */
-
 /**
  * Get and return `name`, `publicId`, `systemId` from
  * `document.doctype`

package/core/gather/gatherers/dobetterweb/domstats.js

@@ -9,7 +9,7 @@
  * and total number of elements used on the page.
  */
 
-/* global getNodeDetails document */
+/* global getNodeDetails */
 
 
 import BaseGatherer from '../../base-gatherer.js';

package/core/gather/gatherers/full-page-screenshot.js

@@ -3,7 +3,7 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-/* globals window getBoundingClientRect requestAnimationFrame */
+/* globals getBoundingClientRect */
 
 import BaseGatherer from '../base-gatherer.js';
 import * as emulation from '../../lib/emulation.js';

package/core/gather/gatherers/image-elements.js

@@ -15,7 +15,7 @@ import BaseGatherer from '../base-gatherer.js';
 import {pageFunctions} from '../../lib/page-functions.js';
 import * as FontSize from './seo/font-size.js';
 
-/* global window, getElementsInDocument, Image, getNodeDetails, ShadowRoot */
+/* global getElementsInDocument, getNodeDetails */
 
 /** @param {Element} element */
 /* c8 ignore start */

package/core/gather/gatherers/inspector-issues.js

@@ -81,7 +81,7 @@ class InspectorIssues extends BaseGatherer {
       propertyRuleIssue: [],
       quirksModeIssue: [],
       cookieIssue: [],
-      selectElementAccessibilityIssue: [],
+      elementAccessibilityIssue: [],
       sharedArrayBufferIssue: [],
       sharedDictionaryIssue: [],
       stylesheetLoadingIssue: [],

package/core/gather/gatherers/link-elements.js

@@ -13,7 +13,7 @@ import {MainResource} from '../../computed/main-resource.js';
 import {Util} from '../../../shared/util.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 
-/* globals HTMLLinkElement getNodeDetails */
+/* globals getNodeDetails */
 
 /**
  * @fileoverview

package/core/gather/gatherers/stacks.js

@@ -8,7 +8,6 @@
  * @fileoverview Gathers a list of detected JS libraries and their versions.
  */
 
-/* global window */
 /* global d41d8cd98f00b204e9800998ecf8427e_LibraryDetectorTests */
 
 import fs from 'fs';

package/core/gather/gatherers/stylesheets.js

@@ -74,7 +74,9 @@ class Stylesheets extends BaseGatherer {
 
     // Force style to recompute.
     // Doesn't appear to be necessary in newer versions of Chrome.
-    await executionContext.evaluateAsync('getComputedStyle(document.body)');
+    await executionContext.evaluate(() => window.getComputedStyle(document.body), {
+      args: [],
+    });
 
     session.off('CSS.styleSheetAdded', this._onStylesheetAdded);
 

package/core/gather/gatherers/trace-elements.d.ts

@@ -23,21 +23,20 @@ declare class TraceElements extends BaseGatherer {
      *
      * @param {LH.Artifacts.TraceImpactedNode[]} impactedNodes
      * @param {Map<number, number>} impactByNodeId
-     * @param {import('../../lib/trace-engine.js').SaneSyntheticLayoutShift} event Only for debugging
      * @return {number|undefined}
      */
-    static getBiggestImpactNodeForShiftEvent(impactedNodes: LH.Artifacts.TraceImpactedNode[], impactByNodeId: Map<number, number>, event: import("../../lib/trace-engine.js").SaneSyntheticLayoutShift): number | undefined;
+    static getBiggestImpactNodeForShiftEvent(impactedNodes: LH.Artifacts.TraceImpactedNode[], impactByNodeId: Map<number, number>): number | undefined;
     /**
      * This function finds the top (up to 15) layout shifts on the page, and returns
      * the id of the largest impacted node of each shift, along with any related nodes
      * that may have caused the shift.
      *
      * @param {LH.Trace} trace
-     * @param {LH.Artifacts.TraceEngineResult['data']} traceEngineResult
+     * @param {LH.Artifacts.TraceEngineResult['parsedTrace']} traceEngineResult
      * @param {LH.Gatherer.Context} context
      * @return {Promise<Array<{nodeId: number}>>}
      */
-    static getTopLayoutShifts(trace: LH.Trace, traceEngineResult: LH.Artifacts.TraceEngineResult["data"], context: LH.Gatherer.Context): Promise<Array<{
+    static getTopLayoutShifts(trace: LH.Trace, traceEngineResult: LH.Artifacts.TraceEngineResult["parsedTrace"], context: LH.Gatherer.Context): Promise<Array<{
         nodeId: number;
     }>>;
     /**

package/core/gather/gatherers/trace-elements.js

@@ -35,7 +35,7 @@ const MAX_LAYOUT_SHIFTS = 15;
  */
 /* c8 ignore start */
 function getNodeDetailsData() {
-  const elem = this.nodeType === document.ELEMENT_NODE ? this : this.parentElement; // eslint-disable-line no-undef
+  const elem = this.nodeType === document.ELEMENT_NODE ? this : this.parentElement;
   let traceElement;
   if (elem) {
     // @ts-expect-error - getNodeDetails put into scope via stringification
@@ -152,49 +152,19 @@ class TraceElements extends BaseGatherer {
    *
    * @param {LH.Artifacts.TraceImpactedNode[]} impactedNodes
    * @param {Map<number, number>} impactByNodeId
-   * @param {import('../../lib/trace-engine.js').SaneSyntheticLayoutShift} event Only for debugging
    * @return {number|undefined}
    */
-  static getBiggestImpactNodeForShiftEvent(impactedNodes, impactByNodeId, event) {
-    try {
-      let biggestImpactNodeId;
-      let biggestImpactNodeScore = Number.NEGATIVE_INFINITY;
-      for (const node of impactedNodes) {
-        const impactScore = impactByNodeId.get(node.node_id);
-        if (impactScore !== undefined && impactScore > biggestImpactNodeScore) {
-          biggestImpactNodeId = node.node_id;
-          biggestImpactNodeScore = impactScore;
-        }
+  static getBiggestImpactNodeForShiftEvent(impactedNodes, impactByNodeId) {
+    let biggestImpactNodeId;
+    let biggestImpactNodeScore = Number.NEGATIVE_INFINITY;
+    for (const node of impactedNodes) {
+      const impactScore = impactByNodeId.get(node.node_id);
+      if (impactScore !== undefined && impactScore > biggestImpactNodeScore) {
+        biggestImpactNodeId = node.node_id;
+        biggestImpactNodeScore = impactScore;
       }
-      return biggestImpactNodeId;
-    } catch (err) {
-      // See https://github.com/GoogleChrome/lighthouse/issues/15870
-      // `impactedNodes` should always be an array here, but it can randomly be something else for
-      // currently unknown reasons. This exception handling will help us identify what
-      // `impactedNodes` really is and also prevent the error from being fatal.
-
-      // It's possible `impactedNodes` is not JSON serializable, so let's add more supplemental
-      // fields just in case.
-      const impactedNodesType = typeof impactedNodes;
-      const impactedNodesClassName = impactedNodes?.constructor?.name;
-
-      let impactedNodesJson;
-      let eventJson;
-      try {
-        impactedNodesJson = JSON.parse(JSON.stringify(impactedNodes));
-        eventJson = JSON.parse(JSON.stringify(event));
-      } catch {}
-
-      Sentry.captureException(err, {
-        extra: {
-          impactedNodes: impactedNodesJson,
-          event: eventJson,
-          impactedNodesType,
-          impactedNodesClassName,
-        },
-      });
-      return;
     }
+    return biggestImpactNodeId;
   }
 
   /**
@@ -203,7 +173,7 @@ class TraceElements extends BaseGatherer {
    * that may have caused the shift.
    *
    * @param {LH.Trace} trace
-   * @param {LH.Artifacts.TraceEngineResult['data']} traceEngineResult
+   * @param {LH.Artifacts.TraceEngineResult['parsedTrace']} traceEngineResult
    * @param {LH.Gatherer.Context} context
    * @return {Promise<Array<{nodeId: number}>>}
    */
@@ -222,7 +192,7 @@ class TraceElements extends BaseGatherer {
         const nodeIds = [];
         const impactedNodes = event.args.data.impacted_nodes || [];
         const biggestImpactedNodeId =
-          this.getBiggestImpactNodeForShiftEvent(impactedNodes, impactByNodeId, event);
+          this.getBiggestImpactNodeForShiftEvent(impactedNodes, impactByNodeId);
         if (biggestImpactedNodeId !== undefined) {
           nodeIds.push(biggestImpactedNodeId);
         }
@@ -399,7 +369,7 @@ class TraceElements extends BaseGatherer {
       traceEngineResult, navigationId);
     const lcpNodeData = await TraceElements.getLcpElement(trace, context);
     const shiftsData = await TraceElements.getTopLayoutShifts(
-      trace, traceEngineResult.data, context);
+      trace, traceEngineResult.parsedTrace, context);
     const animatedElementData = await this.getAnimatedElements(mainThreadEvents);
     const responsivenessElementData = await TraceElements.getResponsivenessElement(trace, context);
 

package/core/gather/gatherers/viewport-dimensions.js

@@ -6,8 +6,6 @@
 
 import BaseGatherer from '../base-gatherer.js';
 
-/* global window */
-
 /**
  * @return {LH.Artifacts.ViewportDimensions}
  */

package/core/index.cjs

@@ -3,7 +3,6 @@
  * Copyright 2022 Google LLC
  * SPDX-License-Identifier: Apache-2.0
  */
-'use strict';
 
 /**
  * @typedef ExportType

package/core/index.d.cts

@@ -1,4 +1,9 @@
 export = lighthouse;
+/**
+ * @license
+ * Copyright 2022 Google LLC
+ * SPDX-License-Identifier: Apache-2.0
+ */
 /**
  * @typedef ExportType
  * @property {import('./index.js')['startFlow']} startFlow

package/core/lib/asset-saver.d.ts

@@ -102,5 +102,5 @@ export function normalizeTimingEntries(timings: LH.Result.MeasureEntry[]): void;
 /**
  * @param {LH.Result} lhr
  */
-export function elideAuditErrorStacks(lhr: LH.Result): void;
+export function elideLhrErrorStacks(lhr: LH.Result): void;
 //# sourceMappingURL=asset-saver.d.ts.map

package/core/lib/asset-saver.js

@@ -515,19 +515,31 @@ function normalizeTimingEntries(timings) {
 }
 
 /**
- * @param {LH.Result} lhr
+ * @param {string} errorStack
+ * @return {string}
  */
-function elideAuditErrorStacks(lhr) {
+function elideErrorStack(errorStack) {
   const baseCallFrameUrl = url.pathToFileURL(LH_ROOT);
+  return errorStack
+    // Make paths relative to the repo root.
+    .replaceAll(baseCallFrameUrl.pathname, '')
+    // Remove line/col info.
+    .replaceAll(/:\d+:\d+/g, '');
+}
+
+/**
+ * @param {LH.Result} lhr
+ */
+function elideLhrErrorStacks(lhr) {
   for (const auditResult of Object.values(lhr.audits)) {
     if (auditResult.errorStack) {
-      auditResult.errorStack = auditResult.errorStack
-        // Make paths relative to the repo root.
-        .replaceAll(baseCallFrameUrl.pathname, '')
-        // Remove line/col info.
-        .replaceAll(/:\d+:\d+/g, '');
+      auditResult.errorStack = elideErrorStack(auditResult.errorStack);
     }
   }
+
+  if (lhr.runtimeError?.errorStack) {
+    lhr.runtimeError.errorStack = elideErrorStack(lhr.runtimeError.errorStack);
+  }
 }
 
 export {
@@ -543,5 +555,5 @@ export {
   saveLanternNetworkData,
   stringifyReplacer,
   normalizeTimingEntries,
-  elideAuditErrorStacks,
+  elideLhrErrorStacks,
 };

package/core/lib/bf-cache-strings.js

@@ -704,7 +704,10 @@ const NotRestoredReasonDescription = {
   WebViewSafeBrowsingAllowlistChanged: {name: ('WebViewSafeBrowsingAllowlistChanged')},
   WebViewDocumentStartJavascriptChanged: {name: ('WebViewDocumentStartJavascriptChanged')},
   CacheControlNoStoreDeviceBoundSessionTerminated: {name: str_(UIStrings.cacheControlNoStore)},
-  CacheLimitPruned: {name: ('CacheLimitPruned')},
+  CacheLimitPrunedOnModerateMemoryPressure:
+      {name: ('CacheLimitPrunedOnModerateMemoryPressure')},
+  CacheLimitPrunedOnCriticalMemoryPressure:
+      {name: ('CacheLimitPrunedOnCriticalMemoryPressure')},
 };
 
 export {