chrome-devtools-frontend 1.0.1675430 → 1.0.1676824

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/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,
@@ -518,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;
   }
@@ -624,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;
@@ -766,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;
 }
@@ -588,7 +584,6 @@ export type HostConfig = Platform.TypeScriptUtilities.RecursivePartial<{
   devToolsGlobalAiButton: GlobalAiButton,
   devToolsGdpProfiles: GdpProfiles,
   devToolsGdpProfilesAvailability: GdpProfilesAvailability,
-  devToolsLiveEdit: LiveEdit,
   devToolsFlexibleLayout: DevToolsFlexibleLayout,
   deviceBoundSessionsDebugging: DeviceBoundSessionsDebugging,
   devToolsEnableDurableMessages: DevToolsEnableDurableMessages,

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 {

package/front_end/core/sdk/ResourceTreeModel.ts

@@ -247,6 +247,18 @@ export class ResourceTreeModel extends SDKModel<EventTypes> {
     }
   }
 
+  navigatedWithinDocument(frameId: Protocol.Page.FrameId, url: string): void {
+    const frame = this.framesInternal.get(frameId);
+    if (!frame) {
+      return;
+    }
+    frame.navigatedWithinDocument(url as Platform.DevToolsPath.UrlString);
+    if (frame.isMainFrame()) {
+      this.target().setInspectedURL(frame.url);
+    }
+    this.dispatchEventToListeners(Events.FrameNavigatedWithinDocument, frame);
+  }
+
   frameDetached(frameId: Protocol.Page.FrameId, isSwap: boolean): void {
     // Do nothing unless cached resource tree is processed - it will overwrite everything.
     if (!this.#cachedResourcesProcessed) {
@@ -584,6 +596,7 @@ export enum Events {
   /* eslint-disable @typescript-eslint/naming-convention -- Used by web_tests. */
   FrameAdded = 'FrameAdded',
   FrameNavigated = 'FrameNavigated',
+  FrameNavigatedWithinDocument = 'FrameNavigatedWithinDocument',
   FrameDetached = 'FrameDetached',
   FrameResized = 'FrameResized',
   FrameWillNavigate = 'FrameWillNavigate',
@@ -607,6 +620,7 @@ export enum Events {
 export interface EventTypes {
   [Events.FrameAdded]: ResourceTreeFrame;
   [Events.FrameNavigated]: ResourceTreeFrame;
+  [Events.FrameNavigatedWithinDocument]: ResourceTreeFrame;
   [Events.FrameDetached]: {frame: ResourceTreeFrame, isSwap: boolean};
   [Events.FrameResized]: void;
   [Events.FrameWillNavigate]: ResourceTreeFrame;
@@ -736,6 +750,10 @@ export class ResourceTreeFrame {
     }
   }
 
+  navigatedWithinDocument(url: Platform.DevToolsPath.UrlString): void {
+    this.#url = url;
+  }
+
   resourceTreeModel(): ResourceTreeModel {
     return this.#model;
   }
@@ -1096,7 +1114,8 @@ export class PageDispatcher implements ProtocolProxyApi.PageDispatcher {
   frameStartedNavigating({}: Protocol.Page.FrameStartedNavigatingEvent): void {
   }
 
-  navigatedWithinDocument({}: Protocol.Page.NavigatedWithinDocumentEvent): void {
+  navigatedWithinDocument({frameId, url}: Protocol.Page.NavigatedWithinDocumentEvent): void {
+    this.#resourceTreeModel.navigatedWithinDocument(frameId, url);
   }
 
   frameResized(): void {

package/front_end/core/sdk/Script.ts

@@ -36,7 +36,6 @@ import * as TextUtils from '../text_utils/text_utils.js';
 import {
   COND_BREAKPOINT_SOURCE_URL,
   type DebuggerModel,
-  Events,
   Location,
   LOGPOINT_SOURCE_URL,
 } from './DebuggerModel.js';
@@ -80,7 +79,6 @@ export class Script implements TextUtils.ContentProvider.ContentProvider, FrameA
   executionContextId: number;
   hash: string;
   readonly #isContentScript: boolean;
-  readonly #isLiveEdit: boolean;
   sourceMapURL?: string;
   debugSymbols: Protocol.Debugger.DebugSymbols|null;
   hasSourceURL: boolean;
@@ -92,13 +90,13 @@ export class Script implements TextUtils.ContentProvider.ContentProvider, FrameA
   readonly #embedderName: Platform.DevToolsPath.UrlString|null;
   readonly isModule: boolean|null;
   readonly buildId: string|null;
-  constructor(
-      debuggerModel: DebuggerModel, scriptId: Protocol.Runtime.ScriptId, sourceURL: Platform.DevToolsPath.UrlString,
-      startLine: number, startColumn: number, endLine: number, endColumn: number, executionContextId: number,
-      hash: string, isContentScript: boolean, isLiveEdit: boolean, sourceMapURL: string|undefined,
-      hasSourceURL: 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) {
+  constructor(debuggerModel: DebuggerModel, scriptId: Protocol.Runtime.ScriptId,
+              sourceURL: Platform.DevToolsPath.UrlString, startLine: number, startColumn: number, endLine: number,
+              endColumn: number, executionContextId: number, hash: string, isContentScript: boolean,
+              sourceMapURL: string|undefined, hasSourceURL: 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) {
     this.debuggerModel = debuggerModel;
     this.scriptId = scriptId;
     this.sourceURL = sourceURL;
@@ -112,7 +110,6 @@ export class Script implements TextUtils.ContentProvider.ContentProvider, FrameA
     this.executionContextId = executionContextId;
     this.hash = hash;
     this.#isContentScript = isContentScript;
-    this.#isLiveEdit = isLiveEdit;
     this.sourceMapURL = sourceMapURL;
     this.debugSymbols = debugSymbols;
     this.hasSourceURL = hasSourceURL;
@@ -175,10 +172,6 @@ export class Script implements TextUtils.ContentProvider.ContentProvider, FrameA
     return this.debuggerModel.runtimeModel().executionContext(this.executionContextId);
   }
 
-  isLiveEdit(): boolean {
-    return this.#isLiveEdit;
-  }
-
   contentURL(): Platform.DevToolsPath.UrlString {
     return this.sourceURL;
   }
@@ -261,8 +254,8 @@ export class Script implements TextUtils.ContentProvider.ContentProvider, FrameA
   requestContentData(): Promise<TextUtils.ContentData.ContentDataOrError> {
     if (!this.#contentPromise) {
       const fileSizeToCache = 65535;  // We won't bother cacheing files under 64K
-      if (this.hash && !this.#isLiveEdit && this.contentLength > fileSizeToCache) {
-        // For large files that aren't live edits and have a hash, we keep a content-addressed cache
+      if (this.hash && this.contentLength > fileSizeToCache) {
+        // For large files that have a hash, we keep a content-addressed cache
         // so we don't need to load multiple copies or disassemble wasm modules multiple times.
         if (!scriptCacheInstance) {
           // Initialize script cache singleton. Add a finalizer for removing keys from the map.
@@ -331,43 +324,6 @@ export class Script implements TextUtils.ContentProvider.ContentProvider, FrameA
     return TextUtils.TextUtils.performSearchInSearchMatches(matches.result || [], query, caseSensitive, isRegex);
   }
 
-  private appendSourceURLCommentIfNeeded(source: string): string {
-    if (!this.hasSourceURL) {
-      return source;
-    }
-    return source + '\n //# sourceURL=' + this.sourceURL;
-  }
-
-  async editSource(newSource: string): Promise<{
-    changed: boolean,
-    status: Protocol.Debugger.SetScriptSourceResponseStatus,
-    exceptionDetails?: Protocol.Runtime.ExceptionDetails,
-  }> {
-    newSource = Script.trimSourceURLComment(newSource);
-    // We append correct #sourceURL to script for consistency only. It's not actually needed for things to work correctly.
-    newSource = this.appendSourceURLCommentIfNeeded(newSource);
-
-    const oldSource = TextUtils.ContentData.ContentData.textOr(await this.requestContentData(), null);
-    if (oldSource === newSource) {
-      return {changed: false, status: Protocol.Debugger.SetScriptSourceResponseStatus.Ok};
-    }
-    const response = await this.debuggerModel.target().debuggerAgent().invoke_setScriptSource(
-        {scriptId: this.scriptId, scriptSource: newSource, allowTopFrameEditing: true});
-    if (response.getError()) {
-      // Something went seriously wrong, like the V8 inspector no longer knowing about this script without
-      // shutting down the Debugger agent etc.
-      throw new Error(`Script#editSource failed for script with id ${this.scriptId}: ${response.getError()}`);
-    }
-
-    if (!response.getError() && response.status === Protocol.Debugger.SetScriptSourceResponseStatus.Ok) {
-      this.#contentPromise =
-          Promise.resolve(new TextUtils.ContentData.ContentData(newSource, /* isBase64 */ false, 'text/javascript'));
-    }
-
-    this.debuggerModel.dispatchEventToListeners(Events.ScriptSourceWasEdited, {script: this, status: response.status});
-    return {changed: true, status: response.status, exceptionDetails: response.exceptionDetails};
-  }
-
   rawLocation(lineNumber: number, columnNumber: number): Location|null {
     if (this.containsLocation(lineNumber, columnNumber)) {
       return new Location(this.debuggerModel, this.scriptId, lineNumber, columnNumber);