chrome-devtools-frontend 1.0.1674545 → 1.0.1676824

package/.agents/skills/devtools-ux-writing-refactor/SKILL.md

@@ -59,14 +59,16 @@ Locate all TypeScript files in the target folder that define `const UIStrings =
    * For panel names that consist of two words, capitalize only the first word (for example, `Developer resources panel`).
    * When you use the panel name in combination with another word, capitalize the panel name but not the other word, unless it is a proper noun (for example, `Console view` or `DevTools Console`).
    * In some cases, ambiguity exists whether a word refers to a UX element in DevTools (such as a panel name) or a concept (such as developer terminology). For example, this occurs with `console`, `issue`, or `network`. Decide based on the surrounding context and code (for example, `Console view`, `Console sidebar`, and `Console prompt` versus `console message`, `console warning`, `console log`, `copy console`, `clear console`, and `console history`; `Show Network` versus `network log` and `network filter`).
+   * **Casing in `@description` comments:** Apply the same capitalization rules to `@description` comments: ONLY panel names use sentence case (for example, `Memory panel`, `Console panel`, `Performance panel`). Feature names, view names, tool names, and sub-components MUST be all lowercase unless they appear at the very start of the description sentence or phrase (for example, `heap profiler`, `heap snapshot view`, `isolate selector`, `allocation sampling`, `command menu`, `summary view`).
 6. **Punctuation and actionability:** Remove trailing periods from single-sentence labels or titles. Ensure that error messages instruct the user how to recover (for example, `Shorten filename to 64 characters or less` instead of `Invalid filename`). Ensure that ARIA labels and multi-sentence tooltips have consistent terminal punctuation.
 7. **Terminology (glossary):** Strictly follow standard DevTools UI terminology and letter casing (`panel`, `tab`, `drawer`, `sidebar`, `datagrid` or `table`, `action bar`, `status bar`, and `live expressions section`). Never use `pane` or call tabs `panes` in UI strings or localization (L10n) comments.
 8. **Localization (L10n) comments:** Ensure that every string in `UIStrings` has a preceding `@description` comment that explains where and when it appears. This information is used for translators who need to understand the context in which the string is used.
+   * **Research usage sites:** Before modifying or writing `@description` comments, always research where the string key (for example, `UIStrings.keyName`) is used in the codebase (`grep_search` or view surrounding code). Determine the exact UI element type (for example, button label, tooltip, table column header, context menu item, checkbox label, aria label, status bar message, or dropdown option) and location (panel, view, sidebar, or dialog).
+   * **Avoid redundancy and repetition:** Do not simply repeat or restate the UI string value in the description (for example, avoid `@description Text for clear all profiles` for `'Clear all profiles'`). Instead, describe the UI role, element type, and location (for example, `@description Tooltip text for the clear button in the profiles sidebar of the Memory panel.`).
+   * **Consistency across similar descriptions:** Ensure consistent phrasing structure across descriptions of similar UI strings in the same module and throughout DevTools (for example, use uniform patterns like `Tooltip text for the <action> button in <location>.` or `Column header in <table_name> datagrid for <purpose>.`).
+   * **Casing in descriptions:** Strictly follow the rule that ONLY panel names use sentence case (for example, `Memory panel`, `Console panel`, `Profiles panel`), while feature names, views, tools, and sub-components are ALL LOWERCASE (for example, `heap profiler`, `heap snapshot view`, `isolate selector`, `allocation sampling`, `summary view`), unless appearing at the start of the description sentence or phrase.
    * **Straight apostrophes in comments:** Always use straight apostrophes (`'`) in code comments (such as `@description` comments and L10n annotations), even if the UI string itself uses curly apostrophes (`’`).
-   * Use precise terminology and correct letter casing (see rule 7). Ensure that the description is easy to understand and provides enough context for translators. For descriptions that are underspecified or
-   ambiguous, figure out the context by looking at the use site of the
-   string and come up with a better description.
-   * You must explicitly document all standard placeholders (`{PH1}`, `{url}`, `{index}`) with runtime data examples (for example, `@example {https://example.com} url`).
+   * **Placeholders:** You must explicitly document all standard placeholders (`{PH1}`, `{url}`, `{index}`) with runtime data examples (for example, `@example {https://example.com} url`).
    * *ICU plural variables:* The i18n tool automatically parses variables in ICU plural format (such as `n` in `{n, plural, =0 {No issues} ...}`) as numeric counts, so they don’t require an `@example` tag in the `@description` comment.
    * *Terminal punctuation:* Every `@description` comment must end with a period (`.`), even if it is a single phrase or sentence.
 
@@ -74,11 +76,26 @@ Locate all TypeScript files in the target folder that define `const UIStrings =
 
 Also consult the style guides at `google3/experimental/users/rachelandrew/tools/chrome_writing/knowledge/style/` for applicable guidelines.
 
-## 5. Pause for confirmation
+## 5. Subagent critique of proposed changes
+
+Before running tests or pausing for user review, enter an iterative critique loop with a subagent:
+1. **Invoke subagent:** Use `invoke_subagent` with `TypeName: "self"` and `Role: "UX Writing Reviewer"`.
+2. **Prompt instructions:** Provide the subagent with the `git diff` of your changes and instruct it to critique the refactoring against the **8-point UX writing checklist**:
+   - Are `@description` comments specific, informative, and non-redundant (not restating string values)?
+   - Are usage sites researched so the UI element type and location are accurately described?
+   - Do `@description` comments and string values strictly follow the casing rule (ONLY panel names in sentence case, feature/view names in all lowercase unless starting a phrase)?
+   - Are contractions, curly apostrophes, ellipses, and placeholders (`{PH1}`) correctly formatted?
+   - Were all references in sibling code/tests properly updated?
+3. **Address feedback and loop:**
+   - If the subagent raises any feedback, concerns, or suggested fixes, apply the necessary code changes.
+   - Re-invoke the subagent with the updated `git diff` to re-critique the changes.
+   - **Repeat this loop continuously as long as the subagent has feedback, until the subagent explicitly approves the diff with no remaining issues.**
+
+## 6. Pause for confirmation
 
 Ask the user for a preliminary review of the proposed changes before proceeding. If the user has feedback, address feedback and pause again for confirmation.
 
-## 6. Mandatory verification and testing
+## 7. Mandatory verification and testing
 
 Don’t finish an edit without running the linter and test suite. In DevTools, i18n placeholder changes or string edits can easily break tests or linter rules.
 
@@ -89,18 +106,18 @@ Don’t finish an edit without running the linter and test suite. In DevTools, i
    npm run lint -- <folder_path>
    ```
 3. **Run unit tests**
-   Run unit tests for the target folder first for fast feedback. Then, run the full test suite to verify that no cross-module regressions or snapshot failures occurred:
+   You MUST run the full test suite using `npm run test` to verify that no cross-module regressions, e2e test failures, or snapshot regressions occurred. Do not only run a subset of tests:
    ```bash
-   npm run test -- <folder_path>
    npm run test
    ```
+   *(Optionally, you may run `npm run test -- <folder_path>` first for rapid initial feedback during iterative development, but running `npm run test` for full test suite coverage is required before completing the task.)*
 4. **Run presubmit checks**
    Ensure that you commit or stage all changes before you run presubmit checks:
    ```bash
    git cl presubmit -u
    ```
 
-## 7. Summarize changes and upload the CL
+## 8. Summarize changes and upload the CL
 
 When all tests and presubmit checks pass, commit your changes and upload the CL to Gerrit. Don’t use `[uxw]` as a prefix.
 

package/WATCHLISTS

@@ -13,9 +13,6 @@
     'all': {
       'filepath': '.',
     },
-    'chromedriver': {
-      'filepath': 'front_end/emulated_devices/',
-    },
     'lighthouse': {
       'filepath': '[Ll]ighthouse',
     },
@@ -26,7 +23,6 @@
 
   'WATCHLISTS': {
     'all': ['devtools-reviews+devtools@chromium.org'],
-    'chromedriver': [],
     'lighthouse': ['lighthouse-eng+devtools-cl@google.com'],
     'third_party': ['devtools-reviews+third_party@chromium.org'],
   },

package/front_end/core/common/ParsedURL.ts

@@ -533,7 +533,11 @@ export class ParsedURL {
       return 'data:' as Platform.DevToolsPath.UrlString;
     }
     const scheme = this.isBlobURL() ? this.blobInnerScheme : this.scheme;
-    return scheme + '://' + this.domain() as Platform.DevToolsPath.UrlString;
+    const domain = this.domain();
+    if (!scheme && !domain) {
+      return '' as Platform.DevToolsPath.UrlString;
+    }
+    return scheme + '://' + domain as Platform.DevToolsPath.UrlString;
   }
 
   urlWithoutScheme(): string {

package/front_end/core/common/SettingRegistration.ts

@@ -82,12 +82,34 @@ const str_ = i18n.i18n.registerUIStrings('core/common/SettingRegistration.ts', U
 const i18nString = i18n.i18n.getLocalizedString.bind(undefined, str_);
 let registeredSettings: SettingRegistration[] = [];
 const settingNameSet = new Set<string>();
+const orderValuesBySettingCategory = new Map<SettingCategory, Set<number>>();
+
+export function registerCategoryOrder(category?: SettingCategory, order?: number): void {
+  if (category && typeof order === 'number') {
+    let orderValues = orderValuesBySettingCategory.get(category);
+    if (!orderValues) {
+      orderValues = new Set();
+      orderValuesBySettingCategory.set(category, orderValues);
+    }
+    if (orderValues.has(order)) {
+      throw new Error(`Duplicate order value '${order}' for settings category '${category}'`);
+    }
+    orderValues.add(order);
+  }
+}
+
+export function removeCategoryOrder(category?: SettingCategory, order?: number): void {
+  if (category && typeof order === 'number') {
+    orderValuesBySettingCategory.get(category)?.delete(order);
+  }
+}
 
 export function registerSettingExtension(registration: SettingRegistration): void {
   const settingName = registration.settingName;
   if (settingNameSet.has(settingName)) {
     throw new Error(`Duplicate setting name '${settingName}'`);
   }
+  registerCategoryOrder(registration.category, registration.order);
   settingNameSet.add(settingName);
   registeredSettings.push(registration);
 }
@@ -98,19 +120,17 @@ export function getRegisteredSettings(): SettingRegistration[] {
 
 export function registerSettingsForTest(settings: SettingRegistration[], forceReset = false): void {
   if (registeredSettings.length === 0 || forceReset) {
-    registeredSettings = settings;
-    settingNameSet.clear();
+    resetSettings();
     for (const setting of settings) {
-      const settingName = setting.settingName;
-      if (settingNameSet.has(settingName)) {
-        throw new Error(`Duplicate setting name '${settingName}'`);
-      }
-      settingNameSet.add(settingName);
+      registerSettingExtension(setting);
     }
   }
 }
 
 export function resetSettings(): void {
+  for (const setting of registeredSettings) {
+    removeCategoryOrder(setting.category, setting.order);
+  }
   registeredSettings = [];
   settingNameSet.clear();
 }
@@ -120,7 +140,8 @@ export function maybeRemoveSettingExtension(settingName: string): boolean {
   if (settingIndex < 0 || !settingNameSet.delete(settingName)) {
     return false;
   }
-  registeredSettings.splice(settingIndex, 1);
+  const [removed] = registeredSettings.splice(settingIndex, 1);
+  removeCategoryOrder(removed.category, removed.order);
   return true;
 }
 

package/front_end/core/common/Settings.ts

@@ -10,7 +10,6 @@ import type {EventDescriptor, EventTargetEvent, GenericEvents} from './EventTarg
 import {ObjectWrapper} from './Object.js';
 import {
   getLocalizedSettingsCategory,
-  type LearnMore,
   maybeRemoveSettingExtension,
   type RegExpSettingItem,
   registerSettingExtension,
@@ -112,7 +111,6 @@ export class Settings {
   readonly #settingRegistrations: SettingRegistration[];
   readonly #sessionStorage = new SettingsStorage({});
   settingNameSet = new Set<string>();
-  orderValuesBySettingCategory = new Map<SettingCategory, Set<number>>();
   #eventSupport = new ObjectWrapper<GenericEvents>();
   #registry = new Map<string, Setting<unknown>>();
   readonly moduleSettings = new Map<string, Setting<unknown>>();
@@ -215,19 +213,9 @@ export class Settings {
 
   private registerModuleSetting(setting: Setting<unknown>): void {
     const settingName = setting.name;
-    const category = setting.category();
-    const order = setting.order();
     if (this.settingNameSet.has(settingName)) {
       throw new Error(`Duplicate Setting name '${settingName}'`);
     }
-    if (category && order) {
-      const orderValues = this.orderValuesBySettingCategory.get(category) || new Set();
-      if (orderValues.has(order)) {
-        throw new Error(`Duplicate order value '${order}' for settings category '${category}'`);
-      }
-      orderValues.add(order);
-      this.orderValuesBySettingCategory.set(category, orderValues);
-    }
     this.settingNameSet.add(settingName);
     this.moduleSettings.set(setting.name, setting);
   }
@@ -529,13 +517,6 @@ export class Setting<V> {
     this.eventSupport.removeEventListener(this.name, listener, thisObject);
   }
 
-  title(): Platform.UIString.LocalizedString {
-    if (this.#registration?.title) {
-      return this.#registration.title();
-    }
-    return '' as Platform.UIString.LocalizedString;
-  }
-
   setRequiresUserAction(requiresUserAction: boolean): void {
     this.#requiresUserAction = requiresUserAction;
   }
@@ -635,57 +616,6 @@ export class Setting<V> {
     return this.#type ?? this.#registration?.settingType ?? null;
   }
 
-  options(): SimpleSettingOption[] {
-    if (this.#registration && this.#registration.options) {
-      return this.#registration.options.map(opt => {
-        const {value, title, text, raw} = opt;
-        return {
-          value,
-          title: title(),
-          text: typeof text === 'function' ? text() : text,
-          raw,
-        };
-      });
-    }
-    return [];
-  }
-
-  reloadRequired(): boolean|null {
-    if (this.#registration) {
-      return this.#registration.reloadRequired || null;
-    }
-    return null;
-  }
-
-  category(): SettingCategory|null {
-    if (this.#registration) {
-      return this.#registration.category || null;
-    }
-    return null;
-  }
-
-  tags(): string|null {
-    if (this.#registration && this.#registration.tags) {
-      // Get localized keys and separate by null character to prevent fuzzy matching from matching across them.
-      return this.#registration.tags.map(tag => tag()).join('\0');
-    }
-    return null;
-  }
-
-  order(): number|null {
-    if (this.#registration) {
-      return this.#registration.order || null;
-    }
-    return null;
-  }
-
-  /**
-   * See {@link LearnMore} for more info
-   */
-  learnMore(): LearnMore|null {
-    return this.#registration?.learnMore ?? null;
-  }
-
   private printSettingsSavingError(message: string, value: string): void {
     const errorMessage =
         'Error saving setting with name: ' + this.name + ', value length: ' + value.length + '. Error: ' + message;
@@ -777,10 +707,3 @@ export {
   SettingRegistration,
   SettingType,
 };
-
-export interface SimpleSettingOption {
-  value: string|boolean;
-  title: string;
-  text?: string;
-  raw?: boolean;
-}

package/front_end/core/root/Runtime.ts

@@ -484,10 +484,6 @@ interface GdpProfilesAvailability {
   enterprisePolicyValue: GdpProfilesEnterprisePolicyValue;
 }
 
-interface LiveEdit {
-  enabled: boolean;
-}
-
 interface ExtensionsOnChromeUrls {
   enabled: boolean;
 }
@@ -540,6 +536,9 @@ interface DevToolsPlusButton {
 interface DevToolsInstrumentationBreakpoints {
   enabled: boolean;
 }
+interface HostConfigDevToolsComments {
+  enabled: boolean;
+}
 
 /**
  * The host configuration that we expect from the DevTools back-end.
@@ -585,7 +584,6 @@ export type HostConfig = Platform.TypeScriptUtilities.RecursivePartial<{
   devToolsGlobalAiButton: GlobalAiButton,
   devToolsGdpProfiles: GdpProfiles,
   devToolsGdpProfilesAvailability: GdpProfilesAvailability,
-  devToolsLiveEdit: LiveEdit,
   devToolsFlexibleLayout: DevToolsFlexibleLayout,
   deviceBoundSessionsDebugging: DeviceBoundSessionsDebugging,
   devToolsEnableDurableMessages: DevToolsEnableDurableMessages,
@@ -599,6 +597,7 @@ export type HostConfig = Platform.TypeScriptUtilities.RecursivePartial<{
   devToolsPlusButton: DevToolsPlusButton,
   devToolsInstrumentationBreakpoints: DevToolsInstrumentationBreakpoints,
   extensionsOnChromeUrls: ExtensionsOnChromeUrls,
+  devToolsComments: HostConfigDevToolsComments,
 }>;
 
 /**

package/front_end/core/sdk/CSSMetadata.ts

@@ -64,9 +64,6 @@ export class CSSMetadata {
       if (Boolean(runtimeFlagStatus) && runtimeFlagStatus !== 'stable') {
         continue;
       }
-      if (!CSS.supports(propertyName, 'initial')) {
-        continue;
-      }
       this.#values.push(propertyName);
 
       if (property.inherited) {
@@ -113,7 +110,7 @@ export class CSSMetadata {
     for (const propertyName of this.#longhands.keys()) {
       // skip "all" because it is a CSS-wide keyword
       // skip shorthands that are defined in generatedPropertyValues because they have their own value sets
-      if (propertyName === 'all' || propertyValueSets.has(propertyName)) {
+      if (propertyName === 'all' || propertyName in SupportedCSSProperties.generatedPropertyValues) {
         continue;
       }
       const longhands = this.#longhands.get(propertyName);
@@ -121,6 +118,10 @@ export class CSSMetadata {
         continue;
       }
       const values = new Array<string>();
+      const propertyValueSet = propertyValueSets.get(propertyName);
+      if (propertyValueSet) {
+        values.push(...propertyValueSet);
+      }
       for (const longhand of longhands) {
         const longhandValues = propertyValueSets.get(longhand);
         if (!longhandValues) {
@@ -143,11 +144,6 @@ export class CSSMetadata {
           }
         }
       }
-      for (const commonKeyword of CommonKeywords) {
-        if (!values.has(commonKeyword) && CSS.supports(propertyName, commonKeyword)) {
-          values.add(commonKeyword);
-        }
-      }
 
       this.#propertyValues.set(propertyName, [...values]);
     }

package/front_end/core/sdk/CSSPropertyParserMatchers.ts

@@ -156,42 +156,22 @@ export class VariableNameMatcher extends matcherBase(VariableNameMatch) {
   }
 
   override matches(node: CodeMirror.SyntaxNode, matching: BottomUpTreeMatching): VariableNameMatch|null {
-    if (node.name !== 'VariableName' && node.name !== 'FeatureName' && node.name !== 'KeywordQuery') {
-      // TODO(b/484268589): The result shouldn't be KeywordQuery, but currently
-      // sometimes Lezer parses it that way. Fix this when Lezer is fixed.
+    if (!node.parent) {
       return null;
     }
-    const rawText = matching.ast.text(node);
-    if (!rawText.startsWith('--')) {
+    // TODO(b/484268589): When the misspelling is removed from Lezer we can remove that case.
+    if (node.name !== 'FeatureName' && node.name !== 'PropertyName' && node.name !== 'ProperyName') {
       return null;
     }
-
-    let cur: CodeMirror.SyntaxNode|null = node.parent;
-    let foundStyleCall: CodeMirror.SyntaxNode|null = null;
-    while (cur) {
-      if (cur.name === 'CallExpression') {
-        return null;
-      }
-      if (cur.name === 'CallQuery') {
-        const callee = cur.getChild('QueryCallee');
-        if (callee && matching.ast.text(callee) === 'style') {
-          foundStyleCall = cur;
-          break;
-        }
-        return null;
-      }
-      cur = cur.parent;
+    if (node.parent.name !== 'StyleFeature' && node.parent.name !== 'StyleRange') {
+      return null;
     }
-
-    if (!foundStyleCall) {
+    const rawText = matching.ast.text(node);
+    if (!rawText.startsWith('--')) {
       return null;
     }
 
-    // When parsing style(--foo > 10px), Lezer thinks it is a KeywordQuery and
-    // includes the > in the token with --foo. We need to strip it.
-    const text = node.name === 'KeywordQuery' ? rawText.split(/\s|[>!=<:]/)[0] : rawText;
-
-    return new VariableNameMatch(node, text, this.matchedStyles, this.style);
+    return new VariableNameMatch(node, rawText, this.matchedStyles, this.style);
   }
 }
 

package/front_end/core/sdk/DebuggerModel.ts

@@ -137,6 +137,12 @@ export const WASM_SYMBOLS_PRIORITY = [
   Protocol.Debugger.DebugSymbolsType.SourceMap,
 ];
 
+export const skipAllPausesSettingDescriptor: Common.Settings.SettingDescriptor<boolean> = {
+  name: 'skip-all-pauses',
+  type: Common.Settings.SettingType.BOOLEAN,
+  defaultValue: false,
+};
+
 export class DebuggerModel extends SDKModel<EventTypes> {
   readonly agent: ProtocolProxyApi.DebuggerApi;
   #runtimeModel: RuntimeModel;
@@ -149,6 +155,7 @@ export class DebuggerModel extends SDKModel<EventTypes> {
   #selectedCallFrame: CallFrame|null = null;
   #debuggerEnabled = false;
   #debuggerId: string|null = null;
+  readonly #skipAllPausesSetting: Common.Settings.Setting<boolean>;
   #skipAllPausesTimeout?: ReturnType<typeof setTimeout>;
   #beforePausedCallback: ((arg0: DebuggerPausedDetails, stepOver: Location|null) => Promise<boolean>)|null = null;
   #computeAutoStepRangesCallback: ((arg0: StepMode, arg1: CallFrame) => Promise<Array<{
@@ -179,10 +186,12 @@ export class DebuggerModel extends SDKModel<EventTypes> {
                                  compiledURL, sourceMappingURL, payload, target.targetManager().getConsole(), script));
 
     const settings = this.target().targetManager().settings;
+    this.#skipAllPausesSetting = settings.resolve(skipAllPausesSettingDescriptor);
     settings.resolve(pauseOnExceptionEnabledSettingDescriptor)
         .addChangeListener(this.pauseOnExceptionStateChanged, this);
     settings.resolve(pauseOnCaughtExceptionSettingDescriptor)
         .addChangeListener(this.pauseOnExceptionStateChanged, this);
+    this.#skipAllPausesSetting.addChangeListener(this.skipAllPausesChanged, this);
     settings.moduleSetting('pause-on-uncaught-exception').addChangeListener(this.pauseOnExceptionStateChanged, this);
     settings.moduleSetting('disable-async-stack-traces').addChangeListener(this.asyncStackTracesStateChanged, this);
     settings.moduleSetting('breakpoints-active').addChangeListener(this.breakpointsActiveChanged, this);
@@ -256,6 +265,11 @@ export class DebuggerModel extends SDKModel<EventTypes> {
     }
     this.#debuggerEnabled = true;
 
+    let skipAllPausesPromise: Promise<Protocol.ProtocolResponseWithError>|undefined;
+    if (this.#skipAllPausesSetting.get()) {
+      skipAllPausesPromise = this.agent.invoke_setSkipAllPauses({skip: true});
+    }
+
     // Set a limit for the total size of collected script sources retained by debugger.
     // 10MB for remote frontends, 100MB for others.
     const isRemoteFrontend = Root.Runtime.Runtime.queryParam('remoteFrontend') || Root.Runtime.Runtime.queryParam('ws');
@@ -267,14 +281,14 @@ export class DebuggerModel extends SDKModel<EventTypes> {
         instrumentation: Protocol.Debugger.SetInstrumentationBreakpointRequestInstrumentation.BeforeScriptExecution,
       });
     }
+    const settings = this.target().targetManager().settings;
     this.pauseOnExceptionStateChanged();
     void this.asyncStackTracesStateChanged();
-    const settings = this.target().targetManager().settings;
     if (!settings.moduleSetting('breakpoints-active').get()) {
       this.breakpointsActiveChanged();
     }
     this.dispatchEventToListeners(Events.DebuggerWasEnabled, this);
-    const [enableResult] = await Promise.all([enablePromise, instrumentationPromise]);
+    const [enableResult] = await Promise.all([enablePromise, instrumentationPromise, skipAllPausesPromise]);
     this.registerDebugger(enableResult);
   }
 
@@ -344,12 +358,24 @@ export class DebuggerModel extends SDKModel<EventTypes> {
   }
 
   private skipAllPauses(skip: boolean): void {
+    if (this.#skipAllPausesSetting.get()) {
+      return;
+    }
     clearTimeout(this.#skipAllPausesTimeout);
 
     void this.agent.invoke_setSkipAllPauses({skip});
   }
 
+  private skipAllPausesChanged(): void {
+    const skip = this.#skipAllPausesSetting.get();
+    clearTimeout(this.#skipAllPausesTimeout);
+    void this.agent.invoke_setSkipAllPauses({skip});
+  }
+
   skipAllPausesUntilReloadOrTimeout(timeout: number): void {
+    if (this.#skipAllPausesSetting.get()) {
+      return;
+    }
     clearTimeout(this.#skipAllPausesTimeout);
 
     void this.agent.invoke_setSkipAllPauses({skip: true});
@@ -649,16 +675,16 @@ export class DebuggerModel extends SDKModel<EventTypes> {
     this.dispatchEventToListeners(Events.DebuggerResumed, this);
   }
 
-  parsedScriptSource(
-      scriptId: Protocol.Runtime.ScriptId, sourceURL: Platform.DevToolsPath.UrlString, startLine: number,
-      startColumn: number, endLine: number, endColumn: number,
-      // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
-      executionContextId: number, hash: string, executionContextAuxData: any, isLiveEdit: boolean,
-      sourceMapURL: string|undefined, hasSourceURLComment: boolean, hasSyntaxError: boolean, length: number,
-      isModule: boolean|null, originStackTrace: Protocol.Runtime.StackTrace|null, codeOffset: number|null,
-      scriptLanguage: string|null, debugSymbols: Protocol.Debugger.DebugSymbols[]|null,
-      embedderName: Platform.DevToolsPath.UrlString|null, buildId: string|null): Script {
+  parsedScriptSource(scriptId: Protocol.Runtime.ScriptId, sourceURL: Platform.DevToolsPath.UrlString, startLine: number,
+                     startColumn: number, endLine: number, endColumn: number,
+                     // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration
+                     // eslint-disable-next-line @typescript-eslint/no-explicit-any
+                     executionContextId: number, hash: string, executionContextAuxData: any,
+                     sourceMapURL: string|undefined, hasSourceURLComment: boolean, hasSyntaxError: boolean,
+                     length: number, isModule: boolean|null, originStackTrace: Protocol.Runtime.StackTrace|null,
+                     codeOffset: number|null, scriptLanguage: string|null,
+                     debugSymbols: Protocol.Debugger.DebugSymbols[]|null,
+                     embedderName: Platform.DevToolsPath.UrlString|null, buildId: string|null): Script {
     const knownScript = this.#scripts.get(scriptId);
     if (knownScript) {
       return knownScript;
@@ -670,10 +696,9 @@ export class DebuggerModel extends SDKModel<EventTypes> {
 
     const selectedDebugSymbol =
         DebuggerModel.selectSymbolSource(debugSymbols, this.target().targetManager().getConsole());
-    const script = new Script(
-        this, scriptId, sourceURL, startLine, startColumn, endLine, endColumn, executionContextId, hash,
-        isContentScript, isLiveEdit, sourceMapURL, hasSourceURLComment, length, isModule, originStackTrace, codeOffset,
-        scriptLanguage, selectedDebugSymbol, embedderName, buildId);
+    const script = new Script(this, scriptId, sourceURL, startLine, startColumn, endLine, endColumn, executionContextId,
+                              hash, isContentScript, sourceMapURL, hasSourceURLComment, length, isModule,
+                              originStackTrace, codeOffset, scriptLanguage, selectedDebugSymbol, embedderName, buildId);
     this.registerScript(script);
     this.dispatchEventToListeners(Events.ParsedScriptSource, script);
 
@@ -878,6 +903,7 @@ export class DebuggerModel extends SDKModel<EventTypes> {
         .removeChangeListener(this.pauseOnExceptionStateChanged, this);
     settings.resolve(pauseOnCaughtExceptionSettingDescriptor)
         .removeChangeListener(this.pauseOnExceptionStateChanged, this);
+    this.#skipAllPausesSetting.removeChangeListener(this.skipAllPausesChanged, this);
     settings.moduleSetting('disable-async-stack-traces').removeChangeListener(this.asyncStackTracesStateChanged, this);
   }
 
@@ -972,7 +998,6 @@ export enum Events {
   GlobalObjectCleared = 'GlobalObjectCleared',
   CallFrameSelected = 'CallFrameSelected',
   DebuggerIsReadyToPause = 'DebuggerIsReadyToPause',
-  ScriptSourceWasEdited = 'ScriptSourceWasEdited',
   /* eslint-enable @typescript-eslint/naming-convention */
 }
 
@@ -987,10 +1012,6 @@ export interface EventTypes {
   [Events.CallFrameSelected]: DebuggerModel;
   [Events.DebuggerIsReadyToPause]: DebuggerModel;
   [Events.DebugInfoAttached]: Script;
-  [Events.ScriptSourceWasEdited]: {
-    script: Script,
-    status: Protocol.Debugger.SetScriptSourceResponseStatus,
-  };
 }
 
 class DebuggerDispatcher implements ProtocolProxyApi.DebuggerDispatcher {
@@ -1026,7 +1047,6 @@ class DebuggerDispatcher implements ProtocolProxyApi.DebuggerDispatcher {
     executionContextId,
     hash,
     executionContextAuxData,
-    isLiveEdit,
     sourceMapURL,
     hasSourceURL,
     length,
@@ -1043,9 +1063,9 @@ class DebuggerDispatcher implements ProtocolProxyApi.DebuggerDispatcher {
     }
     this.#debuggerModel.parsedScriptSource(
         scriptId, url as Platform.DevToolsPath.UrlString, startLine, startColumn, endLine, endColumn,
-        executionContextId, hash, executionContextAuxData, Boolean(isLiveEdit), sourceMapURL, Boolean(hasSourceURL),
-        false, length || 0, isModule || null, stackTrace || null, codeOffset || null, scriptLanguage || null,
-        debugSymbols || null, embedderName as Platform.DevToolsPath.UrlString || null, buildId || null);
+        executionContextId, hash, executionContextAuxData, sourceMapURL, Boolean(hasSourceURL), false, length || 0,
+        isModule || null, stackTrace || null, codeOffset || null, scriptLanguage || null, debugSymbols || null,
+        embedderName as Platform.DevToolsPath.UrlString || null, buildId || null);
   }
 
   scriptFailedToParse({
@@ -1071,11 +1091,11 @@ class DebuggerDispatcher implements ProtocolProxyApi.DebuggerDispatcher {
     if (!this.#debuggerModel.debuggerEnabled()) {
       return;
     }
-    this.#debuggerModel.parsedScriptSource(
-        scriptId, url as Platform.DevToolsPath.UrlString, startLine, startColumn, endLine, endColumn,
-        executionContextId, hash, executionContextAuxData, false, sourceMapURL, Boolean(hasSourceURL), true,
-        length || 0, isModule || null, stackTrace || null, codeOffset || null, scriptLanguage || null, null,
-        embedderName as Platform.DevToolsPath.UrlString || null, buildId || null);
+    this.#debuggerModel.parsedScriptSource(scriptId, url as Platform.DevToolsPath.UrlString, startLine, startColumn,
+                                           endLine, endColumn, executionContextId, hash, executionContextAuxData,
+                                           sourceMapURL, Boolean(hasSourceURL), true, length || 0, isModule || null,
+                                           stackTrace || null, codeOffset || null, scriptLanguage || null, null,
+                                           embedderName as Platform.DevToolsPath.UrlString || null, buildId || null);
   }
 
   breakpointResolved({breakpointId, location}: Protocol.Debugger.BreakpointResolvedEvent): void {