drop diff

Change-Id: Ibf10ea910e34bac5fc44bf13760ceb971680dd35

Author: paulirish

src/McpContext.ts

@@ -64,8 +64,8 @@ interface McpContextOptions {
   experimentalIncludeAllPages?: boolean;
 }
 
-const DEFAULT_TIMEOUT = 30_000;
-const NAVIGATION_TIMEOUT = 60_000;
+const DEFAULT_TIMEOUT = 5_000;
+const NAVIGATION_TIMEOUT = 10_000;
 
 function getNetworkMultiplierFromString(condition: string | null): number {
   const puppeteerCondition =

src/main.ts

@@ -100,7 +100,8 @@ const logDisclaimers = () => {
   console.error(
     `chrome-devtools-mcp exposes content of the browser instance to the MCP clients allowing them to inspect,
 debug, and modify any data in the browser or DevTools.
-Avoid sharing sensitive or personal information that you do not want to share with MCP clients.`,
+Avoid sharing sensitive or personal information that you do not want to share with MCP clients.
+Performance tools may send trace URLs to the Google CrUX API to fetch real-user experience data.`,
   );
 
   if (args.usageStatistics) {
@@ -109,10 +110,6 @@ Avoid sharing sensitive or personal information that you do not want to share wi
 Google collects usage statistics to improve Chrome DevTools MCP. To opt-out, run with --no-usage-statistics.
 For more details, visit: https://github.com/ChromeDevTools/chrome-devtools-mcp#usage-statistics`,
     );
-  }
-  console.error(
-    `Performance tools may send trace URLs to the Google CrUX API to fetch real-user experience data.`,
-  );
 };
 
 const toolMutex = new Mutex();

src/manual-perf-test.ts

@@ -193,15 +193,18 @@ async function stopTracingAndAppendOutput(
   }
 }
 
+/** We tell CrUXManager to fetch data so it's available when DevTools.PerformanceTraceFormatter is invoked */
 async function populateCruxData(result: TraceResult): Promise<void> {
   logger('populateCruxData called');
   const cruxManager = DevTools.CrUXManager.CrUXManager.instance();
   // go/jtfbx
   cruxManager.setEndpointForTesting(
     'https://chromeuxreport.googleapis.com/v1/records:queryRecord?key=AIzaSyBn5gimNjhiEyA_euicSKko6IlD3HdgUfk',
   );
-  const settings = DevTools.Common.Settings.Settings.instance();
-  const cruxSetting = settings.createSetting('field-data', {enabled: true});
+  const cruxSetting =
+    DevTools.Common.Settings.Settings.instance().createSetting('field-data', {
+      enabled: true,
+    });
   cruxSetting.set({enabled: true});
 
   // Gather URLs to fetch CrUX data for
@@ -215,6 +218,7 @@ async function populateCruxData(result: TraceResult): Promise<void> {
     logger('No URLs found for CrUX data');
     return;
   }
+
   logger(
     `Fetching CrUX data for ${urlSet.size} URLs: ${Array.from(urlSet).join(', ')}`,
   );

src/tools/performance.ts

@@ -71,7 +71,7 @@ exports[`McpResponse > adds throttling setting when it is not null 1`] = `
 # test response
 ## Network emulation
 Emulating: Slow 3G
-Default navigation timeout set to 600000 ms
+Default navigation timeout set to 100000 ms
 `;
 
 exports[`McpResponse > allows response text lines to be added 1`] = `