chrome-devtools-frontend 1.0.1015276 → 1.0.1015822

package/config/gni/devtools_grd_files.gni

@@ -756,6 +756,7 @@ grd_files_debug_sources = [
   "front_end/models/issues_manager/SharedArrayBufferIssue.js",
   "front_end/models/issues_manager/SourceFrameIssuesManager.js",
   "front_end/models/issues_manager/TrustedWebActivityIssue.js",
+  "front_end/models/javascript_metadata/DOMPinnedProperties.js",
   "front_end/models/javascript_metadata/JavaScriptMetadata.js",
   "front_end/models/javascript_metadata/NativeFunctions.js",
   "front_end/models/logs/LogManager.js",

package/front_end/core/common/JavaScriptMetaData.ts

@@ -2,6 +2,26 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+export interface DOMPinnedWebIDLProp {
+  global?: boolean;
+  specs?: number;
+  rules?: Array<DOMPinnedWebIDLRule>;
+}
+
+export interface DOMPinnedWebIDLType {
+  inheritance?: string;
+  includes?: Array<string>;
+  props?: {
+    [PropName: string]: DOMPinnedWebIDLProp,
+  };
+  rules?: Array<DOMPinnedWebIDLRule>;
+}
+
+export interface DOMPinnedWebIDLRule {
+  when: string;
+  is: string;
+}
+
 export interface JavaScriptMetaData {
   signaturesForNativeFunction(name: string): string[][]|null;
 

package/front_end/core/i18n/locales/en-US.json

@@ -9890,9 +9890,6 @@
   "panels/sources/ScopeChainSidebarPane.ts | closureS": {
     "message": "Closure ({PH1})"
   },
-  "panels/sources/ScopeChainSidebarPane.ts | couldNotOpenLinearMemory": {
-    "message": "Could not open linear memory inspector: failed locating buffer."
-  },
   "panels/sources/ScopeChainSidebarPane.ts | exception": {
     "message": "Exception"
   },
@@ -11879,6 +11876,9 @@
   "ui/components/linear_memory_inspector/LinearMemoryInspector.ts | addressHasToBeANumberBetweenSAnd": {
     "message": "Address has to be a number between {PH1} and {PH2}"
   },
+  "ui/components/linear_memory_inspector/LinearMemoryInspectorController.ts | couldNotOpenLinearMemory": {
+    "message": "Could not open linear memory inspector: failed locating buffer."
+  },
   "ui/components/linear_memory_inspector/LinearMemoryInspectorPane.ts | noOpenInspections": {
     "message": "No open inspections"
   },

package/front_end/core/i18n/locales/en-XL.json

@@ -9890,9 +9890,6 @@
   "panels/sources/ScopeChainSidebarPane.ts | closureS": {
     "message": "Ĉĺôśûŕê ({PH1})"
   },
-  "panels/sources/ScopeChainSidebarPane.ts | couldNotOpenLinearMemory": {
-    "message": "Ĉóûĺd̂ ńôt́ ôṕêń l̂ín̂éâŕ m̂ém̂ór̂ý îńŝṕêćt̂ór̂: f́âíl̂éd̂ ĺôćât́îńĝ b́ûf́f̂ér̂."
-  },
   "panels/sources/ScopeChainSidebarPane.ts | exception": {
     "message": "Êx́ĉép̂t́îón̂"
   },
@@ -11879,6 +11876,9 @@
   "ui/components/linear_memory_inspector/LinearMemoryInspector.ts | addressHasToBeANumberBetweenSAnd": {
     "message": "Âd́d̂ŕêśŝ h́âś t̂ó b̂é â ńûḿb̂ér̂ b́êt́ŵéêń {PH1} âńd̂ {PH2}"
   },
+  "ui/components/linear_memory_inspector/LinearMemoryInspectorController.ts | couldNotOpenLinearMemory": {
+    "message": "Ĉóûĺd̂ ńôt́ ôṕêń l̂ín̂éâŕ m̂ém̂ór̂ý îńŝṕêćt̂ór̂: f́âíl̂éd̂ ĺôćât́îńĝ b́ûf́f̂ér̂."
+  },
   "ui/components/linear_memory_inspector/LinearMemoryInspectorPane.ts | noOpenInspections": {
     "message": "N̂ó ôṕêń îńŝṕêćt̂íôńŝ"
   },

package/front_end/core/sdk/DOMModel.ts

@@ -64,6 +64,7 @@ export class DOMNode {
   #localNameInternal!: string;
   nodeValueInternal!: string;
   #pseudoTypeInternal!: Protocol.DOM.PseudoType|undefined;
+  #pseudoIdentifier?: string;
   #shadowRootTypeInternal!: Protocol.DOM.ShadowRootType|undefined;
   #frameOwnerFrameIdInternal!: Protocol.Page.FrameId|null;
   #xmlVersion!: string|undefined;
@@ -135,6 +136,7 @@ export class DOMNode {
     this.#localNameInternal = payload.localName;
     this.nodeValueInternal = payload.nodeValue;
     this.#pseudoTypeInternal = payload.pseudoType;
+    this.#pseudoIdentifier = payload.pseudoIdentifier;
     this.#shadowRootTypeInternal = payload.shadowRootType;
     this.#frameOwnerFrameIdInternal = payload.frameId || null;
     this.#xmlVersion = payload.xmlVersion;
@@ -318,6 +320,10 @@ export class DOMNode {
     return this.#pseudoTypeInternal;
   }
 
+  pseudoIdentifier(): string|undefined {
+    return this.#pseudoIdentifier;
+  }
+
   hasPseudoElements(): boolean {
     return this.#pseudoElements.size > 0;
   }
@@ -1644,6 +1650,9 @@ class DOMDispatcher implements ProtocolProxyApi.DOMDispatcher {
   distributedNodesUpdated({insertionPointId, distributedNodes}: Protocol.DOM.DistributedNodesUpdatedEvent): void {
     this.#domModel.distributedNodesUpdated(insertionPointId, distributedNodes);
   }
+
+  topLayerElementsUpdated(): void {
+  }
 }
 
 // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration

package/front_end/core/sdk/DebuggerModel.ts

@@ -580,34 +580,6 @@ export class DebuggerModel extends SDKModel<EventTypes> {
     return result;
   }
 
-  setScriptSource(
-      scriptId: string, newSource: string,
-      callback: (error: string|null, arg1?: Protocol.Runtime.ExceptionDetails|undefined) => void): void {
-    const script = this.#scriptsInternal.get(scriptId);
-    if (script) {
-      void script.editSource(newSource, this.didEditScriptSource.bind(this, scriptId, newSource, callback));
-    }
-  }
-
-  private didEditScriptSource(
-      scriptId: string, newSource: string,
-      callback: (error: string|null, arg1?: Protocol.Runtime.ExceptionDetails|undefined) => void, error: string|null,
-      exceptionDetails?: Protocol.Runtime.ExceptionDetails, callFrames?: Protocol.Debugger.CallFrame[],
-      asyncStackTrace?: Protocol.Runtime.StackTrace, asyncStackTraceId?: Protocol.Runtime.StackTraceId,
-      needsStepIn?: boolean): void {
-    callback(error, exceptionDetails);
-    if (needsStepIn) {
-      void this.stepInto();
-      return;
-    }
-
-    if (!error && callFrames && callFrames.length && this.#debuggerPausedDetailsInternal) {
-      void this.pausedScript(
-          callFrames, this.#debuggerPausedDetailsInternal.reason, this.#debuggerPausedDetailsInternal.auxData,
-          this.#debuggerPausedDetailsInternal.breakpointIds, asyncStackTrace, asyncStackTraceId);
-    }
-  }
-
   get callFrames(): CallFrame[]|null {
     return this.#debuggerPausedDetailsInternal ? this.#debuggerPausedDetailsInternal.callFrames : null;
   }

package/front_end/core/sdk/RemoteObject.ts

@@ -34,6 +34,7 @@
 
 import type * as ProtocolProxyApi from '../../generated/protocol-proxy-api.js';
 import * as Protocol from '../../generated/protocol.js';
+import type {DOMPinnedWebIDLProp, DOMPinnedWebIDLType} from '../common/JavaScriptMetaData.js';
 
 import type {DebuggerModel, FunctionDetails} from './DebuggerModel.js';
 import type {RuntimeModel} from './RuntimeModel.js';
@@ -309,6 +310,8 @@ export class RemoteObject {
   isNode(): boolean {
     return false;
   }
+
+  webIdl?: RemoteObjectWebIdlTypeMetadata;
 }
 
 export class RemoteObjectImpl extends RemoteObject {
@@ -728,6 +731,8 @@ export class RemoteObjectProperty {
   getter: RemoteObject|undefined;
   setter: RemoteObject|undefined;
 
+  webIdl?: RemoteObjectWebIdlPropertyMetadata;
+
   constructor(
       name: string, value: RemoteObject|null, enumerable?: boolean, writable?: boolean, isOwn?: boolean,
       wasThrown?: boolean, symbol?: RemoteObject|null, synthetic?: boolean,
@@ -1188,3 +1193,13 @@ export interface GetPropertiesResult {
   properties: RemoteObjectProperty[]|null;
   internalProperties: RemoteObjectProperty[]|null;
 }
+
+export interface RemoteObjectWebIdlTypeMetadata {
+  info: DOMPinnedWebIDLType;
+  state: Map<string, string>;
+}
+
+export interface RemoteObjectWebIdlPropertyMetadata {
+  info: DOMPinnedWebIDLProp;
+  applicable?: boolean;
+}

package/front_end/core/sdk/Script.ts

@@ -257,25 +257,19 @@ export class Script implements TextUtils.ContentProvider.ContentProvider, FrameA
     return source + '\n //# sourceURL=' + this.sourceURL;
   }
 
-  async editSource(
-      newSource: string,
-      callback:
-          (error: string|null, arg1?: Protocol.Runtime.ExceptionDetails|undefined,
-           arg2?: Array<Protocol.Debugger.CallFrame>|undefined, arg3?: Protocol.Runtime.StackTrace|undefined,
-           arg4?: Protocol.Runtime.StackTraceId|undefined, arg5?: boolean|undefined) => void): Promise<void> {
+  async editSource(newSource: string):
+      Promise<{error: string | null, 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);
 
     if (!this.scriptId) {
-      callback('Script failed to parse');
-      return;
+      return {error: 'Script failed to parse'};
     }
 
     const {content: oldSource} = await this.requestContent();
     if (oldSource === newSource) {
-      callback(null);
-      return;
+      return {error: null};
     }
     const response = await this.debuggerModel.target().debuggerAgent().invoke_setScriptSource(
         {scriptId: this.scriptId, scriptSource: newSource});
@@ -284,10 +278,7 @@ export class Script implements TextUtils.ContentProvider.ContentProvider, FrameA
       this.#contentPromise = Promise.resolve({content: newSource, isEncoded: false});
     }
 
-    const needsStepIn = Boolean(response.stackChanged);
-    callback(
-        response.getError() || null, response.exceptionDetails, response.callFrames, response.asyncStackTrace,
-        response.asyncStackTraceId, needsStepIn);
+    return {error: response.getError() || null, exceptionDetails: response.exceptionDetails};
   }
 
   rawLocation(lineNumber: number, columnNumber: number): Location|null {

package/front_end/generated/InspectorBackendCommands.js

@@ -732,6 +732,7 @@ export function registerCommands(inspectorBackend) {
   inspectorBackend.registerEvent('DOM.documentUpdated', []);
   inspectorBackend.registerEvent('DOM.inlineStyleInvalidated', ['nodeIds']);
   inspectorBackend.registerEvent('DOM.pseudoElementAdded', ['parentId', 'pseudoElement']);
+  inspectorBackend.registerEvent('DOM.topLayerElementsUpdated', []);
   inspectorBackend.registerEvent('DOM.pseudoElementRemoved', ['parentId', 'pseudoElementId']);
   inspectorBackend.registerEvent('DOM.setChildNodes', ['parentId', 'nodes']);
   inspectorBackend.registerEvent('DOM.shadowRootPopped', ['hostId', 'rootId']);
@@ -875,6 +876,7 @@ export function registerCommands(inspectorBackend) {
         {'name': 'selector', 'type': 'string', 'optional': false}
       ],
       ['nodeIds']);
+  inspectorBackend.registerCommand('DOM.getTopLayerElements', [], ['nodeIds']);
   inspectorBackend.registerCommand('DOM.redo', [], []);
   inspectorBackend.registerCommand(
       'DOM.removeAttribute',
@@ -2114,6 +2116,7 @@ export function registerCommands(inspectorBackend) {
     EncryptedMedia: 'encrypted-media',
     ExecutionWhileOutOfViewport: 'execution-while-out-of-viewport',
     ExecutionWhileNotRendered: 'execution-while-not-rendered',
+    FederatedCredentials: 'federated-credentials',
     FocusWithoutUserActivation: 'focus-without-user-activation',
     Fullscreen: 'fullscreen',
     Frobulate: 'frobulate',
@@ -2378,7 +2381,7 @@ export function registerCommands(inspectorBackend) {
   inspectorBackend.registerEvent('Page.domContentEventFired', ['timestamp']);
   inspectorBackend.registerEnum(
       'Page.FileChooserOpenedEventMode', {SelectSingle: 'selectSingle', SelectMultiple: 'selectMultiple'});
-  inspectorBackend.registerEvent('Page.fileChooserOpened', ['frameId', 'backendNodeId', 'mode']);
+  inspectorBackend.registerEvent('Page.fileChooserOpened', ['frameId', 'mode', 'backendNodeId']);
   inspectorBackend.registerEvent('Page.frameAttached', ['frameId', 'parentFrameId', 'stack', 'adScriptId']);
   inspectorBackend.registerEvent('Page.frameClearedScheduledNavigation', ['frameId']);
   inspectorBackend.registerEnum('Page.FrameDetachedEventReason', {Remove: 'remove', Swap: 'swap'});
@@ -3297,9 +3300,10 @@ export function registerCommands(inspectorBackend) {
       [
         {'name': 'scriptId', 'type': 'string', 'optional': false},
         {'name': 'scriptSource', 'type': 'string', 'optional': false},
-        {'name': 'dryRun', 'type': 'boolean', 'optional': true}
+        {'name': 'dryRun', 'type': 'boolean', 'optional': true},
+        {'name': 'allowTopFrameEditing', 'type': 'boolean', 'optional': true}
       ],
-      ['callFrames', 'stackChanged', 'asyncStackTrace', 'asyncStackTraceId', 'exceptionDetails']);
+      ['callFrames', 'stackChanged', 'asyncStackTrace', 'asyncStackTraceId', 'status', 'exceptionDetails']);
   inspectorBackend.registerCommand(
       'Debugger.setSkipAllPauses', [{'name': 'skip', 'type': 'boolean', 'optional': false}], []);
   inspectorBackend.registerCommand(

package/front_end/generated/protocol-mapping.d.ts

@@ -124,6 +124,10 @@ export namespace ProtocolMapping {
      * Called when a pseudo element is added to an element.
      */
     'DOM.pseudoElementAdded': [Protocol.DOM.PseudoElementAddedEvent];
+    /**
+     * Called when top layer elements are changed.
+     */
+    'DOM.topLayerElementsUpdated': [];
     /**
      * Called when a pseudo element is removed from an element.
      */
@@ -1266,6 +1270,12 @@ export namespace ProtocolMapping {
      */
     'DOM.querySelectorAll':
         {paramsType: [Protocol.DOM.QuerySelectorAllRequest]; returnType: Protocol.DOM.QuerySelectorAllResponse;};
+    /**
+     * Returns NodeIds of current top layer elements.
+     * Top layer is rendered closest to the user within a viewport, therefore its elements always
+     * appear on top of all other content.
+     */
+    'DOM.getTopLayerElements': {paramsType: []; returnType: Protocol.DOM.GetTopLayerElementsResponse;};
     /**
      * Re-does the last undone action.
      */
@@ -2961,6 +2971,12 @@ export namespace ProtocolMapping {
     'Debugger.setReturnValue': {paramsType: [Protocol.Debugger.SetReturnValueRequest]; returnType: void;};
     /**
      * Edits JavaScript source live.
+     *
+     * In general, functions that are currently on the stack can not be edited with
+     * a single exception: If the edited function is the top-most stack frame and
+     * that is the only activation of that function on the stack. In this case
+     * the live edit will be successful and a `Debugger.restartFrame` for the
+     * top-most function is automatically triggered.
      */
     'Debugger.setScriptSource': {
       paramsType: [Protocol.Debugger.SetScriptSourceRequest]; returnType: Protocol.Debugger.SetScriptSourceResponse;

package/front_end/generated/protocol-proxy-api.d.ts

@@ -959,6 +959,13 @@ declare namespace ProtocolProxyApi {
     invoke_querySelectorAll(params: Protocol.DOM.QuerySelectorAllRequest):
         Promise<Protocol.DOM.QuerySelectorAllResponse>;
 
+    /**
+     * Returns NodeIds of current top layer elements.
+     * Top layer is rendered closest to the user within a viewport, therefore its elements always
+     * appear on top of all other content.
+     */
+    invoke_getTopLayerElements(): Promise<Protocol.DOM.GetTopLayerElementsResponse>;
+
     /**
      * Re-does the last undone action.
      */
@@ -1128,6 +1135,11 @@ declare namespace ProtocolProxyApi {
      */
     pseudoElementAdded(params: Protocol.DOM.PseudoElementAddedEvent): void;
 
+    /**
+     * Called when top layer elements are changed.
+     */
+    topLayerElementsUpdated(): void;
+
     /**
      * Called when a pseudo element is removed from an element.
      */
@@ -3785,6 +3797,12 @@ declare namespace ProtocolProxyApi {
 
     /**
      * Edits JavaScript source live.
+     *
+     * In general, functions that are currently on the stack can not be edited with
+     * a single exception: If the edited function is the top-most stack frame and
+     * that is the only activation of that function on the stack. In this case
+     * the live edit will be successful and a `Debugger.restartFrame` for the
+     * top-most function is automatically triggered.
      */
     invoke_setScriptSource(params: Protocol.Debugger.SetScriptSourceRequest):
         Promise<Protocol.Debugger.SetScriptSourceResponse>;

package/front_end/generated/protocol.ts

@@ -3783,6 +3783,13 @@ export namespace DOM {
     nodeIds: NodeId[];
   }
 
+  export interface GetTopLayerElementsResponse extends ProtocolResponseWithError {
+    /**
+     * NodeIds of top layer elements
+     */
+    nodeIds: NodeId[];
+  }
+
   export interface RemoveAttributeRequest {
     /**
      * Id of the element to remove attribute from.
@@ -10243,6 +10250,7 @@ export namespace Page {
     EncryptedMedia = 'encrypted-media',
     ExecutionWhileOutOfViewport = 'execution-while-out-of-viewport',
     ExecutionWhileNotRendered = 'execution-while-not-rendered',
+    FederatedCredentials = 'federated-credentials',
     FocusWithoutUserActivation = 'focus-without-user-activation',
     Fullscreen = 'fullscreen',
     Frobulate = 'frobulate',
@@ -11772,14 +11780,14 @@ export namespace Page {
      * Id of the frame containing input node.
      */
     frameId: FrameId;
-    /**
-     * Input node id.
-     */
-    backendNodeId: DOM.BackendNodeId;
     /**
      * Input mode.
      */
     mode: FileChooserOpenedEventMode;
+    /**
+     * Input node id. Only present for file choosers opened via an <input type="file"> element.
+     */
+    backendNodeId?: DOM.BackendNodeId;
   }
 
   /**
@@ -15298,6 +15306,13 @@ export namespace Debugger {
     newValue: Runtime.CallArgument;
   }
 
+  export const enum SetScriptSourceResponseStatus {
+    Ok = 'Ok',
+    CompileError = 'CompileError',
+    BlockedByActiveGenerator = 'BlockedByActiveGenerator',
+    BlockedByActiveFunction = 'BlockedByActiveFunction',
+  }
+
   export interface SetScriptSourceRequest {
     /**
      * Id of the script to edit.
@@ -15312,6 +15327,11 @@ export namespace Debugger {
      * description without actually modifying the code.
      */
     dryRun?: boolean;
+    /**
+     * If true, then `scriptSource` is allowed to change the function on top of the stack
+     * as long as the top-most stack frame is the only activation of that function.
+     */
+    allowTopFrameEditing?: boolean;
   }
 
   export interface SetScriptSourceResponse extends ProtocolResponseWithError {
@@ -15332,7 +15352,13 @@ export namespace Debugger {
      */
     asyncStackTraceId?: Runtime.StackTraceId;
     /**
-     * Exception details if any.
+     * Whether the operation was successful or not. Only `Ok` denotes a
+     * successful live edit while the other enum variants denote why
+     * the live edit failed.
+     */
+    status: SetScriptSourceResponseStatus;
+    /**
+     * Exception details if any. Only present when `status` is `CompileError`.
      */
     exceptionDetails?: Runtime.ExceptionDetails;
   }

package/front_end/models/bindings/ResourceScriptMapping.ts

@@ -316,12 +316,11 @@ export class ResourceScriptFile extends Common.ObjectWrapper.ObjectWrapper<Resou
     if (!this.scriptInternal) {
       return;
     }
-    const debuggerModel = this.#resourceScriptMapping.debuggerModel;
     const breakpoints = BreakpointManager.instance()
                             .breakpointLocationsForUISourceCode(this.#uiSourceCodeInternal)
                             .map(breakpointLocation => breakpointLocation.breakpoint);
     const source = this.#uiSourceCodeInternal.workingCopy();
-    debuggerModel.setScriptSource(this.scriptInternal.scriptId, source, (error, exceptionDetails) => {
+    void this.scriptInternal.editSource(source).then(({error, exceptionDetails}) => {
       void this.scriptSourceWasSet(source, breakpoints, error, exceptionDetails);
     });
   }

package/front_end/models/emulation/EmulatedDevices.ts

@@ -1554,19 +1554,6 @@ const emulatedDevices = [
         'Mozilla/5.0 (Linux; U; en-us; KFAPWI Build/JDQ39) AppleWebKit/535.19 (KHTML, like Gecko) Silk/3.13 Safari/535.19 Silk-Accelerated=true',
     'type': 'tablet',
   },
-  {
-    'show-by-default': false,
-    'title': 'iPad Mini',
-    'screen': {
-      'horizontal': {'width': 1024, 'height': 768},
-      'device-pixel-ratio': 2,
-      'vertical': {'width': 768, 'height': 1024},
-    },
-    'capabilities': ['touch', 'mobile'],
-    'user-agent':
-        'Mozilla/5.0 (iPad; CPU OS 11_0 like Mac OS X) AppleWebKit/604.1.34 (KHTML, like Gecko) Version/11.0 Mobile/15A5341f Safari/604.1',
-    'type': 'tablet',
-  },
   {
     'order': 140,
     'show-by-default': false,