clients(devtools): require third-party-web to be provided

build/build-bundle.js

@@ -136,6 +136,11 @@ async function buildBundle(entryPath, distPath, opts = {minify: true}) {
     shimsObj[`${LH_ROOT}/shared/localization/locales.js`] = 'export const locales = {};';
   }
 
+  // Don't bundle third-party-web (CDT provides its own copy). This prevents duplications of 40+ KB.
+  if (isDevtools(entryPath)) {
+    shimsObj['third-party-web/nostats-subset.js'] = 'export default {};';
+  }
+
   for (const modulePath of modulesToIgnore) {
     shimsObj[modulePath] = 'export default {}';
   }

clients/devtools/devtools-entry.js

@@ -14,6 +14,7 @@ import lighthouse, {navigation, startTimespan, snapshot} from '../../core/index.
 import {lookupLocale} from '../../core/lib/i18n/i18n.js';
 import {registerLocaleData, getCanonicalLocales} from '../../shared/localization/format.js';
 import * as constants from '../../core/config/constants.js';
+import thirdPartyWeb from '../../core/lib/third-party-web.js';
 
 // Rollup seems to overlook some references to `Buffer`, so it must be made explicit.
 // (`parseSourceMapFromDataUrl` breaks without this)
@@ -68,47 +69,16 @@ function lookupCanonicalLocale(locales) {
   return lookupLocale(locales, getCanonicalLocales());
 }
 
-/**
- * TODO: Expose api directly when DevTools usage is updated.
- * @param {string} url
- * @param {{page: LH.Puppeteer.Page, config?: LH.Config, flags?: LH.Flags}} args
- */
-function runLighthouseNavigation(url, {page, ...options}) {
-  return navigation(page, url, options);
-}
-
-/**
- * TODO: Expose api directly when DevTools usage is updated.
- * @param {{page: LH.Puppeteer.Page, config?: LH.Config, flags?: LH.Flags}} args
- */
-function startLighthouseTimespan({page, ...options}) {
-  return startTimespan(page, options);
-}
-
-/**
- * TODO: Expose api directly when DevTools usage is updated.
- * @param {{page: LH.Puppeteer.Page, config?: LH.Config, flags?: LH.Flags}} args
- */
-function runLighthouseSnapshot({page, ...options}) {
-  return snapshot(page, options);
-}
-
 // Expose only in DevTools' worker
 if (typeof self !== 'undefined') {
   // TODO: refactor and delete `global.isDevtools`.
   global.isDevtools = true;
 
-  // @ts-expect-error
-  self.runLighthouseNavigation = runLighthouseNavigation;
   // @ts-expect-error
   self.navigation = navigation;
   // @ts-expect-error
-  self.startLighthouseTimespan = startLighthouseTimespan;
-  // @ts-expect-error
   self.startTimespan = startTimespan;
   // @ts-expect-error
-  self.runLighthouseSnapshot = runLighthouseSnapshot;
-  // @ts-expect-error
   self.snapshot = snapshot;
   // @ts-expect-error
   self.createConfig = createConfig;
@@ -119,6 +89,8 @@ if (typeof self !== 'undefined') {
   // TODO: expose as lookupCanonicalLocale in LighthouseService.ts?
   // @ts-expect-error
   self.lookupLocale = lookupCanonicalLocale;
+  // @ts-expect-error
+  self.thirdPartyWeb = thirdPartyWeb;
 } else {
   // For the bundle smoke test.
   // @ts-expect-error

core/lib/third-party-web.js

@@ -4,7 +4,18 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import thirdPartyWeb from 'third-party-web/nostats-subset.js';
+import thirdPartyWeb_ from 'third-party-web/nostats-subset.js';
+
+let thirdPartyWeb = thirdPartyWeb_;
+
+/**
+ * For use by DevTools.
+ *
+ * @param {typeof import('third-party-web/nostats-subset.js')} providedThirdPartyWeb
+ */
+function provideThirdPartyWeb(providedThirdPartyWeb) {
+  thirdPartyWeb = providedThirdPartyWeb;
+}
 
 /** @typedef {import("third-party-web").IEntity} ThirdPartyEntity */
 /** @typedef {import("third-party-web").IProduct} ThirdPartyProduct */
@@ -45,6 +56,7 @@ function isFirstParty(url, mainDocumentEntity) {
 }
 
 export default {
+  provideThirdPartyWeb,
   getEntity,
   getProduct,
   isThirdParty,