lighthouse 11.7.0-dev.20240410 → 11.7.0-dev.20240411

package/core/lib/lantern/metrics/first-meaningful-paint.js

@@ -6,8 +6,6 @@
 
 import * as Lantern from '../types/lantern.js';
 import {Metric} from '../metric.js';
-// TODO(15841): don't use LighthouseError
-import {LighthouseError} from '../../lh-error.js';
 import {FirstContentfulPaint} from './first-contentful-paint.js';
 
 /** @typedef {import('../base-node.js').Node} Node */
@@ -32,7 +30,7 @@ class FirstMeaningfulPaint extends Metric {
   static getOptimisticGraph(dependencyGraph, processedNavigation) {
     const fmp = processedNavigation.timestamps.firstMeaningfulPaint;
     if (!fmp) {
-      throw new LighthouseError(LighthouseError.errors.NO_FMP);
+      throw new Error('NO_FMP');
     }
     return FirstContentfulPaint.getFirstPaintBasedGraph(dependencyGraph, {
       cutoffTimestamp: fmp,
@@ -51,7 +49,7 @@ class FirstMeaningfulPaint extends Metric {
   static getPessimisticGraph(dependencyGraph, processedNavigation) {
     const fmp = processedNavigation.timestamps.firstMeaningfulPaint;
     if (!fmp) {
-      throw new LighthouseError(LighthouseError.errors.NO_FMP);
+      throw new Error('NO_FMP');
     }
 
     return FirstContentfulPaint.getFirstPaintBasedGraph(dependencyGraph, {

package/core/lib/lantern/metrics/interactive.js

@@ -37,7 +37,7 @@ class Interactive extends Metric {
     return dependencyGraph.cloneWithRelationships(node => {
       // Include everything that might be a long task
       if (node.type === BaseNode.TYPES.CPU) {
-        return node.event.dur > minimumCpuTaskDuration;
+        return node.duration > minimumCpuTaskDuration;
       }
 
       // Include all scripts and high priority requests, exclude all images

package/core/lib/lantern/metrics/largest-contentful-paint.js

@@ -6,8 +6,8 @@
 
 import * as Lantern from '../types/lantern.js';
 import {Metric} from '../metric.js';
-import {LighthouseError} from '../../../lib/lh-error.js';
 import {FirstContentfulPaint} from './first-contentful-paint.js';
+import {LanternError} from '../lantern-error.js';
 
 /** @typedef {import('../base-node.js').Node} Node */
 
@@ -45,7 +45,7 @@ class LargestContentfulPaint extends Metric {
   static getOptimisticGraph(dependencyGraph, processedNavigation) {
     const lcp = processedNavigation.timestamps.largestContentfulPaint;
     if (!lcp) {
-      throw new LighthouseError(LighthouseError.errors.NO_LCP);
+      throw new LanternError('NO_LCP');
     }
 
     return FirstContentfulPaint.getFirstPaintBasedGraph(dependencyGraph, {
@@ -62,7 +62,7 @@ class LargestContentfulPaint extends Metric {
   static getPessimisticGraph(dependencyGraph, processedNavigation) {
     const lcp = processedNavigation.timestamps.largestContentfulPaint;
     if (!lcp) {
-      throw new LighthouseError(LighthouseError.errors.NO_LCP);
+      throw new LanternError('NO_LCP');
     }
 
     return FirstContentfulPaint.getFirstPaintBasedGraph(dependencyGraph, {

package/core/lib/lantern/page-dependency-graph.js

@@ -128,6 +128,9 @@ class PageDependencyGraph {
         continue;
       }
 
+      /** @type {number|undefined} */
+      let correctedEndTs = undefined;
+
       // Capture all events that occurred within the task
       /** @type {Array<LH.TraceEvent>} */
       const children = [];
@@ -136,10 +139,19 @@ class PageDependencyGraph {
         i < mainThreadEvents.length && mainThreadEvents[i].ts < endTime;
         i++
       ) {
+        // Temporary fix for a Chrome bug where some RunTask events can be overlapping.
+        // We correct that here be ensuring each RunTask ends at least 1 microsecond before the next
+        // https://github.com/GoogleChrome/lighthouse/issues/15896
+        // https://issues.chromium.org/issues/329678173
+        if (TraceProcessor.isScheduleableTask(mainThreadEvents[i]) && mainThreadEvents[i].dur) {
+          correctedEndTs = mainThreadEvents[i].ts - 1;
+          break;
+        }
+
         children.push(mainThreadEvents[i]);
       }
 
-      nodes.push(new CPUNode(evt, children));
+      nodes.push(new CPUNode(evt, children, correctedEndTs));
     }
 
     return nodes;
@@ -359,7 +371,7 @@ class PageDependencyGraph {
         isFirst = foundFirstParse = true;
       }
 
-      if (isFirst || node.event.dur >= minimumEvtDur) {
+      if (isFirst || node.duration >= minimumEvtDur) {
         // Don't prune this node. The task is long / important so it will impact simulation.
         continue;
       }

package/core/lib/lantern/simulator/simulator.js

@@ -276,7 +276,7 @@ class Simulator {
       ? this._layoutTaskMultiplier
       : this._cpuSlowdownMultiplier;
     const totalDuration = Math.min(
-      Math.round(cpuNode.event.dur / 1000 * multiplier),
+      Math.round(cpuNode.duration / 1000 * multiplier),
       DEFAULT_MAXIMUM_CPU_TASK_DURATION
     );
     const estimatedTimeElapsed = totalDuration - timingData.timeElapsed;

package/core/lib/lh-error.js

@@ -76,13 +76,13 @@ const UIStrings = {
   criTimeout: 'Timeout waiting for initial Debugger Protocol connection.',
   /**
    * @description Error message explaining that a resource that was required for testing was never collected. "artifactName" will be replaced with the name of the resource that wasn't collected.
-   * @example {WebAppManifest} artifactName
+   * @example {MainDocumentContent} artifactName
    * */
   missingRequiredArtifact: 'Required {artifactName} gatherer did not run.',
   /**
    * @description Error message explaining that there was an error while trying to collect a resource that was required for testing. "artifactName" will be replaced with the name of the resource that wasn't collected; "errorMessage" will be replaced with a string description of the error that occurred.
-   * @example {WebAppManifest} artifactName
-   * @example {Manifest invalid} errorMessage
+   * @example {MainDocumentContent} artifactName
+   * @example {Could not find main document} errorMessage
    * */
   erroredRequiredArtifact: 'Required {artifactName} gatherer encountered an error: {errorMessage}',
 

package/core/runner.js

@@ -106,7 +106,6 @@ class Runner {
           networkUserAgent: artifacts.NetworkUserAgent,
           hostUserAgent: artifacts.HostUserAgent,
           benchmarkIndex: artifacts.BenchmarkIndex,
-          benchmarkIndexes: artifacts.BenchmarkIndexes,
           credits,
         },
         audits: auditResultsById,

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "lighthouse",
   "type": "module",
-  "version": "11.7.0-dev.20240410",
+  "version": "11.7.0-dev.20240411",
   "description": "Automated auditing, performance metrics, and best practices for the web.",
   "main": "./core/index.js",
   "bin": {

package/tsconfig.json

@@ -48,7 +48,6 @@
     "core/test/gather/gatherers/devtools-log-test.js",
     "core/test/gather/gatherers/dobetterweb/optimized-images-test.js",
     "core/test/gather/gatherers/dobetterweb/response-compression-test.js",
-    "core/test/gather/gatherers/dobetterweb/tags-blocking-first-paint-test.js",
     "core/test/gather/gatherers/full-page-screenshot-test.js",
     "core/test/gather/gatherers/global-listeners-test.js",
     "core/test/gather/gatherers/html-without-javascript-test.js",

package/types/artifacts.d.ts

@@ -16,6 +16,7 @@ import speedline from 'speedline-core';
 import * as CDTSourceMap from '../core/lib/cdt/generated/SourceMap.js';
 import {ArbitraryEqualityMap} from '../core/lib/arbitrary-equality-map.js';
 import type { TaskNode as _TaskNode } from '../core/lib/tracehouse/main-thread-tasks.js';
+import type {EnabledHandlers} from '../core/computed/trace-engine-result.js';
 import AuditDetails from './lhr/audit-details.js'
 import Config from './config.js';
 import Gatherer from './gatherer.js';
@@ -45,8 +46,6 @@ interface UniversalBaseArtifacts {
   LighthouseRunWarnings: Array<string | IcuMessage>;
   /** The benchmark index that indicates rough device class. */
   BenchmarkIndex: number;
-  /** Many benchmark indexes. Many. */
-  BenchmarkIndexes?: number[];
   /** An object containing information about the testing configuration used by Lighthouse. */
   settings: Config.Settings;
   /** The timing instrumentation of the gather portion of a run. */
@@ -79,18 +78,20 @@ interface ContextualBaseArtifacts {
 interface PublicGathererArtifacts {
   /** ConsoleMessages deprecation and intervention warnings, console API calls, and exceptions logged by Chrome during page load. */
   ConsoleMessages: Artifacts.ConsoleMessage[];
-  /** All the iframe elements in the page. */
-  IFrameElements: Artifacts.IFrameElement[];
-  /** The contents of the main HTML document network resource. */
-  MainDocumentContent: string;
+  /** The primary log of devtools protocol activity. */
+  DevtoolsLog: DevtoolsLog;
   /** Information on size and loading for all the images in the page. Natural size information for `picture` and CSS images is only available if the image was one of the largest 50 images. */
   ImageElements: Artifacts.ImageElement[];
   /** All the link elements on the page or equivalently declared in `Link` headers. @see https://html.spec.whatwg.org/multipage/links.html */
   LinkElements: Artifacts.LinkElement[];
+  /** The contents of the main HTML document network resource. */
+  MainDocumentContent: string;
   /** The values of the <meta> elements in the head. */
   MetaElements: Array<{name?: string, content?: string, property?: string, httpEquiv?: string, charset?: string, node: Artifacts.NodeDetails}>;
   /** Information on all scripts in the page. */
   Scripts: Artifacts.Script[];
+  /** The primary trace taken over the entire run. */
+  Trace: Trace;
   /** The dimensions and devicePixelRatio of the loaded viewport. */
   ViewportDimensions: Artifacts.ViewportDimensions;
 }
@@ -110,8 +111,6 @@ export interface GathererArtifacts extends PublicGathererArtifacts {
   CacheContents: string[];
   /** CSS coverage information for styles used by page's final state. */
   CSSUsage: {rules: Crdp.CSS.RuleUsage[], stylesheets: Artifacts.CSSStyleSheetInfo[]};
-  /** The primary log of devtools protocol activity. */
-  DevtoolsLog: DevtoolsLog;
   /** The log of devtools protocol activity if there was a page load error and Chrome navigated to a `chrome-error://` page. */
   DevtoolsLogError: DevtoolsLog;
   /** Information on the document's doctype(or null if not present), specifically the name, publicId, and systemId.
@@ -121,12 +120,12 @@ export interface GathererArtifacts extends PublicGathererArtifacts {
   DOMStats: Artifacts.DOMStats;
   /** Information on poorly sized font usage and the text affected by it. */
   FontSize: Artifacts.FontSize;
+  /** All the iframe elements in the page. */
+  IFrameElements: Artifacts.IFrameElement[];
   /** All the input elements, including associated form and label elements. */
   Inputs: {inputs: Artifacts.InputElement[]; forms: Artifacts.FormElement[]; labels: Artifacts.LabelElement[]};
   /** Screenshot of the entire page (rather than just the above the fold content). */
   FullPageScreenshot: LHResult.FullPageScreenshot | null;
-  /** Information about event listeners registered on the global object. */
-  GlobalListeners: Array<Artifacts.GlobalListener>;
   /** The issues surfaced in the devtools Issues panel */
   InspectorIssues: Artifacts.InspectorIssues;
   /** JS coverage information for code used during audit. Keyed by script id. */
@@ -148,12 +147,6 @@ export interface GathererArtifacts extends PublicGathererArtifacts {
   SourceMaps: Array<Artifacts.SourceMap>;
   /** Information on detected tech stacks (e.g. JS libraries) used by the page. */
   Stacks: Artifacts.DetectedStack[];
-  /** Information on <script> and <link> tags blocking first paint. */
-  TagsBlockingFirstPaint: Artifacts.TagBlockingFirstPaint[];
-  /** Information about tap targets including their position and size. */
-  TapTargets: Artifacts.TapTarget[];
-  /** The primary trace taken over the entire run. */
-  Trace: Trace;
   /** The trace if there was a page load error and Chrome navigated to a `chrome-error://` page. */
   TraceError: Trace;
   /** Elements associated with metrics (ie: Largest Contentful Paint element). */
@@ -189,6 +182,8 @@ declare module Artifacts {
     finalDisplayedUrl: string;
   }
 
+  type Rect = AuditDetails.Rect;
+
   interface NodeDetails {
     lhId: string,
     devtoolsNodePath: string,
@@ -294,19 +289,6 @@ declare module Artifacts {
     content?: string;
   }
 
-  interface ScriptElement {
-    type: string | null
-    src: string | null
-    /** The `id` property of the script element; null if it had no `id` or if `source` is 'network'. */
-    id: string | null
-    async: boolean
-    defer: boolean
-    /** Details for node in DOM for the script element */
-    node: NodeDetails | null
-    /** Where the script was discovered, either in the head, the body, or network records. */
-    source: 'head'|'body'|'network'
-  }
-
   /** @see https://sourcemaps.info/spec.html#h.qz3o9nc69um5 */
   type RawSourceMap = {
     /** File version and must be a positive integer. */
@@ -519,27 +501,6 @@ declare module Artifacts {
     resourceSize: number;
   }
 
-  interface TagBlockingFirstPaint {
-    startTime: number;
-    endTime: number;
-    transferSize: number;
-    tag: {
-      tagName: 'LINK'|'SCRIPT';
-      /** The value of `HTMLLinkElement.href` or `HTMLScriptElement.src`. */
-      url: string;
-      /** A record of when changes to the `HTMLLinkElement.media` attribute occurred and if the new media type matched the page. */
-      mediaChanges?: Array<{href: string, media: string, msSinceHTMLEnd: number, matches: boolean}>;
-    };
-  }
-
-  type Rect = AuditDetails.Rect;
-
-  interface TapTarget {
-    node: NodeDetails;
-    href: string;
-    clientRects: Rect[];
-  }
-
   interface TraceElement {
     traceEventType: 'largest-contentful-paint'|'layout-shift'|'animation'|'responsiveness';
     node: NodeDetails;
@@ -548,7 +509,10 @@ declare module Artifacts {
     type?: string;
   }
 
-  type TraceEngineResult = TraceEngine.Handlers.Types.TraceParseData;
+  interface TraceEngineResult {
+    data: TraceEngine.Handlers.Types.EnabledHandlerDataWithMeta<EnabledHandlers>;
+    insights: TraceEngine.Insights.Types.TraceInsightData<EnabledHandlers>;
+  }
 
   interface TraceEngineRootCauses {
     layoutShifts: Record<number, LayoutShiftRootCausesData>;
@@ -811,18 +775,6 @@ declare module Artifacts {
     node: NodeDetails;
   }
 
-  /** Information about an event listener registered on the global object. */
-  interface GlobalListener {
-    /** Event listener type, limited to those events currently of interest. */
-    type: 'pagehide'|'unload'|'visibilitychange';
-    /** The DevTools protocol script identifier. */
-    scriptId: string;
-    /** Line number in the script (0-based). */
-    lineNumber: number;
-    /** Column number in the script (0-based). */
-    columnNumber: number;
-  }
-
   /** Describes a generic console message. */
   interface BaseConsoleMessage {
     /**

package/types/lhr/lhr.d.ts

@@ -71,8 +71,6 @@ declare module Result {
     networkUserAgent: string;
     /** The benchmark index number that indicates rough device class. */
     benchmarkIndex: number;
-    /** Many benchmark indexes. */
-    benchmarkIndexes?: number[];
     /** The version of libraries with which these results were generated. Ex: axe-core. */
     credits?: Record<string, string|undefined>,
   }

package/core/gather/gatherers/dobetterweb/tags-blocking-first-paint.d.ts

@@ -1,47 +0,0 @@
-export default TagsBlockingFirstPaint;
-export type MediaChange = {
-    href: string;
-    media: string;
-    msSinceHTMLEnd: number;
-    matches: boolean;
-};
-export type LinkTag = {
-    tagName: 'LINK';
-    url: string;
-    href: string;
-    rel: string;
-    media: string;
-    disabled: boolean;
-    mediaChanges: Array<MediaChange>;
-};
-export type ScriptTag = {
-    tagName: 'SCRIPT';
-    url: string;
-    src: string;
-};
-declare class TagsBlockingFirstPaint extends BaseGatherer {
-    /**
-     * @param {Array<LH.Artifacts.NetworkRequest>} networkRecords
-     * @return {Map<string, LH.Artifacts.NetworkRequest>}
-     */
-    static _filteredAndIndexedByUrl(networkRecords: Array<LH.Artifacts.NetworkRequest>): Map<string, LH.Artifacts.NetworkRequest>;
-    /**
-     * @param {LH.Gatherer.Driver} driver
-     * @param {Array<LH.Artifacts.NetworkRequest>} networkRecords
-     * @return {Promise<Array<LH.Artifacts.TagBlockingFirstPaint>>}
-     */
-    static findBlockingTags(driver: LH.Gatherer.Driver, networkRecords: Array<LH.Artifacts.NetworkRequest>): Promise<Array<LH.Artifacts.TagBlockingFirstPaint>>;
-    /** @type {LH.Gatherer.GathererMeta<'DevtoolsLog'>} */
-    meta: LH.Gatherer.GathererMeta<'DevtoolsLog'>;
-    /**
-     * @param {LH.Gatherer.Context} context
-     */
-    startSensitiveInstrumentation(context: LH.Gatherer.Context): Promise<void>;
-    /**
-     * @param {LH.Gatherer.Context<'DevtoolsLog'>} context
-     * @return {Promise<LH.Artifacts['TagsBlockingFirstPaint']>}
-     */
-    getArtifact(context: LH.Gatherer.Context<'DevtoolsLog'>): Promise<LH.Artifacts['TagsBlockingFirstPaint']>;
-}
-import BaseGatherer from '../../base-gatherer.js';
-//# sourceMappingURL=tags-blocking-first-paint.d.ts.map

package/core/gather/gatherers/dobetterweb/tags-blocking-first-paint.js

@@ -1,233 +0,0 @@
-/**
- * @license
- * Copyright 2016 Google LLC
- * SPDX-License-Identifier: Apache-2.0
- */
-/**
- * @fileoverview
- *   Identifies stylesheets, HTML Imports, and scripts that potentially block
- *   the first paint of the page by running several scripts in the page context.
- *   Candidate blocking tags are collected by querying for all script tags in
- *   the head of the page and all link tags that are either matching media
- *   stylesheets or non-async HTML imports. These are then compared to the
- *   network requests to ensure they were initiated by the parser and not
- *   injected with script. To avoid false positives from strategies like
- *   (http://filamentgroup.github.io/loadCSS/test/preload.html), a separate
- *   script is run to flag all links that at one point were rel=preload.
- */
-
-
-import {NetworkRecords} from '../../../computed/network-records.js';
-import DevtoolsLog from '../devtools-log.js';
-import BaseGatherer from '../../base-gatherer.js';
-
-/* global document, window, HTMLLinkElement, SVGScriptElement */
-
-/** @typedef {{href: string, media: string, msSinceHTMLEnd: number, matches: boolean}} MediaChange */
-/** @typedef {{tagName: 'LINK', url: string, href: string, rel: string, media: string, disabled: boolean, mediaChanges: Array<MediaChange>}} LinkTag */
-/** @typedef {{tagName: 'SCRIPT', url: string, src: string}} ScriptTag */
-
-/* c8 ignore start */
-function installMediaListener() {
-  // @ts-expect-error - inserted in page to track media changes.
-  window.___linkMediaChanges = [];
-  Object.defineProperty(HTMLLinkElement.prototype, 'media', {
-    set: function(val) {
-      /** @type {MediaChange} */
-      const mediaChange = {
-        href: this.href,
-        media: val,
-        msSinceHTMLEnd: Date.now() - performance.timing.responseEnd,
-        matches: window.matchMedia(val).matches,
-      };
-      // @ts-expect-error - `___linkMediaChanges` created above.
-      window.___linkMediaChanges.push(mediaChange);
-
-      this.setAttribute('media', val);
-    },
-  });
-}
-/* c8 ignore stop */
-
-/**
- * @return {Promise<Array<LinkTag | ScriptTag>>}
- */
-/* c8 ignore start */
-async function collectTagsThatBlockFirstPaint() {
-  /** @type {Array<MediaChange>} */
-  // @ts-expect-error - `___linkMediaChanges` created in `installMediaListener`.
-  const linkMediaChanges = window.___linkMediaChanges;
-
-  try {
-    /** @type {Array<LinkTag>} */
-    const linkTags = [...document.querySelectorAll('link')]
-      .filter(linkTag => {
-        // Ignore malformed links with no href (e.g. `<link rel="stylesheet" href="">`)
-        // The resolved `linkTag.href` will be the main document, but the main document
-        // should never be render blocking.
-        if (!linkTag.getAttribute('href')) return false;
-
-        // Filter stylesheet/HTML imports that block rendering.
-        // https://www.igvita.com/2012/06/14/debunking-responsive-css-performance-myths/
-        // https://www.w3.org/TR/html-imports/#dfn-import-async-attribute
-        const blockingStylesheet = linkTag.rel === 'stylesheet' &&
-          window.matchMedia(linkTag.media).matches && !linkTag.disabled;
-        const blockingImport = linkTag.rel === 'import' && !linkTag.hasAttribute('async');
-        return blockingStylesheet || blockingImport;
-      })
-      .map(tag => {
-        return {
-          tagName: 'LINK',
-          url: tag.href,
-          href: tag.href,
-          rel: tag.rel,
-          media: tag.media,
-          disabled: tag.disabled,
-          mediaChanges: linkMediaChanges.filter(item => item.href === tag.href),
-        };
-      });
-
-    /** @type {Array<ScriptTag>} */
-    const scriptTags = [...document.querySelectorAll('head script[src]')]
-      .filter(/** @return {scriptTag is HTMLScriptElement} */ scriptTag => {
-        // SVGScriptElement can't appear in <head> (it'll be kicked to <body>), but keep tsc happy.
-        // https://html.spec.whatwg.org/multipage/semantics.html#the-head-element
-        if (scriptTag instanceof SVGScriptElement) return false;
-
-        return (
-          !scriptTag.hasAttribute('async') &&
-          !scriptTag.hasAttribute('defer') &&
-          !/^data:/.test(scriptTag.src) &&
-          !/^blob:/.test(scriptTag.src) &&
-          scriptTag.getAttribute('type') !== 'module'
-        );
-      })
-      .map(tag => {
-        return {
-          tagName: 'SCRIPT',
-          url: tag.src,
-          src: tag.src,
-        };
-      });
-
-    return [...linkTags, ...scriptTags];
-  } catch (e) {
-    const friendly = 'Unable to gather Scripts/Stylesheets/HTML Imports on the page';
-    throw new Error(`${friendly}: ${e.message}`);
-  }
-}
-/* c8 ignore stop */
-
-class TagsBlockingFirstPaint extends BaseGatherer {
-  /** @type {LH.Gatherer.GathererMeta<'DevtoolsLog'>} */
-  meta = {
-    supportedModes: ['navigation'],
-    dependencies: {DevtoolsLog: DevtoolsLog.symbol},
-  };
-
-  /**
-   * @param {Array<LH.Artifacts.NetworkRequest>} networkRecords
-   * @return {Map<string, LH.Artifacts.NetworkRequest>}
-   */
-  static _filteredAndIndexedByUrl(networkRecords) {
-    /** @type {Map<string, LH.Artifacts.NetworkRequest>} */
-    const result = new Map();
-
-    for (const record of networkRecords) {
-      if (!record.finished) continue;
-
-      const isParserGenerated = record.initiator.type === 'parser';
-      // A stylesheet only blocks script if it was initiated by the parser
-      // https://html.spec.whatwg.org/multipage/semantics.html#interactions-of-styling-and-scripting
-      const isParserScriptOrStyle = /(css|script)/.test(record.mimeType) && isParserGenerated;
-      const isFailedRequest = record.failed;
-      const isHtml = record.mimeType && record.mimeType.includes('html');
-
-      // Filter stylesheet, javascript, and html import mimetypes.
-      // Include 404 scripts/links generated by the parser because they are likely blocking.
-      if (isHtml || isParserScriptOrStyle || (isFailedRequest && isParserGenerated)) {
-        result.set(record.url, record);
-      }
-    }
-
-    return result;
-  }
-
-  /**
-   * @param {LH.Gatherer.Driver} driver
-   * @param {Array<LH.Artifacts.NetworkRequest>} networkRecords
-   * @return {Promise<Array<LH.Artifacts.TagBlockingFirstPaint>>}
-   */
-  static async findBlockingTags(driver, networkRecords) {
-    const firstRequestEndTime = networkRecords.reduce(
-      (min, record) => Math.min(min, record.networkEndTime),
-      Infinity
-    );
-    const tags = await driver.executionContext.evaluate(collectTagsThatBlockFirstPaint, {args: []});
-    const requests = TagsBlockingFirstPaint._filteredAndIndexedByUrl(networkRecords);
-
-    /** @type {Array<LH.Artifacts.TagBlockingFirstPaint>} */
-    const result = [];
-    for (const tag of tags) {
-      const request = requests.get(tag.url);
-      if (!request || request.isLinkPreload) continue;
-
-      let endTime = request.networkEndTime;
-      let mediaChanges;
-
-      if (tag.tagName === 'LINK') {
-        // Even if the request was initially blocking or appeared to be blocking once the
-        // page was loaded, the media attribute could have been changed during load, capping the
-        // amount of time it was render blocking. See https://github.com/GoogleChrome/lighthouse/issues/2832.
-        const timesResourceBecameNonBlocking = tag.mediaChanges
-          .filter(change => !change.matches)
-          .map(change => change.msSinceHTMLEnd);
-        if (timesResourceBecameNonBlocking.length > 0) {
-          const earliestNonBlockingTime = Math.min(...timesResourceBecameNonBlocking);
-          const lastTimeResourceWasBlocking = Math.max(
-            request.networkRequestTime,
-            firstRequestEndTime + earliestNonBlockingTime / 1000
-          );
-          endTime = Math.min(endTime, lastTimeResourceWasBlocking);
-        }
-
-        mediaChanges = tag.mediaChanges;
-      }
-
-      const {tagName, url} = tag;
-
-      result.push({
-        tag: {tagName, url, mediaChanges},
-        transferSize: request.transferSize,
-        startTime: request.networkRequestTime,
-        endTime,
-      });
-
-      // Prevent duplicates from showing up again
-      requests.delete(tag.url);
-    }
-
-    return result;
-  }
-
-  /**
-   * @param {LH.Gatherer.Context} context
-   */
-  async startSensitiveInstrumentation(context) {
-    const {executionContext} = context.driver;
-    // Don't return return value of `evaluateOnNewDocument`.
-    await executionContext.evaluateOnNewDocument(installMediaListener, {args: []});
-  }
-
-  /**
-   * @param {LH.Gatherer.Context<'DevtoolsLog'>} context
-   * @return {Promise<LH.Artifacts['TagsBlockingFirstPaint']>}
-   */
-  async getArtifact(context) {
-    const devtoolsLog = context.dependencies.DevtoolsLog;
-    const networkRecords = await NetworkRecords.request(devtoolsLog, context);
-    return TagsBlockingFirstPaint.findBlockingTags(context.driver, networkRecords);
-  }
-}
-
-export default TagsBlockingFirstPaint;