lighthouse 9.5.0-dev.20220907 → 9.5.0-dev.20220909

package/CONTRIBUTING.md

@@ -128,7 +128,7 @@ accept your pull requests.
 
 ## Tracking Errors
 
-We track our errors in the wild with Sentry. In general, do not worry about wrapping your audits or gatherers in try/catch blocks and reporting every error that could possibly occur; `core/runner.js` and `core/gather/gather-runner.js` already catch and report any errors that occur while running a gatherer or audit, including errors fatal to the entire run. However, there are some situations when you might want to explicitly handle an error and report it to Sentry or wrap it to avoid reporting. Generally, you can interact with Sentry simply by requiring the `core/lib/sentry.js` file and call its methods. The module exports a delegate that will correctly handle the error reporting based on the user's opt-in preference and will simply no-op if they haven't so you don't need to check.
+We track our errors in the wild with Sentry. In general, do not worry about wrapping your audits or gatherers in try/catch blocks and reporting every error that could possibly occur; `core/runner.js` and `core/fraggle-rock/gather/*-runner.js` already catch and report any errors that occur while running a gatherer or audit, including errors fatal to the entire run. However, there are some situations when you might want to explicitly handle an error and report it to Sentry or wrap it to avoid reporting. Generally, you can interact with Sentry simply by requiring the `core/lib/sentry.js` file and call its methods. The module exports a delegate that will correctly handle the error reporting based on the user's opt-in preference and will simply no-op if they haven't so you don't need to check.
 
 
 #### If you have an expected error that is recoverable but want to track how frequently it happens, *use Sentry.captureException*.

package/cli/test/smokehouse/core-tests.js

@@ -42,7 +42,7 @@ import perfFonts from './test-definitions/perf-fonts.js';
 import perfFrameMetrics from './test-definitions/perf-frame-metrics.js';
 import perfPreload from './test-definitions/perf-preload.js';
 import perfTraceElements from './test-definitions/perf-trace-elements.js';
-// import pubads from './test-definitions/pubads.js';
+import pubads from './test-definitions/pubads.js';
 import pwaAirhorner from './test-definitions/pwa-airhorner.js';
 import pwaCaltrain from './test-definitions/pwa-caltrain.js';
 import pwaChromestatus from './test-definitions/pwa-chromestatus.js';
@@ -103,8 +103,7 @@ const smokeTests = [
   perfFrameMetrics,
   perfPreload,
   perfTraceElements,
-  // TODO(esmodules): enable when pubads is bundled again
-  // pubads,
+  pubads,
   pwaAirhorner,
   pwaChromestatus,
   pwaSvgomg,

package/cli/test/smokehouse/lighthouse-runners/bundle.js

@@ -19,7 +19,7 @@ import {once} from 'events';
 import puppeteer from 'puppeteer-core';
 import ChromeLauncher from 'chrome-launcher';
 
-import {CriConnection} from '../../../../core/gather/connections/cri.js';
+import {CriConnection} from '../../../../core/legacy/gather/connections/cri.js';
 import {LH_ROOT} from '../../../../root.js';
 import {loadArtifacts, saveArtifacts} from '../../../../core/lib/asset-saver.js';
 

package/core/audits/bootup-time.js

@@ -7,8 +7,8 @@
 import {Audit} from './audit.js';
 import {taskGroups} from '../lib/tracehouse/task-groups.js';
 import * as i18n from '../lib/i18n/i18n.js';
-import NetworkRecords from '../computed/network-records.js';
-import MainThreadTasks from '../computed/main-thread-tasks.js';
+import {NetworkRecords} from '../computed/network-records.js';
+import {MainThreadTasks} from '../computed/main-thread-tasks.js';
 import {getExecutionTimingsByURL} from '../lib/tracehouse/task-summary.js';
 
 const UIStrings = {

package/core/audits/byte-efficiency/byte-efficiency-audit.js

@@ -6,11 +6,11 @@
 
 import {Audit} from '../audit.js';
 import {linearInterpolation} from '../../lib/statistics.js';
-import Interactive from '../../computed/metrics/lantern-interactive.js';
+import {LanternInteractive} from '../../computed/metrics/lantern-interactive.js';
 import * as i18n from '../../lib/i18n/i18n.js';
-import NetworkRecords from '../../computed/network-records.js';
-import LoadSimulator from '../../computed/load-simulator.js';
-import PageDependencyGraph from '../../computed/page-dependency-graph.js';
+import {NetworkRecords} from '../../computed/network-records.js';
+import {LoadSimulator} from '../../computed/load-simulator.js';
+import {PageDependencyGraph} from '../../computed/page-dependency-graph.js';
 
 const str_ = i18n.createIcuMessageFn(import.meta.url, {});
 
@@ -190,8 +190,9 @@ class ByteEfficiencyAudit extends Audit {
     });
 
     const savingsOnOverallLoad = simulationBeforeChanges.timeInMs - simulationAfterChanges.timeInMs;
-    const savingsOnTTI = Interactive.getLastLongTaskEndTime(simulationBeforeChanges.nodeTimings) -
-      Interactive.getLastLongTaskEndTime(simulationAfterChanges.nodeTimings);
+    const savingsOnTTI =
+      LanternInteractive.getLastLongTaskEndTime(simulationBeforeChanges.nodeTimings) -
+      LanternInteractive.getLastLongTaskEndTime(simulationAfterChanges.nodeTimings);
 
     let savings = savingsOnTTI;
     if (options.includeLoad) savings = Math.max(savings, savingsOnOverallLoad);

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

@@ -9,7 +9,7 @@
 /** @typedef {{url: string, sourceTransferBytes?: number}} SubItem */
 
 import {ByteEfficiencyAudit} from './byte-efficiency-audit.js';
-import ModuleDuplication from '../../computed/module-duplication.js';
+import {ModuleDuplication} from '../../computed/module-duplication.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 import {getRequestForScript} from '../../lib/script-helpers.js';
 

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

@@ -20,7 +20,7 @@
 import fs from 'fs';
 
 import {ByteEfficiencyAudit} from './byte-efficiency-audit.js';
-import JsBundles from '../../computed/js-bundles.js';
+import {JSBundles} from '../../computed/js-bundles.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 import thirdPartyWeb from '../../lib/third-party-web.js';
 import {getRequestForScript} from '../../lib/script-helpers.js';
@@ -402,7 +402,7 @@ class LegacyJavascript extends ByteEfficiencyAudit {
    */
   static async audit_(artifacts, networkRecords, context) {
     const mainDocumentEntity = thirdPartyWeb.getEntity(artifacts.URL.finalUrl);
-    const bundles = await JsBundles.request(artifacts, context);
+    const bundles = await JSBundles.request(artifacts, context);
 
     /** @type {Item[]} */
     const items = [];

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

@@ -14,8 +14,8 @@ import {NetworkRequest} from '../../lib/network-request.js';
 import {Sentry} from '../../lib/sentry.js';
 import UrlUtils from '../../lib/url-utils.js';
 import * as i18n from '../../lib/i18n/i18n.js';
-import Interactive from '../../computed/metrics/interactive.js';
-import ProcessedTrace from '../../computed/processed-trace.js';
+import {Interactive} from '../../computed/metrics/interactive.js';
+import {ProcessedTrace} from '../../computed/processed-trace.js';
 
 const UIStrings = {
   /** Imperative title of a Lighthouse audit that tells the user to defer loading offscreen images. Offscreen images are images located outside of the visible browser viewport. As they are unseen by the user and slow down page load, they should be loaded later, closer to when the user is going to see them. This is displayed in a list of audit titles that Lighthouse generates. */

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

@@ -13,12 +13,12 @@ import {Audit} from '../audit.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 import {BaseNode} from '../../lib/dependency-graph/base-node.js';
 import {ByteEfficiencyAudit} from './byte-efficiency-audit.js';
-import UnusedCSS from '../../computed/unused-css.js';
+import {UnusedCSS} from '../../computed/unused-css.js';
 import {NetworkRequest} from '../../lib/network-request.js';
-import ProcessedTrace from '../../computed/processed-trace.js';
-import ProcessedNavigation from '../../computed/processed-navigation.js';
-import LoadSimulator from '../../computed/load-simulator.js';
-import FirstContentfulPaint from '../../computed/metrics/first-contentful-paint.js';
+import {ProcessedTrace} from '../../computed/processed-trace.js';
+import {ProcessedNavigation} from '../../computed/processed-navigation.js';
+import {LoadSimulator} from '../../computed/load-simulator.js';
+import {FirstContentfulPaint} from '../../computed/metrics/first-contentful-paint.js';
 
 /** @typedef {import('../../lib/dependency-graph/simulator/simulator').Simulator} Simulator */
 /** @typedef {import('../../lib/dependency-graph/base-node.js').Node} Node */

package/core/audits/byte-efficiency/total-byte-weight.js

@@ -7,7 +7,7 @@
 import {Audit} from '../audit.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 import {NetworkRequest} from '../../lib/network-request.js';
-import NetworkRecords from '../../computed/network-records.js';
+import {NetworkRecords} from '../../computed/network-records.js';
 
 const UIStrings = {
   /** Title of a diagnostic audit that provides detail on large network resources required during page load. 'Payloads' is roughly equivalent to 'resources'. This descriptive title is shown to users when the amount is acceptable and no user action is required. */

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

@@ -5,7 +5,7 @@
  */
 
 import {ByteEfficiencyAudit} from './byte-efficiency-audit.js';
-import UnusedCSS from '../../computed/unused-css.js';
+import {UnusedCSS} from '../../computed/unused-css.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 import {computeCSSTokenLength as computeTokenLength} from '../../lib/minification-estimator.js';
 

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

@@ -5,7 +5,7 @@
  */
 
 import {ByteEfficiencyAudit} from './byte-efficiency-audit.js';
-import UnusedCSS from '../../computed/unused-css.js';
+import {UnusedCSS} from '../../computed/unused-css.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 
 const UIStrings = {

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

@@ -5,8 +5,8 @@
  */
 
 import {ByteEfficiencyAudit} from './byte-efficiency-audit.js';
-import UnusedJavaScriptSummary from '../../computed/unused-javascript-summary.js';
-import JsBundles from '../../computed/js-bundles.js';
+import {UnusedJavascriptSummary} from '../../computed/unused-javascript-summary.js';
+import {JSBundles} from '../../computed/js-bundles.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 import {getRequestForScript} from '../../lib/script-helpers.js';
 
@@ -79,7 +79,7 @@ class UnusedJavaScript extends ByteEfficiencyAudit {
    * @return {Promise<import('./byte-efficiency-audit.js').ByteEfficiencyProduct>}
    */
   static async audit_(artifacts, networkRecords, context) {
-    const bundles = await JsBundles.request(artifacts, context);
+    const bundles = await JSBundles.request(artifacts, context);
     const {
       unusedThreshold = UNUSED_BYTES_IGNORE_THRESHOLD,
       bundleSourceUnusedThreshold = UNUSED_BYTES_IGNORE_BUNDLE_SOURCE_THRESHOLD,
@@ -95,7 +95,7 @@ class UnusedJavaScript extends ByteEfficiencyAudit {
 
       const bundle = bundles.find(b => b.script.scriptId === scriptId);
       const unusedJsSummary =
-        await UnusedJavaScriptSummary.request({scriptId, scriptCoverage, bundle}, context);
+        await UnusedJavascriptSummary.request({scriptId, scriptCoverage, bundle}, context);
       if (unusedJsSummary.wastedBytes === 0 || unusedJsSummary.totalBytes === 0) continue;
 
       const transfer = ByteEfficiencyAudit

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

@@ -11,7 +11,7 @@ import {NetworkRequest} from '../../lib/network-request.js';
 import UrlUtils from '../../lib/url-utils.js';
 import {linearInterpolation} from '../../lib/statistics.js';
 import * as i18n from '../../lib/i18n/i18n.js';
-import NetworkRecords from '../../computed/network-records.js';
+import {NetworkRecords} from '../../computed/network-records.js';
 
 const UIStrings = {
   /** Title of a diagnostic audit that provides detail on the cache policy applies to the page's static assets. Cache refers to browser disk cache, which keeps old versions of network resources around for future use. This is displayed in a list of audit titles that Lighthouse generates. */

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

@@ -15,7 +15,7 @@
 
 import {ByteEfficiencyAudit} from './byte-efficiency-audit.js';
 import {NetworkRequest} from '../../lib/network-request.js';
-import ImageRecords from '../../computed/image-records.js';
+import {ImageRecords} from '../../computed/image-records.js';
 import UrlUtils from '../../lib/url-utils.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 

package/core/audits/critical-request-chains.js

@@ -6,7 +6,7 @@
 
 import {Audit} from './audit.js';
 import * as i18n from '../lib/i18n/i18n.js';
-import ComputedChains from '../computed/critical-request-chains.js';
+import {CriticalRequestChains as ComputedChains} from '../computed/critical-request-chains.js';
 
 const UIStrings = {
   /** Imperative title of a Lighthouse audit that tells the user to reduce the depth of critical network requests to enhance initial load of a page. Critical request chains are series of dependent network requests that are important for page rendering. For example, here's a 4-request-deep chain: The biglogo.jpg image is required, but is requested via the styles.css style code, which is requested by the initialize.js javascript, which is requested by the page's HTML. This is displayed in a list of audit titles that Lighthouse generates. */

package/core/audits/csp-xss.js

@@ -5,7 +5,7 @@
  */
 
 import {Audit} from './audit.js';
-import MainResource from '../computed/main-resource.js';
+import {MainResource} from '../computed/main-resource.js';
 import * as i18n from '../lib/i18n/i18n.js';
 import {evaluateRawCspsForXss, getTranslatedDescription} from '../lib/csp-evaluator.js';
 

package/core/audits/deprecations.js

@@ -9,7 +9,7 @@
  */
 
 import {Audit} from './audit.js';
-import JsBundles from '../computed/js-bundles.js';
+import {JSBundles} from '../computed/js-bundles.js';
 import * as i18n from '../lib/i18n/i18n.js';
 import {getIssueDetailDescription} from '../lib/deprecations-strings.js';
 
@@ -56,7 +56,7 @@ class Deprecations extends Audit {
    * @return {Promise<LH.Audit.Product>}
    */
   static async audit(artifacts, context) {
-    const bundles = await JsBundles.request(artifacts, context);
+    const bundles = await JSBundles.request(artifacts, context);
 
     const deprecations = artifacts.InspectorIssues.deprecationIssue
       .map(deprecation => {

package/core/audits/diagnostics.js

@@ -5,10 +5,10 @@
  */
 
 import {Audit} from './audit.js';
-import MainThreadTasksComputed from '../computed/main-thread-tasks.js';
-import NetworkRecordsComputed from '../computed/network-records.js';
-import NetworkAnalysisComputed from '../computed/network-analysis.js';
-import MainResource from '../computed/main-resource.js';
+import {MainThreadTasks} from '../computed/main-thread-tasks.js';
+import {NetworkRecords} from '../computed/network-records.js';
+import {NetworkAnalysis} from '../computed/network-analysis.js';
+import {MainResource} from '../computed/main-resource.js';
 
 class Diagnostics extends Audit {
   /**
@@ -33,9 +33,9 @@ class Diagnostics extends Audit {
   static async audit(artifacts, context) {
     const trace = artifacts.traces[Audit.DEFAULT_PASS];
     const devtoolsLog = artifacts.devtoolsLogs[Audit.DEFAULT_PASS];
-    const tasks = await MainThreadTasksComputed.request(trace, context);
-    const records = await NetworkRecordsComputed.request(devtoolsLog, context);
-    const analysis = await NetworkAnalysisComputed.request(devtoolsLog, context);
+    const tasks = await MainThreadTasks.request(trace, context);
+    const records = await NetworkRecords.request(devtoolsLog, context);
+    const analysis = await NetworkAnalysis.request(devtoolsLog, context);
     const mainResource = await MainResource.request({devtoolsLog, URL: artifacts.URL}, context);
 
     const toplevelTasks = tasks.filter(t => !t.parent);

package/core/audits/dobetterweb/charset.js

@@ -14,7 +14,7 @@
 
 import {Audit} from '../audit.js';
 import * as i18n from '../../lib/i18n/i18n.js';
-import MainResource from '../../computed/main-resource.js';
+import {MainResource} from '../../computed/main-resource.js';
 
 const UIStrings = {
   /** Title of a Lighthouse audit that provides detail on if the charset is set properly for a page. This title is shown when the charset is defined correctly. Charset defines the character encoding (eg UTF-8) of the page content. */

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

@@ -16,11 +16,11 @@ import {Audit} from '../audit.js';
 import ThirdParty from '../../lib/third-party-web.js';
 import UrlUtils from '../../lib/url-utils.js';
 import {ByteEfficiencyAudit} from '../byte-efficiency/byte-efficiency-audit.js';
-import Interactive from '../../computed/metrics/lantern-interactive.js';
+import {LanternInteractive} from '../../computed/metrics/lantern-interactive.js';
 import {NetworkRequest} from '../../lib/network-request.js';
-import NetworkRecords from '../../computed/network-records.js';
-import LoadSimulator from '../../computed/load-simulator.js';
-import PageDependencyGraph from '../../computed/page-dependency-graph.js';
+import {NetworkRecords} from '../../computed/network-records.js';
+import {LoadSimulator} from '../../computed/load-simulator.js';
+import {PageDependencyGraph} from '../../computed/page-dependency-graph.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 
 const UIStrings = {
@@ -106,8 +106,8 @@ class UsesHTTP2Audit extends Audit {
     });
 
     const savingsOnOverallLoad = simulationBefore.timeInMs - simulationAfter.timeInMs;
-    const savingsOnTTI = Interactive.getLastLongTaskEndTime(simulationBefore.nodeTimings) -
-      Interactive.getLastLongTaskEndTime(simulationAfter.nodeTimings);
+    const savingsOnTTI = LanternInteractive.getLastLongTaskEndTime(simulationBefore.nodeTimings) -
+      LanternInteractive.getLastLongTaskEndTime(simulationAfter.nodeTimings);
     const savings = Math.max(savingsOnTTI, savingsOnOverallLoad);
 
     // Round waste to nearest 10ms

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

@@ -12,7 +12,7 @@
 import log from 'lighthouse-logger';
 
 import {Audit} from './audit.js';
-import JsBundles from '../computed/js-bundles.js';
+import {JSBundles} from '../computed/js-bundles.js';
 import * as i18n from '../lib/i18n/i18n.js';
 
 const UIStrings = {
@@ -81,7 +81,7 @@ class ErrorLogs extends Audit {
   static async audit(artifacts, context) {
     /** @type {AuditOptions} */
     const auditOptions = context.options;
-    const bundles = await JsBundles.request(artifacts, context);
+    const bundles = await JSBundles.request(artifacts, context);
 
     /** @type {Array<{source: string, description: string|undefined, sourceLocation: LH.Audit.Details.SourceLocationValue|undefined}>} */
     const consoleRows = artifacts.ConsoleMessages

package/core/audits/final-screenshot.js

@@ -6,8 +6,8 @@
 
 import {Audit} from './audit.js';
 import {LighthouseError} from '../lib/lh-error.js';
-import ProcessedTrace from '../computed/processed-trace.js';
-import Screenshots from '../computed/screenshots.js';
+import {ProcessedTrace} from '../computed/processed-trace.js';
+import {Screenshots} from '../computed/screenshots.js';
 
 class FinalScreenshot extends Audit {
   /**

package/core/audits/font-display.js

@@ -8,7 +8,7 @@ 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';
+import {NetworkRecords} from '../computed/network-records.js';
 
 const PASSING_FONT_DISPLAY_REGEX = /^(block|fallback|optional|swap)$/;
 const CSS_URL_REGEX = /url\((.*?)\)/;

package/core/audits/installable-manifest.js

@@ -6,7 +6,7 @@
 
 import {Audit} from './audit.js';
 import * as i18n from '../lib/i18n/i18n.js';
-import ManifestValues from '../computed/manifest-values.js';
+import {ManifestValues} from '../computed/manifest-values.js';
 
 /* eslint-disable max-len */
 const UIStrings = {

package/core/audits/is-on-https.js

@@ -7,7 +7,7 @@
 import {Audit} from './audit.js';
 import UrlUtils from '../lib/url-utils.js';
 import {NetworkRequest} from '../lib/network-request.js';
-import NetworkRecords from '../computed/network-records.js';
+import {NetworkRecords} from '../computed/network-records.js';
 import * as i18n from '../lib/i18n/i18n.js';
 
 const UIStrings = {

package/core/audits/long-tasks.js

@@ -5,11 +5,11 @@
  */
 
 import {Audit} from './audit.js';
-import NetworkRecords from '../computed/network-records.js';
+import {NetworkRecords} from '../computed/network-records.js';
 import * as i18n from '../lib/i18n/i18n.js';
-import MainThreadTasks from '../computed/main-thread-tasks.js';
-import PageDependencyGraph from '../computed/page-dependency-graph.js';
-import LoadSimulator from '../computed/load-simulator.js';
+import {MainThreadTasks} from '../computed/main-thread-tasks.js';
+import {PageDependencyGraph} from '../computed/page-dependency-graph.js';
+import {LoadSimulator} from '../computed/load-simulator.js';
 import {getJavaScriptURLs, getAttributableURLForTask} from '../lib/tracehouse/task-summary.js';
 
 /** We don't always have timing data for short tasks, if we're missing timing data. Treat it as though it were 0ms. */

package/core/audits/main-thread-tasks.js

@@ -5,7 +5,7 @@
  */
 
 import {Audit} from './audit.js';
-import MainThreadTasksComputed from '../computed/main-thread-tasks.js';
+import {MainThreadTasks as MainThreadTasksComputed} from '../computed/main-thread-tasks.js';
 
 class MainThreadTasks extends Audit {
   /**

package/core/audits/mainthread-work-breakdown.js

@@ -12,7 +12,7 @@
 import {Audit} from './audit.js';
 import {taskGroups} from '../lib/tracehouse/task-groups.js';
 import * as i18n from '../lib/i18n/i18n.js';
-import MainThreadTasks from '../computed/main-thread-tasks.js';
+import {MainThreadTasks} from '../computed/main-thread-tasks.js';
 
 const UIStrings = {
   /** Title of a diagnostic audit that provides detail on the main thread work the browser did to load the page. This descriptive title is shown to users when the amount is acceptable and no user action is required. */

package/core/audits/maskable-icon.js

@@ -5,7 +5,7 @@
  */
 
 import {Audit} from './audit.js';
-import ManifestValues from '../computed/manifest-values.js';
+import {ManifestValues} from '../computed/manifest-values.js';
 import * as i18n from '../lib/i18n/i18n.js';
 
 const UIStrings = {

package/core/audits/metrics/cumulative-layout-shift.js

@@ -5,7 +5,7 @@
  */
 
 import {Audit} from '../audit.js';
-import ComputedCLS from '../../computed/metrics/cumulative-layout-shift.js';
+import {CumulativeLayoutShift as ComputedCLS} from '../../computed/metrics/cumulative-layout-shift.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 
 const UIStrings = {

package/core/audits/metrics/experimental-interaction-to-next-paint.js

@@ -5,7 +5,7 @@
  */
 
 import {Audit} from '../audit.js';
-import ComputedResponsivenes from '../../computed/metrics/responsiveness.js';
+import {Responsiveness as ComputedResponsivenes} from '../../computed/metrics/responsiveness.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 
 const UIStrings = {

package/core/audits/metrics/first-contentful-paint-3g.js

@@ -5,7 +5,7 @@
  */
 
 import {Audit} from '../audit.js';
-import ComputedFcp from '../../computed/metrics/first-contentful-paint.js';
+import {FirstContentfulPaint as ComputedFcp} from '../../computed/metrics/first-contentful-paint.js';
 import * as constants from '../../config/constants.js';
 
 const regular3G = constants.throttling.mobileRegular3G;

package/core/audits/metrics/first-contentful-paint.js

@@ -6,7 +6,7 @@
 
 import {Audit} from '../audit.js';
 import * as i18n from '../../lib/i18n/i18n.js';
-import ComputedFcp from '../../computed/metrics/first-contentful-paint.js';
+import {FirstContentfulPaint as ComputedFcp} from '../../computed/metrics/first-contentful-paint.js';
 
 const UIStrings = {
   /** Description of the First Contentful Paint (FCP) metric, which marks the time at which the first text or image is painted by the browser. This is displayed within a tooltip when the user hovers on the metric name to see more. No character length limits. 'Learn More' becomes link text to additional documentation. */

package/core/audits/metrics/first-meaningful-paint.js

@@ -6,7 +6,7 @@
 
 import {Audit} from '../audit.js';
 import * as i18n from '../../lib/i18n/i18n.js';
-import ComputedFmp from '../../computed/metrics/first-meaningful-paint.js';
+import {FirstMeaningfulPaint as ComputedFmp} from '../../computed/metrics/first-meaningful-paint.js';
 
 const UIStrings = {
   /** Description of the First Meaningful Paint (FMP) metric, which marks the time at which a majority of the content has been painted by the browser. This is displayed within a tooltip when the user hovers on the metric name to see more. No character length limits. 'Learn More' becomes link text to additional documentation. */

package/core/audits/metrics/interactive.js

@@ -6,7 +6,7 @@
 
 import {Audit} from '../audit.js';
 import * as i18n from '../../lib/i18n/i18n.js';
-import Interactive from '../../computed/metrics/interactive.js';
+import {Interactive} from '../../computed/metrics/interactive.js';
 
 const UIStrings = {
   /** Description of the Time to Interactive (TTI) metric, which evaluates when a page has completed its primary network activity and main thread work. This is displayed within a tooltip when the user hovers on the metric name to see more. No character length limits. 'Learn More' becomes link text to additional documentation. */

package/core/audits/metrics/largest-contentful-paint.js

@@ -6,7 +6,7 @@
 
 import {Audit} from '../audit.js';
 import * as i18n from '../../lib/i18n/i18n.js';
-import ComputedLcp from '../../computed/metrics/largest-contentful-paint.js';
+import {LargestContentfulPaint as ComputedLcp} from '../../computed/metrics/largest-contentful-paint.js';
 
 const UIStrings = {
   /** Description of the Largest Contentful Paint (LCP) metric, which marks the time at which the largest text or image is painted by the browser. This is displayed within a tooltip when the user hovers on the metric name to see more. No character length limits. 'Learn More' becomes link text to additional documentation. */

package/core/audits/metrics/max-potential-fid.js

@@ -5,7 +5,7 @@
  */
 
 import {Audit} from '../audit.js';
-import ComputedFid from '../../computed/metrics/max-potential-fid.js';
+import {MaxPotentialFID as ComputedFid} from '../../computed/metrics/max-potential-fid.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 
 const UIStrings = {

package/core/audits/metrics/speed-index.js

@@ -6,7 +6,7 @@
 
 import {Audit} from '../audit.js';
 import * as i18n from '../../lib/i18n/i18n.js';
-import ComputedSi from '../../computed/metrics/speed-index.js';
+import {SpeedIndex as ComputedSi} from '../../computed/metrics/speed-index.js';
 
 const UIStrings = {
   /** Description of the Speed Index metric, which summarizes how quickly the page looked visually complete. This is displayed within a tooltip when the user hovers on the metric name to see more. No character length limits. 'Learn More' becomes link text to additional documentation. */

package/core/audits/metrics/total-blocking-time.js

@@ -5,7 +5,7 @@
  */
 
 import {Audit} from '../audit.js';
-import ComputedTBT from '../../computed/metrics/total-blocking-time.js';
+import {TotalBlockingTime as ComputedTBT} from '../../computed/metrics/total-blocking-time.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 
 const UIStrings = {

package/core/audits/metrics.js

@@ -5,7 +5,7 @@
  */
 
 import {Audit} from './audit.js';
-import ComputedTimingSummary from '../computed/metrics/timing-summary.js';
+import {TimingSummary} from '../computed/metrics/timing-summary.js';
 
 /** @type {Set<keyof LH.Artifacts.TimingSummary>} */
 const DECIMAL_METRIC_KEYS = new Set([
@@ -42,7 +42,7 @@ class Metrics extends Audit {
     const trace = artifacts.traces[Audit.DEFAULT_PASS];
     const devtoolsLog = artifacts.devtoolsLogs[Audit.DEFAULT_PASS];
     const URL = artifacts.URL;
-    const summary = await ComputedTimingSummary
+    const summary = await TimingSummary
       .request({trace, devtoolsLog, gatherContext, settings: context.settings, URL}, context);
     const metrics = summary.metrics;
     const debugInfo = summary.debugInfo;

package/core/audits/network-requests.js

@@ -6,8 +6,8 @@
 
 import {Audit} from './audit.js';
 import UrlUtils from '../lib/url-utils.js';
-import NetworkRecords from '../computed/network-records.js';
-import MainResource from '../computed/main-resource.js';
+import {NetworkRecords} from '../computed/network-records.js';
+import {MainResource} from '../computed/main-resource.js';
 
 class NetworkRequests extends Audit {
   /**

package/core/audits/network-rtt.js

@@ -6,8 +6,8 @@
 
 import {Audit} from './audit.js';
 import * as i18n from '../lib/i18n/i18n.js';
-import NetworkRecords from '../computed/network-records.js';
-import NetworkAnalysisComputed from '../computed/network-analysis.js';
+import {NetworkRecords} from '../computed/network-records.js';
+import {NetworkAnalysis} from '../computed/network-analysis.js';
 
 const UIStrings = {
   /** Descriptive title of a Lighthouse audit that tells the user the round trip times to each origin the page connected to. This is displayed in a list of audit titles that Lighthouse generates. */
@@ -49,7 +49,7 @@ class NetworkRTT extends Audit {
       };
     }
 
-    const analysis = await NetworkAnalysisComputed.request(devtoolsLog, context);
+    const analysis = await NetworkAnalysis.request(devtoolsLog, context);
 
     /** @type {number} */
     let maxRtt = 0;

package/core/audits/network-server-latency.js

@@ -6,8 +6,8 @@
 
 import {Audit} from './audit.js';
 import * as i18n from '../lib/i18n/i18n.js';
-import NetworkRecords from '../computed/network-records.js';
-import NetworkAnalysisComputed from '../computed/network-analysis.js';
+import {NetworkRecords} from '../computed/network-records.js';
+import {NetworkAnalysis} from '../computed/network-analysis.js';
 
 const UIStrings = {
   /** Descriptive title of a Lighthouse audit that tells the user the server latencies observed from each origin the page connected to. This is displayed in a list of audit titles that Lighthouse generates. */
@@ -49,7 +49,7 @@ class NetworkServerLatency extends Audit {
       };
     }
 
-    const analysis = await NetworkAnalysisComputed.request(devtoolsLog, context);
+    const analysis = await NetworkAnalysis.request(devtoolsLog, context);
 
     /** @type {number} */
     let maxLatency = 0;