lighthouse 9.6.8 → 10.0.0-dev.20230210

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

@@ -1,68 +0,0 @@
-/**
- * @license Copyright 2016 The Lighthouse Authors. All Rights Reserved.
- * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
- * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
- */
-
-/**
- * @fileoverview Audit a page to see if it is using passive event listeners on
- * scroll-blocking touch and wheel event listeners.
- */
-
-'use strict';
-
-const ViolationAudit = require('../violation-audit.js');
-const i18n = require('../../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. 'Learn More' 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](https://web.dev/uses-passive-event-listeners/).',
-};
-
-const str_ = i18n.createMessageInstanceIdFn(__filename, 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),
-      requiredArtifacts: ['ConsoleMessages', 'SourceMaps', 'ScriptElements'],
-    };
-  }
-
-  /**
-   * @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', itemType: 'source-location', text: 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,
-    };
-  }
-}
-
-module.exports = PassiveEventsAudit;
-module.exports.UIStrings = UIStrings;

package/lighthouse-core/audits/errors-in-console.js

@@ -1,118 +0,0 @@
-/**
- * @license Copyright 2017 The Lighthouse Authors. All Rights Reserved.
- * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
- * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
- */
-'use strict';
-
-/**
- * @fileoverview Audits a page to determine whether it contains console errors.
- * This is done by collecting Chrome console log messages and filtering out the non-error ones.
- */
-
-const log = require('lighthouse-logger');
-const Audit = require('./audit.js');
-const JsBundles = require('../computed/js-bundles.js');
-const i18n = require('../lib/i18n/i18n.js');
-
-const UIStrings = {
-  /** Title of a Lighthouse audit that provides detail on browser errors. This descriptive title is shown to users when no browser errors were logged into the devtools console. */
-  title: 'No browser errors logged to the console',
-  /** Title of a Lighthouse audit that provides detail on browser errors. This descriptive title is shown to users when browser errors occurred and were logged into the devtools console. */
-  failureTitle: 'Browser errors were logged to the console',
-  /** Description of a Lighthouse audit that tells the user why errors being logged to the devtools console are a cause for concern and so should be fixed. This is displayed after a user expands the section to see more. No character length limits. */
-  description: 'Errors logged to the console indicate unresolved problems. ' +
-    'They can come from network request failures and other browser concerns. ' +
-    '[Learn more](https://web.dev/errors-in-console/)',
-};
-
-const str_ = i18n.createMessageInstanceIdFn(__filename, UIStrings);
-
-/** @typedef {{ignoredPatterns?: Array<RegExp|string>}} AuditOptions */
-
-class ErrorLogs extends Audit {
-  /**
-   * @return {LH.Audit.Meta}
-   */
-  static get meta() {
-    return {
-      id: 'errors-in-console',
-      title: str_(UIStrings.title),
-      failureTitle: str_(UIStrings.failureTitle),
-      description: str_(UIStrings.description),
-      requiredArtifacts: ['ConsoleMessages', 'SourceMaps', 'ScriptElements'],
-    };
-  }
-
-  /** @return {AuditOptions} */
-  static get defaultOptions() {
-    // Any failed network requests with error messsage aren't actionable
-    return {ignoredPatterns: ['ERR_BLOCKED_BY_CLIENT.Inspector']};
-  }
-
-  /**
-   * @template {{description: string | undefined}} T
-   * @param {Array<T>} items
-   * @param {AuditOptions} options
-   * @return {Array<T>}
-   */
-  static filterAccordingToOptions(items, options) {
-    const {ignoredPatterns, ...restOfOptions} = options;
-    const otherOptionKeys = Object.keys(restOfOptions);
-    if (otherOptionKeys.length) log.warn(this.meta.id, 'Unrecognized options', otherOptionKeys);
-    if (!ignoredPatterns) return items;
-
-    return items.filter(({description}) => {
-      if (!description) return true;
-      for (const pattern of ignoredPatterns) {
-        if (pattern instanceof RegExp && pattern.test(description)) return false;
-        if (typeof pattern === 'string' && description.includes(pattern)) return false;
-      }
-
-      return true;
-    });
-  }
-
-  /**
-   * @param {LH.Artifacts} artifacts
-   * @param {LH.Audit.Context} context
-   * @return {Promise<LH.Audit.Product>}
-   */
-  static async audit(artifacts, context) {
-    /** @type {AuditOptions} */
-    const auditOptions = context.options;
-    const bundles = await JsBundles.request(artifacts, context);
-
-    /** @type {Array<{source: string, description: string|undefined, sourceLocation: LH.Audit.Details.SourceLocationValue|undefined}>} */
-    const consoleRows = artifacts.ConsoleMessages
-      .filter(item => item.level === 'error')
-      .map(item => {
-        const bundle = bundles.find(bundle => bundle.script.src === item.url);
-        return {
-          source: item.source,
-          description: item.text,
-          sourceLocation: Audit.makeSourceLocationFromConsoleMessage(item, bundle),
-        };
-      });
-
-    const tableRows = ErrorLogs.filterAccordingToOptions(consoleRows, auditOptions)
-      .sort((a, b) => (a.description || '').localeCompare(b.description || ''));
-
-    /** @type {LH.Audit.Details.Table['headings']} */
-    const headings = [
-      {key: 'sourceLocation', itemType: 'source-location', text: str_(i18n.UIStrings.columnSource)},
-      {key: 'description', itemType: 'code', text: str_(i18n.UIStrings.columnDescription)},
-    ];
-
-    const details = Audit.makeTableDetails(headings, tableRows);
-    const numErrors = tableRows.length;
-
-    return {
-      score: Number(numErrors === 0),
-      details,
-    };
-  }
-}
-
-module.exports = ErrorLogs;
-module.exports.UIStrings = UIStrings;

package/lighthouse-core/audits/final-screenshot.js

@@ -1,59 +0,0 @@
-/**
- * @license Copyright 2018 The Lighthouse Authors. All Rights Reserved.
- * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
- * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
- */
-'use strict';
-
-const Audit = require('./audit.js');
-const LHError = require('../lib/lh-error.js');
-const ProcessedTrace = require('../computed/processed-trace.js');
-const Screenshots = require('../computed/screenshots.js');
-
-class FinalScreenshot extends Audit {
-  /**
-   * @return {LH.Audit.Meta}
-   */
-  static get meta() {
-    return {
-      id: 'final-screenshot',
-      scoreDisplayMode: Audit.SCORING_MODES.INFORMATIVE,
-      title: 'Final Screenshot',
-      description: 'The last screenshot captured of the pageload.',
-      requiredArtifacts: ['traces', 'GatherContext'],
-    };
-  }
-
-  /**
-   * @param {LH.Artifacts} artifacts
-   * @param {LH.Audit.Context} context
-   * @return {Promise<LH.Audit.Product>}
-   */
-  static async audit(artifacts, context) {
-    const trace = artifacts.traces[Audit.DEFAULT_PASS];
-    const processedTrace = await ProcessedTrace.request(trace, context);
-    const screenshots = await Screenshots.request(trace, context);
-    const {timeOrigin} = processedTrace.timestamps;
-    const finalScreenshot = screenshots[screenshots.length - 1];
-
-    if (!finalScreenshot) {
-      // If a timespan didn't happen to contain frames, that's fine. Just mark not applicable.
-      if (artifacts.GatherContext.gatherMode === 'timespan') return {notApplicable: true, score: 1};
-
-      // If it was another mode, that's a fatal error.
-      throw new LHError(LHError.errors.NO_SCREENSHOTS);
-    }
-
-    return {
-      score: 1,
-      details: {
-        type: 'screenshot',
-        timing: Math.round((finalScreenshot.timestamp - timeOrigin) / 1000),
-        timestamp: finalScreenshot.timestamp,
-        data: finalScreenshot.datauri,
-      },
-    };
-  }
-}
-
-module.exports = FinalScreenshot;

package/lighthouse-core/audits/font-display.js

@@ -1,193 +0,0 @@
-/**
- * @license Copyright 2017 The Lighthouse Authors. All Rights Reserved.
- * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
- * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
- */
-'use strict';
-
-const Audit = require('./audit.js');
-const URL = require('../lib/url-shim.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 i18n = require('../lib/i18n/i18n.js');
-const Sentry = require('../lib/sentry.js');
-const NetworkRecords = require('../computed/network-records.js');
-
-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. 'Learn More' 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](https://web.dev/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.createMessageInstanceIdFn(__filename, 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'],
-      requiredArtifacts: ['devtoolsLogs', 'CSSUsage', 'URL'],
-    };
-  }
-
-  /**
-   * @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.CSSUsage.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 = URL.isValid(stylesheet.header.sourceURL) ?
-              stylesheet.header.sourceURL : artifacts.URL.finalUrl;
-            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 = URL.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.devtoolsLogs[this.DEFAULT_PASS];
-    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.endTime - record.startTime) * 1000, 3000);
-
-        return {
-          url: record.url,
-          wastedMs,
-        };
-      });
-
-    /** @type {LH.Audit.Details.Table['headings']} */
-    const headings = [
-      {key: 'url', itemType: 'url', text: str_(i18n.UIStrings.columnURL)},
-      {key: 'wastedMs', itemType: 'ms', text: str_(i18n.UIStrings.columnWastedMs)},
-    ];
-
-    const details = Audit.makeTableDetails(headings, results);
-
-    return {
-      score: Number(results.length === 0),
-      details,
-      warnings: FontDisplay.getWarningsForFontUrls(warningURLs),
-    };
-  }
-}
-
-module.exports = FontDisplay;
-module.exports.UIStrings = UIStrings;

package/lighthouse-core/audits/full-page-screenshot.js

@@ -1,43 +0,0 @@
-/**
- * @license Copyright 2020 Google Inc. All Rights Reserved.
- * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
- * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
- */
-'use strict';
-
-const Audit = require('./audit.js');
-
-class FullPageScreenshot extends Audit {
-  /**
-   * @return {LH.Audit.Meta}
-   */
-  static get meta() {
-    return {
-      id: 'full-page-screenshot',
-      scoreDisplayMode: Audit.SCORING_MODES.INFORMATIVE,
-      title: 'Full-page screenshot',
-      description: 'A full-height screenshot of the final rendered page',
-      requiredArtifacts: ['FullPageScreenshot'],
-    };
-  }
-
-  /**
-   * @param {LH.Artifacts} artifacts
-   * @return {Promise<LH.Audit.Product>}
-   */
-  static async audit(artifacts) {
-    if (!artifacts.FullPageScreenshot) {
-      return {score: 0, notApplicable: true};
-    }
-
-    return {
-      score: 1,
-      details: {
-        type: 'full-page-screenshot',
-        ...artifacts.FullPageScreenshot,
-      },
-    };
-  }
-}
-
-module.exports = FullPageScreenshot;

package/lighthouse-core/audits/image-aspect-ratio.js

@@ -1,122 +0,0 @@
-/**
- * @license Copyright 2017 The Lighthouse Authors. All Rights Reserved.
- * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
- * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
- */
-/**
- * @fileoverview Checks to see if the aspect ratio of the images used on
- *   the page are equal to the aspect ratio of their display sizes. The
- *   audit will list all images that don't match with their display size
- *   aspect ratio.
- */
-'use strict';
-
-const Audit = require('./audit.js');
-const URL = require('../lib/url-shim.js');
-const i18n = require('../lib/i18n/i18n.js');
-
-const UIStrings = {
-  /** Title of a Lighthouse audit that provides detail on the aspect ratios of all images on the page. This descriptive title is shown to users when all images use correct aspect ratios. */
-  title: 'Displays images with correct aspect ratio',
-  /** Title of a Lighthouse audit that provides detail on the aspect ratios of all images on the page. This descriptive title is shown to users when not all images use correct aspect ratios. */
-  failureTitle: 'Displays images with incorrect aspect ratio',
-  /** Description of a Lighthouse audit that tells the user why they should maintain the correct aspect ratios for all images. This is displayed after a user expands the section to see more. No character length limits. 'Learn More' becomes link text to additional documentation. */
-  description: 'Image display dimensions should match natural aspect ratio. ' +
-    '[Learn more](https://web.dev/image-aspect-ratio/).',
-  /**  Label for a column in a data table; entries in the column will be the numeric aspect ratio of an image as displayed in a web page. */
-  columnDisplayed: 'Aspect Ratio (Displayed)',
-  /**  Label for a column in a data table; entries in the column will be the numeric aspect ratio of the raw (actual) image. */
-  columnActual: 'Aspect Ratio (Actual)',
-};
-
-const str_ = i18n.createMessageInstanceIdFn(__filename, UIStrings);
-
-const THRESHOLD_PX = 2;
-
-/** @typedef {Required<LH.Artifacts.ImageElement>} WellDefinedImage */
-
-class ImageAspectRatio extends Audit {
-  /**
-   * @return {LH.Audit.Meta}
-   */
-  static get meta() {
-    return {
-      id: 'image-aspect-ratio',
-      title: str_(UIStrings.title),
-      failureTitle: str_(UIStrings.failureTitle),
-      description: str_(UIStrings.description),
-      requiredArtifacts: ['ImageElements'],
-    };
-  }
-
-  /**
-   * @param {WellDefinedImage} image
-   * @return {{url: string, node: LH.Audit.Details.NodeValue, displayedAspectRatio: string, actualAspectRatio: string, doRatiosMatch: boolean}}
-   */
-  static computeAspectRatios(image) {
-    const url = URL.elideDataURI(image.src);
-    const actualAspectRatio = image.naturalDimensions.width / image.naturalDimensions.height;
-    const displayedAspectRatio = image.displayedWidth / image.displayedHeight;
-
-    const targetDisplayHeight = image.displayedWidth / actualAspectRatio;
-    const doRatiosMatch = Math.abs(targetDisplayHeight - image.displayedHeight) < THRESHOLD_PX;
-
-    return {
-      url,
-      node: Audit.makeNodeItem(image.node),
-      displayedAspectRatio: `${image.displayedWidth} x ${image.displayedHeight}
-        (${displayedAspectRatio.toFixed(2)})`,
-      actualAspectRatio: `${image.naturalDimensions.width} x ${image.naturalDimensions.height}
-        (${actualAspectRatio.toFixed(2)})`,
-      doRatiosMatch,
-    };
-  }
-
-  /**
-   * @param {LH.Artifacts} artifacts
-   * @return {LH.Audit.Product}
-   */
-  static audit(artifacts) {
-    const images = artifacts.ImageElements;
-
-    /** @type {Array<{url: string, node: LH.Audit.Details.NodeValue, displayedAspectRatio: string, actualAspectRatio: string, doRatiosMatch: boolean}>} */
-    const results = [];
-    images.filter(image => {
-      // - filter out css background images since we don't have a reliable way to tell if it's a
-      //   sprite sheet, repeated for effect, etc
-      // - filter out images that don't have following properties:
-      //   networkRecord, width, height, `object-fit` property
-      // - filter all svgs as they have no natural dimensions to audit
-      // - filter out images that have falsy naturalWidth or naturalHeight
-      return !image.isCss &&
-        URL.guessMimeType(image.src) !== 'image/svg+xml' &&
-        image.naturalDimensions &&
-        image.naturalDimensions.height > 5 &&
-        image.naturalDimensions.width > 5 &&
-        image.displayedWidth &&
-        image.displayedHeight &&
-        image.computedStyles.objectFit === 'fill';
-    }).forEach(image => {
-      const wellDefinedImage = /** @type {WellDefinedImage} */ (image);
-      const processed = ImageAspectRatio.computeAspectRatios(wellDefinedImage);
-
-      if (!processed.doRatiosMatch) results.push(processed);
-    });
-
-    /** @type {LH.Audit.Details.Table['headings']} */
-    const headings = [
-      {key: 'node', itemType: 'node', text: ''},
-      {key: 'url', itemType: 'url', text: str_(i18n.UIStrings.columnURL)},
-      {key: 'displayedAspectRatio', itemType: 'text', text: str_(UIStrings.columnDisplayed)},
-      {key: 'actualAspectRatio', itemType: 'text', text: str_(UIStrings.columnActual)},
-    ];
-
-    return {
-      score: Number(results.length === 0),
-      details: Audit.makeTableDetails(headings, results),
-    };
-  }
-}
-
-module.exports = ImageAspectRatio;
-module.exports.UIStrings = UIStrings;