lighthouse 12.8.2 → 13.0.0

package/core/audits/dobetterweb/uses-http2.js

@@ -1,276 +0,0 @@
-/**
- * @license
- * Copyright 2016 Google LLC
- * SPDX-License-Identifier: Apache-2.0
- */
-
-/**
- * @fileoverview Audit a page to ensure that resource loaded over its own
- * origin are over the http/2 protocol.
- */
-
-import {Audit} from '../audit.js';
-import {EntityClassification} from '../../computed/entity-classification.js';
-import UrlUtils from '../../lib/url-utils.js';
-import {NetworkRequest} from '../../lib/network-request.js';
-import {NetworkRecords} from '../../computed/network-records.js';
-import {LoadSimulator} from '../../computed/load-simulator.js';
-import {LanternLargestContentfulPaint} from '../../computed/metrics/lantern-largest-contentful-paint.js';
-import {LanternFirstContentfulPaint} from '../../computed/metrics/lantern-first-contentful-paint.js';
-import * as i18n from '../../lib/i18n/i18n.js';
-
-const UIStrings = {
-  /** Imperative title of a Lighthouse audit that tells the user to enable HTTP/2. This is displayed in a list of audit titles that Lighthouse generates. */
-  title: 'Use HTTP/2',
-  /** Description of a Lighthouse audit that tells the user why they should use HTTP/2. This is displayed after a user expands the section to see more. No character length limits. The last sentence starting with 'Learn' becomes link text to additional documentation. */
-  description: 'HTTP/2 offers many benefits over HTTP/1.1, including binary headers and ' +
-      'multiplexing. [Learn more about HTTP/2](https://developer.chrome.com/docs/lighthouse/best-practices/uses-http2/).',
-  /** [ICU Syntax] Label identifying the number of network requests that were not served with HTTP/2. */
-  displayValue: `{itemCount, plural,
-    =1 {1 request not served via HTTP/2}
-    other {# requests not served via HTTP/2}
-    }`,
-  /**  Label for a column in a data table; entries in the column will be the HTTP Protocol used to make a network request. */
-  columnProtocol: 'Protocol',
-};
-
-const str_ = i18n.createIcuMessageFn(import.meta.url, UIStrings);
-
-/** @type {Set<LH.Artifacts.NetworkRequest['resourceType']>} */
-const STATIC_RESOURCE_TYPES = new Set([
-  NetworkRequest.TYPES.Document,
-  NetworkRequest.TYPES.Font,
-  NetworkRequest.TYPES.Image,
-  NetworkRequest.TYPES.Stylesheet,
-  NetworkRequest.TYPES.Script,
-  NetworkRequest.TYPES.Media,
-]);
-
-class UsesHTTP2Audit extends Audit {
-  /**
-   * @return {LH.Audit.Meta}
-   */
-  static get meta() {
-    return {
-      id: 'uses-http2',
-      title: str_(UIStrings.title),
-      description: str_(UIStrings.description),
-      scoreDisplayMode: Audit.SCORING_MODES.METRIC_SAVINGS,
-      guidanceLevel: 3,
-      supportedModes: ['timespan', 'navigation'],
-      requiredArtifacts: ['URL', 'DevtoolsLog', 'Trace', 'GatherContext', 'SourceMaps'],
-    };
-  }
-
-  /**
-   * Computes the estimated effect of all results being converted to http/2 on the provided graph.
-   *
-   * @param {Array<{url: string}>} results
-   * @param {LH.Gatherer.Simulation.GraphNode} graph
-   * @param {LH.Gatherer.Simulation.Simulator} simulator
-   * @param {{label?: string}=} options
-   * @return {{savings: number, simulationBefore: LH.Gatherer.Simulation.Result, simulationAfter: LH.Gatherer.Simulation.Result}}
-   */
-  static computeWasteWithGraph(results, graph, simulator, options) {
-    options = Object.assign({label: ''}, options);
-    const beforeLabel = `${this.meta.id}-${options.label}-before`;
-    const afterLabel = `${this.meta.id}-${options.label}-after`;
-
-    const urlsToChange = new Set(results.map(result => result.url));
-    const simulationBefore = simulator.simulate(graph, {label: beforeLabel});
-
-    // Update all the protocols to reflect implementing our recommendations
-    /** @type {Map<string, string>} */
-    const originalProtocols = new Map();
-    graph.traverse(node => {
-      if (node.type !== 'network') return;
-      if (!urlsToChange.has(node.request.url)) return;
-
-      originalProtocols.set(node.request.requestId, node.request.protocol);
-      node.request.protocol = 'h2';
-    });
-
-    const simulationAfter = simulator.simulate(graph, {label: afterLabel});
-
-    // Restore the original protocol after we've done our simulation
-    graph.traverse(node => {
-      if (node.type !== 'network') return;
-      const originalProtocol = originalProtocols.get(node.request.requestId);
-      if (originalProtocol === undefined) return;
-      node.request.protocol = originalProtocol;
-    });
-
-    const savings = simulationBefore.timeInMs - simulationAfter.timeInMs;
-
-    return {
-      // Round waste to nearest 10ms
-      savings: Math.round(Math.max(savings, 0) / 10) * 10,
-      simulationBefore,
-      simulationAfter,
-    };
-  }
-
-  /**
-   * Determines whether a network request is a "static resource" that would benefit from H2 multiplexing.
-   * XHRs, tracking pixels, etc generally don't benefit as much because they aren't requested en-masse
-   * for the same origin at the exact same time.
-   *
-   * @param {LH.Artifacts.NetworkRequest} networkRequest
-   * @param {LH.Artifacts.EntityClassification} classifiedEntities
-   * @return {boolean}
-   */
-  static isMultiplexableStaticAsset(networkRequest, classifiedEntities) {
-    if (!STATIC_RESOURCE_TYPES.has(networkRequest.resourceType)) return false;
-
-    // Resources from third-parties that are less than 100 bytes are usually tracking pixels, not actual resources.
-    // They can masquerade as static types though (gifs, documents, etc)
-    if (networkRequest.resourceSize < 100) {
-      const entity = classifiedEntities.entityByUrl.get(networkRequest.url);
-      if (entity) {
-        // Third-party assets are multiplexable in their first-party context.
-        if (classifiedEntities.firstParty?.name === entity.name) return true;
-        // Skip recognizable third-parties' requests.
-        if (!entity.isUnrecognized) return false;
-      }
-    }
-
-    return true;
-  }
-
-  /**
-   * Determine the set of resources that aren't HTTP/2 but should be.
-   * We're a little conservative about what we surface for a few reasons:
-   *
-   *    - The simulator approximation of HTTP/2 is a little more generous than reality.
-   *    - There's a bit of debate surrounding HTTP/2 due to its worse performance in environments with high packet loss.**
-   *    - It's something that you'd have absolutely zero control over with a third-party (can't defer to fix it for example).
-   *
-   * Therefore, we only surface requests that were...
-   *
-   *    - Served over HTTP/1.1 or earlier
-   *    - Served over an origin that serves at least 6 static asset requests
-   *      (if there aren't more requests than browser's max/host, multiplexing isn't as big a deal)
-   *    - Not served on localhost (h2 is a pain to deal with locally & and CI)
-   *
-   * ** = https://news.ycombinator.com/item?id=19086639
-   *      https://www.twilio.com/blog/2017/10/http2-issues.html
-   *      https://www.cachefly.com/http-2-is-not-a-magic-bullet/
-   *
-   * @param {Array<LH.Artifacts.NetworkRequest>} networkRecords
-   * @param {LH.Artifacts.EntityClassification} classifiedEntities
-   * @return {Array<{url: string, protocol: string}>}
-   */
-  static determineNonHttp2Resources(networkRecords, classifiedEntities) {
-    /** @type {Array<{url: string, protocol: string}>} */
-    const nonHttp2Resources = [];
-
-    /** @type {Set<string>} */
-    const seenURLs = new Set();
-    /** @type {Map<string, Array<LH.Artifacts.NetworkRequest>>} */
-    const groupedByOrigin = new Map();
-    for (const record of networkRecords) {
-      if (!UsesHTTP2Audit.isMultiplexableStaticAsset(record, classifiedEntities)) continue;
-      if (UrlUtils.isLikeLocalhost(record.parsedURL.host)) continue;
-      const existing = groupedByOrigin.get(record.parsedURL.securityOrigin) || [];
-      existing.push(record);
-      groupedByOrigin.set(record.parsedURL.securityOrigin, existing);
-    }
-
-    for (const record of networkRecords) {
-      // Skip duplicates.
-      if (seenURLs.has(record.url)) continue;
-      // Check if record is not served through the service worker, servicer worker uses http/1.1 as a protocol.
-      // These can generate false positives (bug: https://github.com/GoogleChrome/lighthouse/issues/7158).
-      if (record.fetchedViaServiceWorker) continue;
-      // Test the protocol to see if it was http/1.1.
-      const isOldHttp = /HTTP\/[01][.\d]?/i.test(record.protocol);
-      if (!isOldHttp) continue;
-      // Check if the origin has enough requests to bother flagging.
-      const group = groupedByOrigin.get(record.parsedURL.securityOrigin) || [];
-      if (group.length < 6) continue;
-
-      seenURLs.add(record.url);
-      nonHttp2Resources.push({protocol: record.protocol, url: record.url});
-    }
-
-    return nonHttp2Resources;
-  }
-
-  /**
-   * @param {LH.Artifacts} artifacts
-   * @param {LH.Audit.Context} context
-   * @return {Promise<LH.Audit.Product>}
-   */
-  static async audit(artifacts, context) {
-    const devtoolsLog = artifacts.DevtoolsLog;
-    const URL = artifacts.URL;
-    const networkRecords = await NetworkRecords.request(devtoolsLog, context);
-    const classifiedEntities = await EntityClassification.request({URL, devtoolsLog}, context);
-    const resources = UsesHTTP2Audit.determineNonHttp2Resources(networkRecords, classifiedEntities);
-
-    let displayValue;
-    if (resources.length > 0) {
-      displayValue = str_(UIStrings.displayValue, {itemCount: resources.length});
-    }
-
-    // TODO: Compute actual savings for timespan mode.
-    if (artifacts.GatherContext.gatherMode === 'timespan') {
-      /** @type {LH.Audit.Details.Table['headings']} */
-      const headings = [
-        {key: 'url', valueType: 'url', label: str_(i18n.UIStrings.columnURL)},
-        {key: 'protocol', valueType: 'text', label: str_(UIStrings.columnProtocol)},
-      ];
-
-      const details = Audit.makeTableDetails(headings, resources);
-
-      return {
-        displayValue,
-        score: resources.length ? 0 : 1,
-        details,
-      };
-    }
-
-    const settings = context?.settings || {};
-    const simulatorOptions = {
-      devtoolsLog,
-      settings,
-    };
-    const simulator = await LoadSimulator.request(simulatorOptions, context);
-    const metricComputationInput = Audit.makeMetricComputationDataInput(artifacts, context);
-
-    const {
-      pessimisticGraph: fcpGraph,
-    } = await LanternFirstContentfulPaint.request(metricComputationInput, context);
-    const {
-      pessimisticGraph: lcpGraph,
-    } = await LanternLargestContentfulPaint.request(metricComputationInput, context);
-
-    const wasteFcp =
-      UsesHTTP2Audit.computeWasteWithGraph(resources,
-        fcpGraph, simulator, {label: 'fcp'});
-    const wasteLcp =
-      UsesHTTP2Audit.computeWasteWithGraph(resources,
-        lcpGraph, simulator, {label: 'lcp'});
-
-    /** @type {LH.Audit.Details.Opportunity['headings']} */
-    const headings = [
-      {key: 'url', valueType: 'url', label: str_(i18n.UIStrings.columnURL)},
-      {key: 'protocol', valueType: 'text', label: str_(UIStrings.columnProtocol)},
-    ];
-
-    const details = Audit.makeOpportunityDetails(headings, resources,
-      {overallSavingsMs: wasteLcp.savings});
-
-    return {
-      displayValue,
-      numericValue: wasteLcp.savings,
-      numericUnit: 'millisecond',
-      score: resources.length ? 0 : 1,
-      details,
-      metricSavings: {LCP: wasteLcp.savings, FCP: wasteFcp.savings},
-    };
-  }
-}
-
-export default UsesHTTP2Audit;
-export {UIStrings};

package/core/audits/dobetterweb/uses-passive-event-listeners.d.ts

@@ -1,16 +0,0 @@
-export default PassiveEventsAudit;
-declare class PassiveEventsAudit extends ViolationAudit {
-    /**
-     * @param {LH.Artifacts} artifacts
-     * @param {LH.Audit.Context} context
-     * @return {Promise<LH.Audit.Product>}
-     */
-    static audit(artifacts: LH.Artifacts, context: LH.Audit.Context): Promise<LH.Audit.Product>;
-}
-export namespace UIStrings {
-    let title: string;
-    let failureTitle: string;
-    let description: string;
-}
-import ViolationAudit from '../violation-audit.js';
-//# sourceMappingURL=uses-passive-event-listeners.d.ts.map

package/core/audits/dobetterweb/uses-passive-event-listeners.js

@@ -1,69 +0,0 @@
-/**
- * @license
- * Copyright 2016 Google LLC
- * SPDX-License-Identifier: Apache-2.0
- */
-
-/**
- * @fileoverview Audit a page to see if it is using passive event listeners on
- * scroll-blocking touch and wheel event listeners.
- */
-
-
-import ViolationAudit from '../violation-audit.js';
-import * as i18n from '../../lib/i18n/i18n.js';
-
-const UIStrings = {
-  /** Title of a Lighthouse audit that provides detail on the page's use of passive event listeners used to improve the scrolling performance of the page. This descriptive title is shown to users when the page does use passive listeners. */
-  title: 'Uses passive listeners to improve scrolling performance',
-  /** Title of a Lighthouse audit that provides detail on the page's use of passive event listeners used to improve the scrolling performance of the page. This descriptive title is shown to users when the page does not use passive listeners. */
-  failureTitle: 'Does not use passive listeners to improve scrolling performance',
-  /** Description of a Lighthouse audit that tells the user why they should use passive event listeners on the page. This is displayed after a user expands the section to see more. No character length limits. The last sentence starting with 'Learn' becomes link text to additional documentation. */
-  description: 'Consider marking your touch and wheel event listeners as `passive` ' +
-      'to improve your page\'s scroll performance. ' +
-      '[Learn more about adopting passive event listeners](https://developer.chrome.com/docs/lighthouse/best-practices/uses-passive-event-listeners/).',
-};
-
-const str_ = i18n.createIcuMessageFn(import.meta.url, UIStrings);
-
-class PassiveEventsAudit extends ViolationAudit {
-  /**
-   * @return {LH.Audit.Meta}
-   */
-  static get meta() {
-    return {
-      id: 'uses-passive-event-listeners',
-      title: str_(UIStrings.title),
-      failureTitle: str_(UIStrings.failureTitle),
-      description: str_(UIStrings.description),
-      guidanceLevel: 3,
-      requiredArtifacts: ['ConsoleMessages', 'SourceMaps', 'Scripts'],
-      scoreDisplayMode: ViolationAudit.SCORING_MODES.METRIC_SAVINGS,
-    };
-  }
-
-  /**
-   * @param {LH.Artifacts} artifacts
-   * @param {LH.Audit.Context} context
-   * @return {Promise<LH.Audit.Product>}
-   */
-  static async audit(artifacts, context) {
-    const results =
-      await ViolationAudit.getViolationResults(artifacts, context, /passive event listener/);
-
-    /** @type {LH.Audit.Details.Table['headings']} */
-    const headings = [
-      {key: 'source', valueType: 'source-location', label: str_(i18n.UIStrings.columnSource)},
-    ];
-    // TODO(bckenny): see TODO in geolocation-on-start
-    const details = ViolationAudit.makeTableDetails(headings, results);
-
-    return {
-      score: Number(results.length === 0),
-      details,
-    };
-  }
-}
-
-export default PassiveEventsAudit;
-export {UIStrings};

package/core/audits/font-display.d.ts

@@ -1,32 +0,0 @@
-export default FontDisplay;
-declare class FontDisplay extends Audit {
-    /**
-     * @param {LH.Artifacts} artifacts
-     * @param {RegExp} passingFontDisplayRegex
-     * @return {{passingURLs: Set<string>, failingURLs: Set<string>}}
-     */
-    static findFontDisplayDeclarations(artifacts: LH.Artifacts, passingFontDisplayRegex: RegExp): {
-        passingURLs: Set<string>;
-        failingURLs: Set<string>;
-    };
-    /**
-     * Some pages load many fonts we can't check, so dedupe on origin.
-     * @param {Array<string>} warningUrls
-     * @return {Array<LH.IcuMessage>}
-     */
-    static getWarningsForFontUrls(warningUrls: Array<string>): Array<LH.IcuMessage>;
-    /**
-     * @param {LH.Artifacts} artifacts
-     * @param {LH.Audit.Context} context
-     * @return {Promise<LH.Audit.Product>}
-     */
-    static audit(artifacts: LH.Artifacts, context: LH.Audit.Context): Promise<LH.Audit.Product>;
-}
-export namespace UIStrings {
-    let title: string;
-    let failureTitle: string;
-    let description: string;
-    let undeclaredFontOriginWarning: string;
-}
-import { Audit } from './audit.js';
-//# sourceMappingURL=font-display.d.ts.map

package/core/audits/font-display.js

@@ -1,195 +0,0 @@
-/**
- * @license
- * Copyright 2017 Google LLC
- * SPDX-License-Identifier: Apache-2.0
- */
-
-import {Audit} from './audit.js';
-import UrlUtils from '../lib/url-utils.js';
-import * as i18n from '../lib/i18n/i18n.js';
-import {Sentry} from '../lib/sentry.js';
-import {NetworkRecords} from '../computed/network-records.js';
-
-const PASSING_FONT_DISPLAY_REGEX = /^(block|fallback|optional|swap)$/;
-const CSS_URL_REGEX = /url\((.*?)\)/;
-const CSS_URL_GLOBAL_REGEX = new RegExp(CSS_URL_REGEX, 'g');
-
-const UIStrings = {
-  /** Title of a diagnostic audit that provides detail on if all the text on a webpage was visible while the page was loading its webfonts. This descriptive title is shown to users when the amount is acceptable and no user action is required. */
-  title: 'All text remains visible during webfont loads',
-  /** Title of a diagnostic audit that provides detail on the load of the page's webfonts. Often the text is invisible for seconds before the webfont resource is loaded. This imperative title is shown to users when there is a significant amount of execution time that could be reduced. */
-  failureTitle: 'Ensure text remains visible during webfont load',
-  /** Description of a Lighthouse audit that tells the user *why* they should use the font-display CSS feature. This is displayed after a user expands the section to see more. No character length limits. The last sentence starting with 'Learn' becomes link text to additional documentation. */
-  description:
-    'Leverage the `font-display` CSS feature to ensure text is user-visible while ' +
-    'webfonts are loading. ' +
-    '[Learn more about `font-display`](https://developer.chrome.com/docs/lighthouse/performance/font-display/).',
-  /**
-   * @description [ICU Syntax] A warning message that is shown when Lighthouse couldn't automatically check some of the page's fonts, telling the user that they will need to manually check the fonts coming from a certain URL origin.
-   * @example {https://font.cdn.com/} fontOrigin
-   */
-  undeclaredFontOriginWarning:
-    '{fontCountForOrigin, plural, ' +
-    // eslint-disable-next-line max-len
-    '=1 {Lighthouse was unable to automatically check the `font-display` value for the origin {fontOrigin}.} ' +
-    // eslint-disable-next-line max-len
-    'other {Lighthouse was unable to automatically check the `font-display` values for the origin {fontOrigin}.}}',
-};
-
-const str_ = i18n.createIcuMessageFn(import.meta.url, UIStrings);
-
-class FontDisplay extends Audit {
-  /**
-   * @return {LH.Audit.Meta}
-   */
-  static get meta() {
-    return {
-      id: 'font-display',
-      title: str_(UIStrings.title),
-      failureTitle: str_(UIStrings.failureTitle),
-      description: str_(UIStrings.description),
-      supportedModes: ['navigation'],
-      guidanceLevel: 3,
-      requiredArtifacts: ['DevtoolsLog', 'Stylesheets', 'URL'],
-      scoreDisplayMode: Audit.SCORING_MODES.METRIC_SAVINGS,
-    };
-  }
-
-  /**
-   * @param {LH.Artifacts} artifacts
-   * @param {RegExp} passingFontDisplayRegex
-   * @return {{passingURLs: Set<string>, failingURLs: Set<string>}}
-   */
-  static findFontDisplayDeclarations(artifacts, passingFontDisplayRegex) {
-    /** @type {Set<string>} */
-    const passingURLs = new Set();
-    /** @type {Set<string>} */
-    const failingURLs = new Set();
-
-    // Go through all the stylesheets to find all @font-face declarations
-    for (const stylesheet of artifacts.Stylesheets) {
-      // Eliminate newlines so we can more easily scan through with a regex
-      const newlinesStripped = stylesheet.content.replace(/(\r|\n)+/g, ' ');
-      // Find the @font-faces
-      const fontFaceDeclarations = newlinesStripped.match(/@font-face\s*{(.*?)}/g) || [];
-      // Go through all the @font-face declarations to find a declared `font-display: ` property
-      for (const declaration of fontFaceDeclarations) {
-        // We'll try to find the URL it's referencing.
-        const rawFontURLs = declaration.match(CSS_URL_GLOBAL_REGEX);
-        // If no URLs, we can't really do anything; bail
-        if (!rawFontURLs) continue;
-        // Find the font-display value by matching a single token, optionally surrounded by whitespace,
-        // followed either by a semicolon or the end of a block.
-        const fontDisplayMatch = declaration.match(/font-display\s*:\s*(\w+)\s*(;|\})/);
-        const rawFontDisplay = fontDisplayMatch?.[1] || '';
-        const hasPassingFontDisplay = passingFontDisplayRegex.test(rawFontDisplay);
-        const targetURLSet = hasPassingFontDisplay ? passingURLs : failingURLs;
-
-        // Finally convert the raw font URLs to the absolute URLs and add them to the set.
-        const relativeURLs = rawFontURLs
-          // @ts-expect-error - guaranteed to match from previous regex, pull URL group out
-          .map(s => s.match(CSS_URL_REGEX)[1].trim())
-          .map(s => {
-            // remove any quotes surrounding the URL
-            if (/^('|").*\1$/.test(s)) {
-              return s.substr(1, s.length - 2);
-            }
-
-            return s;
-          });
-
-        // Convert the relative CSS URL to an absolute URL and add it to the target set.
-        for (const relativeURL of relativeURLs) {
-          try {
-            const relativeRoot = UrlUtils.isValid(stylesheet.header.sourceURL) ?
-              stylesheet.header.sourceURL : artifacts.URL.finalDisplayedUrl;
-            const absoluteURL = new URL(relativeURL, relativeRoot);
-            targetURLSet.add(absoluteURL.href);
-          } catch (err) {
-            Sentry.captureException(err, {tags: {audit: this.meta.id}});
-          }
-        }
-      }
-    }
-
-    return {passingURLs, failingURLs};
-  }
-
-  /**
-   * Some pages load many fonts we can't check, so dedupe on origin.
-   * @param {Array<string>} warningUrls
-   * @return {Array<LH.IcuMessage>}
-   */
-  static getWarningsForFontUrls(warningUrls) {
-    /** @type {Map<string, number>} */
-    const warningCountByOrigin = new Map();
-    for (const warningUrl of warningUrls) {
-      const origin = UrlUtils.getOrigin(warningUrl);
-      if (!origin) continue;
-
-      const count = warningCountByOrigin.get(origin) || 0;
-      warningCountByOrigin.set(origin, count + 1);
-    }
-
-    const warnings = [...warningCountByOrigin].map(([fontOrigin, fontCountForOrigin]) => {
-      return str_(UIStrings.undeclaredFontOriginWarning, {fontCountForOrigin, fontOrigin});
-    });
-    return warnings;
-  }
-
-  /**
-   * @param {LH.Artifacts} artifacts
-   * @param {LH.Audit.Context} context
-   * @return {Promise<LH.Audit.Product>}
-   */
-  static async audit(artifacts, context) {
-    const devtoolsLogs = artifacts.DevtoolsLog;
-    const networkRecords = await NetworkRecords.request(devtoolsLogs, context);
-    const {passingURLs, failingURLs} =
-      FontDisplay.findFontDisplayDeclarations(artifacts, PASSING_FONT_DISPLAY_REGEX);
-    /** @type {Array<string>} */
-    const warningURLs = [];
-
-    const results = networkRecords
-      // Find all fonts...
-      .filter(record => record.resourceType === 'Font')
-      // ...and that aren't data URLs, the blocking concern doesn't really apply
-      .filter(record => !/^data:/.test(record.url))
-      .filter(record => !/^blob:/.test(record.url))
-      // ...that have a failing font-display value
-      .filter(record => {
-        // Failing URLs should be considered.
-        if (failingURLs.has(record.url)) return true;
-        // Everything else shouldn't be, but we should warn if we don't recognize the URL at all.
-        if (!passingURLs.has(record.url)) warningURLs.push(record.url);
-        return false;
-      })
-      .map(record => {
-        // In reality the end time should be calculated with paint time included
-        // all browsers wait 3000ms to block text so we make sure 3000 is our max wasted time
-        const wastedMs = Math.min(record.networkEndTime - record.networkRequestTime, 3000);
-
-        return {
-          url: record.url,
-          wastedMs,
-        };
-      });
-
-    /** @type {LH.Audit.Details.Table['headings']} */
-    const headings = [
-      {key: 'url', valueType: 'url', label: str_(i18n.UIStrings.columnURL)},
-      {key: 'wastedMs', valueType: 'ms', label: str_(i18n.UIStrings.columnWastedMs)},
-    ];
-
-    const details = Audit.makeTableDetails(headings, results);
-
-    return {
-      score: Number(results.length === 0),
-      details,
-      warnings: FontDisplay.getWarningsForFontUrls(warningURLs),
-    };
-  }
-}
-
-export default FontDisplay;
-export {UIStrings};

package/core/audits/largest-contentful-paint-element.d.ts

@@ -1,34 +0,0 @@
-export default LargestContentfulPaintElement;
-declare class LargestContentfulPaintElement extends Audit {
-    /**
-     * @param {LH.Artifacts} artifacts
-     * @return {LH.Audit.Details.Table|undefined}
-     */
-    static makeElementTable(artifacts: LH.Artifacts): LH.Audit.Details.Table | undefined;
-    /**
-     * @param {number} metricLcp
-     * @param {LH.Artifacts.MetricComputationDataInput} metricComputationData
-     * @param {LH.Audit.Context} context
-     * @return {Promise<LH.Audit.Details.Table>}
-     */
-    static makePhaseTable(metricLcp: number, metricComputationData: LH.Artifacts.MetricComputationDataInput, context: LH.Audit.Context): Promise<LH.Audit.Details.Table>;
-    /**
-     * @param {LH.Artifacts} artifacts
-     * @param {LH.Audit.Context} context
-     * @return {Promise<LH.Audit.Product>}
-     */
-    static audit(artifacts: LH.Artifacts, context: LH.Audit.Context): Promise<LH.Audit.Product>;
-}
-export namespace UIStrings {
-    let title: string;
-    let description: string;
-    let columnPhase: string;
-    let columnPercentOfLCP: string;
-    let columnTiming: string;
-    let itemTTFB: string;
-    let itemLoadDelay: string;
-    let itemLoadTime: string;
-    let itemRenderDelay: string;
-}
-import { Audit } from './audit.js';
-//# sourceMappingURL=largest-contentful-paint-element.d.ts.map