chrome-devtools-frontend 1.0.1698733 → 1.0.1699061

package/front_end/ui/legacy/Widget.ts

@@ -1454,3 +1454,36 @@ Node.prototype.removeChildren = function(): void {
   }
   return originalRemoveChildren.call(this);
 };
+
+export interface WrapperWidgetParams {
+  widget: Widget;
+}
+
+export class WrapperWidget extends Widget {
+  #widget: Widget|null = null;
+  constructor(element: HTMLElement, _deps: never[], params?: WrapperWidgetParams) {
+    super(element);
+    this.element.style.setProperty('display', 'contents');
+    if (params?.widget) {
+      this.widget = params.widget;
+    }
+  }
+
+  set widget(widget: Widget|null) {
+    if (this.#widget === widget) {
+      return;
+    }
+    if (this.#widget) {
+      this.#widget.detach();
+    }
+    this.#widget = widget;
+    if (this.#widget) {
+      this.#widget.show(this.element, undefined, /* suppressOrphanWidgetError */ true);
+    }
+  }
+  override focus(): void {
+    if (this.#widget) {
+      this.#widget.focus();
+    }
+  }
+}

package/front_end/ui/legacy/components/cookie_table/CookiesTable.ts

@@ -54,6 +54,7 @@ export interface ViewInput {
   data: CookieData[];
   selectedKey?: string;
   editable?: boolean;
+  deletable?: boolean;
   renderInline?: boolean;
   portBindingEnabled?: boolean;
   schemeBindingEnabled?: boolean;
@@ -233,6 +234,7 @@ export class CookiesTable extends UI.Widget.VBox {
                id="cookies-table"
                striped
                ?inline=${input.renderInline}
+               ?deletable=${input.deletable}
                @create=${(e: CustomEvent<CookieData>) => input.onCreate(e.detail)}
                @refresh=${input.onRefresh}
                @deselect=${() => input.onSelect(undefined)}
@@ -448,6 +450,7 @@ export class CookiesTable extends UI.Widget.VBox {
       data: this.data,
       selectedKey: this.selectedKey,
       editable: this.#editable,
+      deletable: Boolean(this.#deleteCallback),
       renderInline: this.renderInline,
       schemeBindingEnabled: this.schemeBindingEnabled,
       portBindingEnabled: this.portBindingEnabled,

package/front_end/ui/legacy/components/data_grid/DataGridElement.ts

@@ -48,6 +48,8 @@ const DUMMY_COLUMN_ID = 'dummy';  // SortableDataGrid.create requires at least o
  * visible rows are layed out and sorting is provided out of the box.
  *
  * @property filters Set of text filters to be applied to the data grid.
+ * @attribute deletable If true, rows can be deleted through the context menu or the Delete
+ *            key, which dispatches a `delete` event on the corresponding <tr> element.
  * @attribute inline If true, the data grid will render inline instead of taking a full container height.
  * @attribute resize Column resize method, one of 'nearest' (default), 'first' or 'last'.
  * @attribute striped If true, the data grid will have striped rows.
@@ -58,7 +60,7 @@ type DataGridElementNode = SortableNode|DynamicHeightNode;
 const elementToNode = new WeakMap<Element, DataGridElementNode>();
 
 export class DataGridElement extends UI.UIUtils.HTMLElementWithLightDOMTemplate {
-  static readonly observedAttributes: string[] = ['striped', 'name', 'inline', 'resize', 'highlight'];
+  static readonly observedAttributes: string[] = ['striped', 'name', 'inline', 'resize', 'highlight', 'deletable'];
 
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
   #dataGrid!: DataGridImpl<any>;
@@ -220,6 +222,9 @@ export class DataGridElement extends UI.UIUtils.HTMLElementWithLightDOMTemplate
           this.#revealHighlightedNode(Number(newValue));
         });
         break;
+      case 'deletable':
+        this.#dataGrid.deleteCallback = this.deletable ? this.#deleteCallback.bind(this) : undefined;
+        break;
     }
   }
 
@@ -231,6 +236,14 @@ export class DataGridElement extends UI.UIUtils.HTMLElementWithLightDOMTemplate
     return hasBooleanAttribute(this, 'striped');
   }
 
+  set deletable(deletable: boolean) {
+    this.toggleAttribute('deletable', deletable);
+  }
+
+  get deletable(): boolean {
+    return hasBooleanAttribute(this, 'deletable');
+  }
+
   set inline(striped: boolean) {
     this.toggleAttribute('inline', striped);
   }
@@ -584,9 +597,6 @@ export class DataGridElement extends UI.UIUtils.HTMLElementWithLightDOMTemplate
     if (args[0] === 'refresh') {
       this.#dataGrid.refreshCallback = this.#refreshCallback.bind(this);
     }
-    if (args[0] === 'delete') {
-      this.#dataGrid.deleteCallback = this.#deleteCallback.bind(this);
-    }
   }
 
   #refreshCallback(): void {

package/front_end/ui/legacy/components/object_ui/ObjectPropertiesSection.ts

@@ -388,6 +388,10 @@ export class ObjectTreeExpansionTracker {
   }
 }
 
+const ARRAY_LOAD_THRESHOLD = 100;
+const ARRAY_BUCKET_THRESHOLD = 100;
+const ARRAY_SPARSE_ITERATION_THRESHOLD = 250000;
+
 export abstract class ObjectTreeNodeBase extends Common.ObjectWrapper.ObjectWrapper<ObjectTreeNodeBase.EventTypes> {
   #children?: NodeChildren;
   protected readonly options: ObjectTreeOptions;
@@ -715,7 +719,7 @@ class ArrayGroupTreeNode extends ObjectTreeNodeBase {
   }
 
   override async populateChildrenIfNeededImpl(): Promise<NodeChildren> {
-    if (this.#range.count > ArrayGroupingTreeElement.bucketThreshold) {
+    if (this.#range.count > ARRAY_BUCKET_THRESHOLD) {
       const ranges = await arrayRangeGroups(this.object, this.#range.fromIndex, this.#range.toIndex);
       const arrayRanges = ranges?.ranges.map(
           ([fromIndex, toIndex, count]) => new ArrayGroupTreeNode(this.object, {fromIndex, toIndex, count}, this, {
@@ -729,7 +733,7 @@ class ArrayGroupTreeNode extends ObjectTreeNodeBase {
     const result = await this.#object.callFunction(buildArrayFragment, [
       {value: this.#range.fromIndex},
       {value: this.#range.toIndex},
-      {value: ArrayGroupingTreeElement.sparseIterationThreshold},
+      {value: ARRAY_SPARSE_ITERATION_THRESHOLD},
     ]);
     if (!result.object || result.wasThrown) {
       return {};
@@ -1240,9 +1244,6 @@ export class ObjectPropertiesSectionWidget extends UI.Widget.Widget {
   };
 }
 
-/** @constant */
-const ARRAY_LOAD_THRESHOLD = 100;
-
 const maxRenderableStringLength = 10000;
 
 export const enum ObjectPropertiesMode {
@@ -1289,7 +1290,7 @@ export function populateObjectTreeContextMenu(
       {checked: object.includeNullOrUndefinedValues, jslogContext: 'show-all'});
 }
 
-export interface ObjectTreeViewInput {
+interface ObjectTreeViewInput {
   renderAsSubtree: boolean;
   objectTree?: ObjectTree;
   linkifier?: Components.Linkifier.Linkifier;
@@ -1298,8 +1299,8 @@ export interface ObjectTreeViewInput {
   skipGettersAndSetters: boolean;
   onExpand: (expanded: boolean) => void;
 }
-export type ObjectTreeView = (input: ObjectTreeViewInput, output: object, target: HTMLElement) => void;
-export const OBJECT_TREE_DEFAULT_VIEW: ObjectTreeView = (input, output, target) => {
+type ObjectTreeView = (input: ObjectTreeViewInput, output: object, target: HTMLElement) => void;
+const OBJECT_TREE_DEFAULT_VIEW: ObjectTreeView = (input, output, target) => {
   const objectTree = input.objectTree;
   if (!objectTree) {
     render(nothing, target);
@@ -1454,7 +1455,7 @@ export function renderObjectTree(
       widgetRef(ObjectTreeWidget, () => {})}></ul>`;
 }
 
-export interface ObjectPropertiesSectionViewInput {
+interface ObjectPropertiesSectionViewInput {
   objectTree: ObjectTree;
   title?: Element|TemplateResult;
   linkifier?: Components.Linkifier.Linkifier;
@@ -1464,8 +1465,8 @@ export interface ObjectPropertiesSectionViewInput {
   onExpand: (expanded: boolean) => void;
 }
 
-export type ObjectPropertiesSectionView =
-    (input: ObjectPropertiesSectionViewInput, output: object, target: HTMLElement) => void;
+type ObjectPropertiesSectionView = (input: ObjectPropertiesSectionViewInput, output: object, target: HTMLElement) =>
+    void;
 
 export const OBJECT_PROPERTIES_SECTION_DEFAULT_VIEW: ObjectPropertiesSectionView = (input, _output, target) => {
 
@@ -1674,9 +1675,9 @@ export interface ObjectPropertyViewInput {
   node: ObjectTreeNode;
   search?: UI.TreeOutline.TreeSearch<ObjectTreeNodeBase>;
 }
-export type ObjectPropertyViewOutput = undefined;
-export type ObjectPropertyView =
-    (input: ObjectPropertyViewInput, output: ObjectPropertyViewOutput, target: HTMLElement) => void;
+type ObjectPropertyViewOutput = undefined;
+type ObjectPropertyView = (input: ObjectPropertyViewInput, output: ObjectPropertyViewOutput, target: HTMLElement) =>
+    void;
 export const OBJECT_PROPERTY_DEFAULT_VIEW: ObjectPropertyView = (input, _output, target) => {
   const {property} = input.node;
   const isInternalEntries = property.synthetic && input.node.name === '[[Entries]]';
@@ -1897,7 +1898,7 @@ export class ObjectPropertyWidget extends UI.Widget.Widget {
   }
 }
 
-export class ObjectPropertyTreeElement extends UI.TreeOutline.TreeElement {
+class ObjectPropertyTreeElement extends UI.TreeOutline.TreeElement {
   property: ObjectTreeNode;
   override toggleOnClick: boolean;
   private linkifier: Components.Linkifier.Linkifier|undefined;
@@ -2232,8 +2233,8 @@ async function arrayRangeGroups(object: SDK.RemoteObject.RemoteObject, fromIndex
   return await object.callFunctionJSON(packArrayRanges, [
     {value: fromIndex},
     {value: toIndex},
-    {value: ArrayGroupingTreeElement.bucketThreshold},
-    {value: ArrayGroupingTreeElement.sparseIterationThreshold},
+    {value: ARRAY_BUCKET_THRESHOLD},
+    {value: ARRAY_SPARSE_ITERATION_THRESHOLD},
   ]);
 
   /**
@@ -2361,7 +2362,7 @@ function buildArrayFragment(
   return result;
 }
 
-export class ArrayGroupingTreeElement extends UI.TreeOutline.TreeElement {
+class ArrayGroupingTreeElement extends UI.TreeOutline.TreeElement {
   override toggleOnClick: boolean;
   private readonly linkifier: Components.Linkifier.Linkifier|undefined;
   readonly #child: ArrayGroupTreeNode;
@@ -2442,13 +2443,9 @@ export class ArrayGroupingTreeElement extends UI.TreeOutline.TreeElement {
   override onattach(): void {
     this.listItemElement.classList.add('object-properties-section-name');
   }
-
-  // These should be module constants but they are modified by layout tests.
-  static bucketThreshold = 100;
-  static sparseIterationThreshold = 250000;
 }
 
-export interface ExpandableTextViewInput {
+interface ExpandableTextViewInput {
   copyText: () => void;
   expandText: () => void;
   expanded: boolean;
@@ -2456,8 +2453,8 @@ export interface ExpandableTextViewInput {
   byteCount: number;
   text: string;
 }
-export type ExpandableTextView = (input: ExpandableTextViewInput, output: object, target: HTMLElement) => void;
-export const EXPANDABLE_TEXT_DEFAULT_VIEW: ExpandableTextView = (input, output, target) => {
+type ExpandableTextView = (input: ExpandableTextViewInput, output: object, target: HTMLElement) => void;
+const EXPANDABLE_TEXT_DEFAULT_VIEW: ExpandableTextView = (input, output, target) => {
   const totalBytesText = i18n.ByteUtilities.bytesToString(input.byteCount);
   const canExpand = input.text.length < ExpandableTextPropertyValue.MAX_DISPLAYABLE_TEXT_LENGTH;
   const onContextMenu = (e: Event): void => {

package/front_end/ui/visual_logging/KnownContextValues.ts

@@ -283,8 +283,11 @@ export const knownContextValues: Set<string> = new Set([
   'accept',
   'accept-execute-code',
   'access-control-allow-origin',
+  'accessibility-announcements.filter-by-type',
   'accessibility-default',
   'accessibility-tree',
+  'accessibility.clear-announcements',
+  'accessibility.toggle-recording',
   'accessibility.view',
   'accuracy',
   'achromatopsia',

package/package.json

@@ -90,5 +90,5 @@
     "yargs": "18.1.0",
     "yargs-unparser": "2.0.0"
   },
-  "version": "1.0.1698733"
+  "version": "1.0.1699061"
 }

package/.agents/skills/devtools-ci/SKILL.md

@@ -1,65 +0,0 @@
----
-name: devtools-ci
-description: >-
-  Triage and analyze Chromium/DevTools LUCI build results and trybot failures,
-  including high-level builder status, compile errors, and detailed test
-  failures. Use this to inspect trybot buildbucket (bb) results for a patchset
-  overview or to query ResultDB directly for comprehensive, actionable information
-  about specific test regressions and infrastructure issues.
----
-
-# LUCI Build Triage & Analysis
-
-This skill provides access to specialized triage tools maintained within the
-`depot_tools` repository. These tools are used to inspect Buildbucket results
-and ResultDB test failures.
-
-## Environment & Auth Discovery
-
-- **Environment**: Always execute these scripts using **`vpython3`** instead
-  of `python3` to ensure all hermetic dependencies (like `httplib2`) are
-  satisfied.
-- **Authentication**: Use `bb auth-info` to proactively check if you are
-  logged into Buildbucket. If not, inform the user they must run `bb auth-login`.
-
-## Locating the Tools
-
-The triage scripts and their detailed documentation are located relative to
-the `depot_tools` installation:
-
-1.  **Locate Depot Tools**: Run `which gerrit_client.py` to find your local
-    `depot_tools` root.
-2.  **Navigation**:
-    - **Buildbucket (`bb_wrapper.py`)**: Found in `<DEPOT_TOOLS_ROOT>/agents/skills/buildbucket/`
-      - *Strengths*: High-level Build info (is it failing? what steps failed?),
-        compile errors, and infra issues.
-    - **Test Results (`luci_triage.py`)**: Found in `<DEPOT_TOOLS_ROOT>/agents/skills/luci-test-results/`
-      - *Strengths*: Detailed Test info (specific test case failures in
-        ResultDB).
-
-## Troubleshooting
-
-- **"No Artifacts Found"**: This usually occurs if:
-  1. The build failed so early (e.g., bot setup) that no artifacts were produced.
-  2. The build is too old and logs have been purged.
-  3. There is an authentication issue.
-  - **Fallback**: Try `bb log <build_id> <step_name>` to see the raw output
-    for the failed step.
-
-## Available Skills in Depot Tools
-
-Once you have located the `depot_tools` directory, read the following files
-to understand how to use each tool:
-
-- **`buildbucket`**: Used for high-level builder status and compile failures.
-  - Read: `<DEPOT_TOOLS_ROOT>/agents/skills/buildbucket/SKILL.md`
-- **`luci-test-results`**: Used for detailed ResultDB test failure analysis.
-  - Read: `<DEPOT_TOOLS_ROOT>/agents/skills/luci-test-results/SKILL.md`
-
-## Usage at Runtime
-
-When you need to triage a build:
-1.  Locate `depot_tools` as described above.
-2.  Read the relevant `SKILL.md` file from `depot_tools`.
-3.  Follow the instructions in that file to execute the Python scripts (e.g.,
-    `bb_wrapper.py` or `luci_triage.py`) using their absolute paths.

package/.agents/skills/evaluate-ai-css-completion/SKILL.md

@@ -1,143 +0,0 @@
----
-name: evaluate-ai-css-completion
-description: Expose a temporary evaluation hook in DevTools and run a Puppeteer script to validate CSS code completion trigger rates.
----
-
-# Evaluate CSS Completion Skill
-
-This skill allows you to temporarily expose an evaluation hook in DevTools to measure CSS code completion trigger rates using a Puppeteer script.
-
-> [!WARNING]
-> **NEVER commit the `devtools_app.ts` patch.** It is only for local evaluation. Always revert it before uploading your CL.
-
-## Step 1: Apply the Temporary Patch
-
-Modify `front_end/entrypoints/devtools_app/devtools_app.ts` to expose the global `testCssCompletion` hook.
-
-Add the following code at the end of `front_end/entrypoints/devtools_app/devtools_app.ts`:
-
-```typescript
-// --- TEMPORARY EVALUATION HOOK ---
-// TEMPORARY PATCH - REMOVE BEFORE COMMIT
-
-import * as Host from '../../core/host/host.js';
-import * as AiCodeCompletion from '../../models/ai_code_completion/ai_code_completion.js';
-
-(self as any).testCss = {
-  getCases() {
-    return [
-      {
-        name: 'Generic CSS Test Case',
-        url: null,
-        prefix: 'h1 { font-s',
-        suffix: ' }',
-      },
-      // Add more test cases here
-    ];
-  },
-
-  async evaluate(uiSourceCodeUrl: string | null, prefix: string, suffix: string, additionalFiles?: any[]) {
-    const aidaClient = new Host.AidaClient.AidaClient();
-    const completion = new AiCodeCompletion.AiCodeCompletion.AiCodeCompletion(
-      {aidaClient},
-      AiCodeCompletion.AiCodeCompletion.ContextFlavor.STYLES
-    );
-
-    const formattedAdditionalFiles = additionalFiles?.map(f => ({
-      path: f.path || f.name,
-      content: f.content || f.text,
-      included_reason: f.included_reason ?? Host.AidaClient.Reason.RELATED_FILE,
-    }));
-
-    const result = await completion.completeCode(
-      prefix,
-      suffix,
-      prefix.length,
-      Host.AidaClient.AidaInferenceLanguage.CSS,
-      formattedAdditionalFiles
-    );
-
-    return {
-      hasSuggestion: result.response !== null && result.response.generatedSamples.length > 0,
-      suggestions: result.response?.generatedSamples.map(s => s.generationString) ?? [],
-      injectedFiles: formattedAdditionalFiles?.map(f => f.path) ?? [],
-    };
-  }
-};
-// ----------------------------------
-```
-
-## Step 2: Launch Chrome with Local DevTools
-
-Use the `npm start` script to build DevTools, launch Chrome Canary with remote debugging, and disable watch mode (so we build manually instead).
-
-You must specify a persistent user data directory so you can log in once and reuse the session:
-
-```bash
-npm start -- --no-watch --browser=canary --remote-debugging-port=9222 --user-data-dir=/tmp/devtools-ai-evaluate-css-completion
-```
-
-> [!IMPORTANT]
-> **Instructions for the Agent:**
-> After launching Chrome, you MUST print the following checklist to the user and wait for their explicit confirmation before running the evaluation script:
-> 1. Sign in to Chrome with your corporate account in the new window.
-> 2. Open DevTools, go to **Settings** (gear icon) > **AI Innovations**, and ensure **Code Completions** is enabled.
-> 3. Focus the DevTools window and press `Cmd + Option + I` (on macOS) or `Ctrl + Shift + I` (on Windows/Linux) to open DevTools-on-DevTools (inspecting DevTools itself). Go to the Network tab of the second DevTools instance and check the **Disable cache** checkbox. This ensures your local builds are loaded instead of cached versions when reloaded.
-> 4. Ask the user to reply when they are ready.
-
-## Step 3: Configure Test Cases
-
-Before running the evaluation, configure your test cases directly in `front_end/entrypoints/devtools_app/devtools_app.ts` inside the `getCases()` method of the patch you applied in Step 1.
-
-After modifying the test cases, manually run a build to compile the changes:
-```bash
-autoninja -C out/Default
-```
-
-For example, to test spacing sensitivity, update the returned array:
-```typescript
-  getCases() {
-    return [
-      {
-        name: 'With CSS Context',
-        url: null,
-        prefix: 'h1 { font-s',
-        suffix: ' }',
-        additionalFiles: [
-          {
-            path: 'other.css',
-            content: 'body { color: red; }',
-          }
-        ]
-      }
-    ];
-  }
-```
-
-## Step 4: Run the Evaluation Script
-
-Run the Puppeteer script to execute the evaluation:
-
-```bash
-node .agents/skills/evaluate-ai-css-completion/scripts/evaluate.js
-```
-
-The script will connect to the DevTools instance, trigger completions, and output the results.
-
-### Debugging and Diagnostics
-
-If you get 0% trigger rate or unexpected errors, check the following:
-
-1.  **Browser Logs:** The evaluation script forwards browser console logs (prefixed with `BROWSER LOG:`). Look for these in your terminal.
-2.  **Host Config:** The script prints the DevTools `Host Config` at the start. Verify that `"devToolsAiCodeCompletionStyles": { "enabled": true }` is present.
-3.  **GCA/AIDA Logs:** The script automatically enables verbose AIDA logging. Look for `GCA request succeeded:` in the logs:
-    *   If the response has `"usageMetadata": { "candidatesTokenCount": X }` (where X > 0) but the `candidates` array is missing or empty, the backend generated tokens but they were filtered out (e.g., due to safety classifiers or recitation blocks).
-4.  **Context Requirement:** The CSS model is sensitive to context. A minimal prompt like `h1 { font-s` might not trigger AIDA. Ensure your test cases have enough context in the `prefix` (e.g., several preceding CSS rules) or pass them via `additionalFiles` to simulate a larger stylesheet.
-
-## Step 5: Revert the Patch
-
-Once the evaluation is complete and you have recorded the results, revert the patch in `devtools_app.ts`:
-
-```bash
-git checkout front_end/entrypoints/devtools_app/devtools_app.ts
-```

package/.agents/skills/evaluate-ai-css-completion/scripts/evaluate.js

@@ -1,104 +0,0 @@
-// Copyright 2026 The Chromium Authors
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-/* eslint-disable no-console */
-
-const puppeteer = require('puppeteer-core');
-
-async function runEvaluation() {
-  console.log('Connecting to Chrome...');
-  const browser = await puppeteer.connect({browserURL: 'http://127.0.0.1:9222', defaultViewport: null});
-
-  try {
-    console.log('Opening test page (example.com)...');
-    const testPage = await browser.newPage();
-    await testPage.goto('https://example.com');
-
-    console.log('Waiting for DevTools target...');
-    const devtoolsTarget = await browser.waitForTarget(t => {
-      return t.url().startsWith('devtools://') || t.url().includes('inspector.html');
-    }, {timeout: 10000});
-
-    console.log('Connecting to DevTools page...');
-    const devtoolsPage = await devtoolsTarget.page();
-    if (!devtoolsPage) {
-      throw new Error('Could not get DevTools page object. Make sure DevTools is undocked or opened in a tab.');
-    }
-    console.log('DevTools URL:', devtoolsPage.url());
-
-    devtoolsPage.on('console', msg => console.log('BROWSER LOG:', msg.text()));
-
-    console.log('Reloading DevTools to ensure patch is loaded...');
-    try {
-      await devtoolsPage.setCacheEnabled(false);
-      await devtoolsPage.reload({waitUntil: 'domcontentloaded', timeout: 10000});
-    } catch (e) {
-      console.log('Reload warning (proceeding anyway):', e.message);
-    }
-
-    await devtoolsPage.evaluate(() => localStorage.setItem('debugAiCodeCompletionEnabled', 'true'));
-    await devtoolsPage.evaluate(() => localStorage.setItem('debugAiServicesEnabled', 'true'));
-    await devtoolsPage.evaluate(() => localStorage.setItem('debugAiAssistancePanelEnabled', 'true'));
-    await devtoolsPage.evaluate(() => localStorage.setItem('aiAssistanceStructuredLogEnabled', 'true'));
-
-    console.log('Waiting for DevTools to initialize and hook to be available...');
-    try {
-      await devtoolsPage.waitForFunction(() => typeof self.testCss?.getCases === 'function', {timeout: 15000});
-    } catch {
-      throw new Error('testCss.getCases hook not found. Did you apply the patch and build? Is this the local build?');
-    }
-
-    console.log('Fetching test cases from DevTools...');
-    const hostConfig = await devtoolsPage.evaluate(() => self.testCss.getHostConfig());
-    console.log('Host Config:', JSON.stringify(hostConfig, null, 2));
-    const testCases = await devtoolsPage.evaluate(() => self.testCss.getCases());
-    console.log(`Found ${testCases.length} test cases. Running each case 5 times.`);
-
-    let totalQueries = 0;
-    for (const tc of testCases) {
-      console.log(`\nRunning: ${tc.name}`);
-      let successCount = 0;
-      const suggestions = new Set();
-
-      for (let run = 1; run <= 5; run++) {
-        // Delay to avoid rate limiting
-        await new Promise(resolve => setTimeout(resolve, 1200));
-        try {
-          const result = await devtoolsPage.evaluate(async (url, prefix, suffix, additionalFiles) => {
-            return await self.testCss.evaluate(url, prefix, suffix, additionalFiles);
-          }, tc.url, tc.prefix, tc.suffix, tc.additionalFiles);
-
-          if (result.hasSuggestion) {
-            successCount++;
-            for (const s of result.suggestions) {
-              suggestions.add(s);
-            }
-          }
-        } catch (e) {
-          console.error(`  [Run ${run}] Failed:`, e.message);
-        }
-
-        totalQueries++;
-        if (totalQueries % 20 === 0) {
-          console.log('[INFO] Pausing for 8 seconds to prevent rate limiting...');
-          await new Promise(resolve => setTimeout(resolve, 8000));
-        }
-      }
-
-      console.log(`  Result: [Success Rate: ${successCount}/5]`);
-      if (suggestions.size > 0) {
-        console.log('  Unique Suggestions:', Array.from(suggestions));
-      }
-    }
-
-    // Cleanup tabs
-    await devtoolsPage.close();
-    await testPage.close();
-  } finally {
-    console.log('Disconnecting from Chrome...');
-    await browser.disconnect();
-  }
-}
-
-runEvaluation().catch(console.error);

package/.agents/skills/fixing-skipped-tests/SKILL.md

@@ -1,32 +0,0 @@
----
-name: fixing-skipped-tests
-description: Use this skill when unskipping a test that was previously skipped.
----
-# Fixing Skipped Tests
-
-This skill outlines the workflow for fixing skipped tests in the DevTools codebase, typically given a Chromium bug link (e.g., `crbug.com/1234567`).
-
-## Workflow
-
-1. **Extract the Bug ID**
-   - The input is typically a bug link like `crbug.com/<bugid>`. Extract the `<bugid>` from the link.
-
-2. **Create a Branch**
-   - Create a new branch named `fixed-<bugid>`.
-   - Ensure you use the instructions from the `devtools-version-control` skill to create and switch branches appropriately.
-
-3. **Search for the Skipped Tests**
-   - Search the `test/TestExpectations` file for the `<bugid>` to find skipped tests.
-   - Tests are no longer skipped using `.skip` in the code, but rather by having a `[ Skip ]` expectation in `test/TestExpectations`.
-
-4. **Iterative Fix Process**
-   - **Step 1:** Remove the `[ Skip ]` expectation for *one* of the tests associated with the bug in `test/TestExpectations` (or change it to `[ Pass Failure ]` if you are investigating flakiness).
-   - **Step 2:** Re-run the specific test using its exact test ID (e.g., `npm run test path/to/foo.test.ts:test_name`) using the guidelines from the `devtools-verification` skill. If you are dealing with flaky tests, you can use the `--repeat=x` flag (e.g. `npm run test -- --repeat=20 path/to/foo.test.ts:test_name`) to run the test multiple times to reproduce the flakiness. This flag works for both E2E and unit tests.
-   - **Step 3:** Analyze the test failure and make the necessary fixes to the code or the test itself.
-   - **Step 4:** Re-run the test to ensure the fix is correct.
-   - **Step 5:** Remove the `[crbug.com/<bugid>]` prefix from the test name string in the code if it is present.
-   - **Step 6:** Remove any comments directly above the test in the code that explain the reason why the test was skipped (e.g., `// TODO(crbug...): Flaky`).
-   - **Step 7:** Apply the fix to the remaining tests, un-skipping them one by one or in batches by modifying `test/TestExpectations`, and verifying until all are passing.
-
-5. **Verify Full Build**
-   - Before finishing, run the full verification process (TypeScript checks, linters, full test run) as required by the repository best practices.