chrome-devtools-frontend 1.0.943017 → 1.0.944427

package/front_end/entrypoints/node_app/node_app.js

@@ -5,7 +5,7 @@ import '../shell/shell.js';
 import '../../panels/js_profiler/js_profiler-meta.js';
 import '../node_main/node_main-meta.js';
 import './node_app-meta.js';
-import * as Startup from '../startup/startup.js';  // eslint-disable-line rulesdir/es_modules_import
+import * as Startup from '../startup/startup.js';
 
 // Side-effect start the `node_main` module, which implements runnables in
 // the NodeMain class

package/front_end/entrypoints/node_main/node_main-meta.ts

@@ -5,7 +5,6 @@
 import * as i18n from '../../core/i18n/i18n.js';
 import * as UI from '../../ui/legacy/legacy.js';
 
-// eslint-disable-next-line rulesdir/es_modules_import
 import type * as NodeMain from './node_main.js';
 
 const UIStrings = {

package/front_end/entrypoints/shell/shell.json

@@ -1,6 +1,5 @@
 {
   "modules": [
-    { "name": "ui/legacy/components/text_editor", "type": "autostart" },
-    { "name": "ui/legacy/components/source_frame", "type": "autostart" }
+    { "name": "ui/legacy/components/text_editor", "type": "autostart" }
   ]
 }

package/front_end/entrypoints/wasmparser_worker/wasmparser_worker-entrypoint.ts

@@ -2,7 +2,7 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-import * as WasmParserWorker from './wasmparser_worker.js';  // eslint-disable-line rulesdir/es_modules_import
+import * as WasmParserWorker from './wasmparser_worker.js';
 
 self.onmessage = (event: {data: {method: string, params: {content: string}}}): void => {
   const method = event.data.method;

package/front_end/entrypoints/worker_app/worker_app.js

@@ -13,6 +13,6 @@ import '../../panels/network/network-meta.js';
 import '../../panels/application/application-meta.js';
 import '../../panels/timeline/timeline-meta.js';
 import './WorkerMain.js';
-import * as Startup from '../startup/startup.js';  // eslint-disable-line rulesdir/es_modules_import
+import * as Startup from '../startup/startup.js';
 
 Startup.RuntimeInstantiator.startApplication('worker_app');

package/front_end/generated/InspectorBackendCommands.js

@@ -1700,6 +1700,7 @@ export function registerCommands(inspectorBackend) {
       ['innerRequestId', 'innerRequestURL', 'errorMessage', 'bundleRequestId']);
   inspectorBackend.registerEvent('Network.reportingApiReportAdded', ['report']);
   inspectorBackend.registerEvent('Network.reportingApiReportUpdated', ['report']);
+  inspectorBackend.registerEvent('Network.reportingApiEndpointsChangedForOrigin', ['origin', 'endpoints']);
   inspectorBackend.registerCommand(
       'Network.setAcceptedEncodings', [{'name': 'encodings', 'type': 'object', 'optional': false}], []);
   inspectorBackend.registerCommand('Network.clearAcceptedEncodingsOverride', [], []);
@@ -1791,7 +1792,8 @@ export function registerCommands(inspectorBackend) {
         {'name': 'priority', 'type': 'string', 'optional': true},
         {'name': 'sameParty', 'type': 'boolean', 'optional': true},
         {'name': 'sourceScheme', 'type': 'string', 'optional': true},
-        {'name': 'sourcePort', 'type': 'number', 'optional': true}
+        {'name': 'sourcePort', 'type': 'number', 'optional': true},
+        {'name': 'partitionKey', 'type': 'string', 'optional': true}
       ],
       ['success']);
   inspectorBackend.registerCommand(

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

@@ -305,6 +305,7 @@ export namespace ProtocolMapping {
      */
     'Network.reportingApiReportAdded': [Protocol.Network.ReportingApiReportAddedEvent];
     'Network.reportingApiReportUpdated': [Protocol.Network.ReportingApiReportUpdatedEvent];
+    'Network.reportingApiEndpointsChangedForOrigin': [Protocol.Network.ReportingApiEndpointsChangedForOriginEvent];
     /**
      * Fired when the node should be inspected. This happens after call to `setInspectMode` or when
      * user manually inspects an element.

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

@@ -2182,6 +2182,8 @@ declare namespace ProtocolProxyApi {
     reportingApiReportAdded(params: Protocol.Network.ReportingApiReportAddedEvent): void;
 
     reportingApiReportUpdated(params: Protocol.Network.ReportingApiReportUpdatedEvent): void;
+
+    reportingApiEndpointsChangedForOrigin(params: Protocol.Network.ReportingApiEndpointsChangedForOriginEvent): void;
   }
 
   export interface OverlayApi {

package/front_end/generated/protocol.d.ts

@@ -7513,6 +7513,15 @@ declare namespace Protocol {
        * This is a temporary ability and it will be removed in the future.
        */
       sourcePort: integer;
+      /**
+       * Cookie partition key. The site of the top-level URL the browser was visiting at the start
+       * of the request to the endpoint that set the cookie.
+       */
+      partitionKey?: string;
+      /**
+       * True if cookie partition key is opaque.
+       */
+      partitionKeyOpaque?: boolean;
     }
 
     /**
@@ -7653,6 +7662,12 @@ declare namespace Protocol {
        * This is a temporary ability and it will be removed in the future.
        */
       sourcePort?: integer;
+      /**
+       * Cookie partition key. The site of the top-level URL the browser was visiting at the start
+       * of the request to the endpoint that set the cookie.
+       * If not set, the cookie will be set as not partitioned.
+       */
+      partitionKey?: string;
     }
 
     export const enum AuthChallengeSource {
@@ -7976,6 +7991,17 @@ declare namespace Protocol {
       status: ReportStatus;
     }
 
+    export interface ReportingApiEndpoint {
+      /**
+       * The URL of the endpoint to which reports may be delivered.
+       */
+      url: string;
+      /**
+       * Name of the endpoint group.
+       */
+      groupName: string;
+    }
+
     /**
      * An object providing the result of a network resource load.
      */
@@ -8332,6 +8358,12 @@ declare namespace Protocol {
        * This is a temporary ability and it will be removed in the future.
        */
       sourcePort?: integer;
+      /**
+       * Cookie partition key. The site of the top-level URL the browser was visiting at the start
+       * of the request to the endpoint that set the cookie.
+       * If not set, the cookie will be set as not partitioned.
+       */
+      partitionKey?: string;
     }
 
     export interface SetCookieResponse extends ProtocolResponseWithError {
@@ -9098,6 +9130,14 @@ declare namespace Protocol {
     export interface ReportingApiReportUpdatedEvent {
       report: ReportingApiReport;
     }
+
+    export interface ReportingApiEndpointsChangedForOriginEvent {
+      /**
+       * Origin of the document(s) which configured the endpoints.
+       */
+      origin: string;
+      endpoints: ReportingApiEndpoint[];
+    }
   }
 
   /**

package/front_end/legacy_test_runner/console_test_runner/console_test_runner.js

@@ -131,7 +131,11 @@ ConsoleTestRunner.dumpConsoleTableMessage = function(viewMessage, forceInvalidat
   if (forceInvalidate) {
     Console.ConsoleView.instance().viewport.invalidate();
   }
-  const table = viewMessage.element();
+  const formattedTable = viewMessage.element().querySelector('.console-message-formatted-table');
+  if (!formattedTable) {
+    return false;
+  }
+  const table = formattedTable.querySelector('span').shadowRoot;
   const headers = table.querySelectorAll('th > div:first-child');
   if (!headers.length) {
     return false;

package/front_end/models/bindings/DefaultScriptMapping.ts

@@ -75,11 +75,7 @@ export class DefaultScriptMapping implements DebuggerSourceMapping {
     if (!uiSourceCode) {
       return null;
     }
-    const lineNumber = rawLocation.lineNumber - (script.isInlineScriptWithSourceURL() ? script.lineOffset : 0);
-    let columnNumber = rawLocation.columnNumber || 0;
-    if (script.isInlineScriptWithSourceURL() && !lineNumber && columnNumber) {
-      columnNumber -= script.columnOffset;
-    }
+    const {lineNumber, columnNumber = 0} = rawLocation;
     return uiSourceCode.uiLocation(lineNumber, columnNumber);
   }
 
@@ -89,10 +85,6 @@ export class DefaultScriptMapping implements DebuggerSourceMapping {
     if (!script) {
       return [];
     }
-    if (script.isInlineScriptWithSourceURL()) {
-      return [this.#debuggerModel.createRawLocation(
-          script, lineNumber + script.lineOffset, lineNumber ? columnNumber : columnNumber + script.columnOffset)];
-    }
     return [this.#debuggerModel.createRawLocation(script, lineNumber, columnNumber)];
   }
 

package/front_end/models/bindings/ResourceScriptMapping.ts

@@ -127,30 +127,22 @@ export class ResourceScriptMapping implements DebuggerSourceMapping {
     if (!scriptFile.hasScripts([script])) {
       return null;
     }
-    const lineNumber = rawLocation.lineNumber - (script.isInlineScriptWithSourceURL() ? script.lineOffset : 0);
-    let columnNumber = rawLocation.columnNumber || 0;
-    if (script.isInlineScriptWithSourceURL() && !lineNumber && columnNumber) {
-      columnNumber -= script.columnOffset;
-    }
+    const {lineNumber, columnNumber = 0} = rawLocation;
     return uiSourceCode.uiLocation(lineNumber, columnNumber);
   }
 
   uiLocationToRawLocations(uiSourceCode: Workspace.UISourceCode.UISourceCode, lineNumber: number, columnNumber: number):
       SDK.DebuggerModel.Location[] {
     const scriptFile = this.#uiSourceCodeToScriptFile.get(uiSourceCode);
-    if (!scriptFile || typeof scriptFile.script === 'undefined') {
+    if (!scriptFile) {
       return [];
     }
 
-    const script = scriptFile.script;
+    const {script} = scriptFile;
     if (!script) {
       return [];
     }
 
-    if (script.isInlineScriptWithSourceURL()) {
-      return [this.debuggerModel.createRawLocation(
-          script, lineNumber + script.lineOffset, lineNumber ? columnNumber : columnNumber + script.columnOffset)];
-    }
     return [this.debuggerModel.createRawLocation(script, lineNumber, columnNumber)];
   }
 

package/front_end/models/formatter/SourceFormatter.ts

@@ -165,21 +165,6 @@ class ScriptMapping implements Bindings.DebuggerWorkspaceBinding.DebuggerSourceM
     if (!formatData || !script) {
       return null;
     }
-    if (script.isInlineScriptWithSourceURL()) {
-      // Inline scripts with #sourceURL= have lineEndings wrt. the inline script (and not wrt. the containing document),
-      // but `rawLocation` will always use locations wrt. the containing document, because that is what the back-end is
-      // sending. This is a hack, because what we are really doing here is deciding the location based on /how/ the
-      // script is displayed, which is really something this layer cannot and should not have to decide: The
-      // SourceFormatter should not have to know whether a script is displayed inline (in its containing document) or
-      // stand-alone.
-      const [relativeLineNumber, relativeColumnNumber] = script.toRelativeLocation(rawLocation);
-      const [formattedLineNumber, formattedColumnNumber] =
-          formatData.mapping.originalToFormatted(relativeLineNumber, relativeColumnNumber);
-      return formatData.formattedSourceCode.uiLocation(formattedLineNumber, formattedColumnNumber);
-    }
-    // Here we either have an inline script without a #sourceURL= or a stand-alone script. For stand-alone scripts, no
-    // translation must be applied. For inline scripts, also no translation must be applied, because the line-endings
-    // tables in the mapping are the same as in the containing document.
     const [lineNumber, columnNumber] =
         formatData.mapping.originalToFormatted(rawLocation.lineNumber, rawLocation.columnNumber || 0);
     return formatData.formattedSourceCode.uiLocation(lineNumber, columnNumber);

package/front_end/models/persistence/persistence-meta.ts

@@ -8,7 +8,6 @@ import * as SDK from '../../core/sdk/sdk.js';
 import * as UI from '../../ui/legacy/legacy.js';
 import * as Workspace from '../workspace/workspace.js';
 
-// eslint-disable-next-line rulesdir/es_modules_import
 import type * as Persistence from './persistence.js';
 
 const UIStrings = {

package/front_end/models/timeline_model/TimelineJSProfile.ts

@@ -59,7 +59,6 @@ export class TimelineJSProfileProcessor {
 
   static generateJSFrameEvents(events: SDK.TracingModel.Event[], config: {
     showAllEvents: boolean,
-    showRuntimeCallStats: boolean,
     showNativeFunctions: boolean,
   }): SDK.TracingModel.Event[] {
     function equalFrames(frame1: Protocol.Runtime.CallFrame, frame2: Protocol.Runtime.CallFrame): boolean {
@@ -85,7 +84,7 @@ export class TimelineJSProfileProcessor {
     const lockedJsStackDepth: number[] = [];
     let ordinal = 0;
     let fakeJSInvocation = false;
-    const {showAllEvents, showRuntimeCallStats, showNativeFunctions} = config;
+    const {showAllEvents, showNativeFunctions} = config;
 
     function onStartEvent(e: SDK.TracingModel.Event): void {
       if (fakeJSInvocation) {
@@ -135,15 +134,10 @@ export class TimelineJSProfileProcessor {
       jsFramesStack.length = depth;
     }
 
-    function showNativeName(name: string): boolean {
-      return showRuntimeCallStats && Boolean(TimelineJSProfileProcessor.nativeGroup(name));
-    }
-
     function filterStackFrames(stack: Protocol.Runtime.CallFrame[]): void {
       if (showAllEvents) {
         return;
       }
-      let previousNativeFrameName: (string|null)|null = null;
       let j = 0;
       for (let i = 0; i < stack.length; ++i) {
         const frame = stack[i];
@@ -153,16 +147,9 @@ export class TimelineJSProfileProcessor {
           continue;
         }
         const isNativeRuntimeFrame = TimelineJSProfileProcessor.isNativeRuntimeFrame(frame);
-        if (isNativeRuntimeFrame && !showNativeName(frame.functionName)) {
-          continue;
-        }
-        const nativeFrameName =
-            isNativeRuntimeFrame ? TimelineJSProfileProcessor.nativeGroup(frame.functionName) : null;
-        if (previousNativeFrameName && previousNativeFrameName === nativeFrameName) {
-          continue;
+        if (!isNativeRuntimeFrame) {
+          stack[j++] = frame;
         }
-        previousNativeFrameName = nativeFrameName;
-        stack[j++] = frame;
       }
       stack.length = j;
     }

package/front_end/models/timeline_model/TimelineModel.ts

@@ -678,7 +678,6 @@ export class TimelineModelImpl {
     if (jsSamples || events.some(e => e.name === RecordType.JSSample)) {
       const jsFrameEvents = TimelineJSProfileProcessor.generateJSFrameEvents(events, {
         showAllEvents: Root.Runtime.experiments.isEnabled('timelineShowAllEvents'),
-        showRuntimeCallStats: Root.Runtime.experiments.isEnabled('timelineV8RuntimeCallStats'),
         showNativeFunctions: Common.Settings.Settings.instance().moduleSetting('showNativeFunctionsInJSProfile').get(),
       });
       if (jsFrameEvents && jsFrameEvents.length) {

package/front_end/panels/accessibility/accessibility-meta.ts

@@ -5,7 +5,6 @@
 import * as i18n from '../../core/i18n/i18n.js';
 import * as UI from '../../ui/legacy/legacy.js';
 
-// eslint-disable-next-line rulesdir/es_modules_import
 import type * as Accessibility from './accessibility.js';
 
 let loadedAccessibilityModule: (typeof Accessibility|undefined);

package/front_end/panels/animation/animation-meta.ts

@@ -5,7 +5,6 @@
 import * as i18n from '../../core/i18n/i18n.js';
 import * as UI from '../../ui/legacy/legacy.js';
 
-// eslint-disable-next-line rulesdir/es_modules_import
 import type * as Animation from './animation.js';
 
 let loadedAnimationModule: (typeof Animation|undefined);

package/front_end/panels/application/ApplicationPanelCacheSection.ts

@@ -20,7 +20,7 @@ const UIStrings = {
   /**
   *@description Text in Application Panel Sidebar of the Application panel
   */
-  backForwardCache: 'Back-Forward Cache',
+  backForwardCache: 'Back/forward cache',
   /**
   *@description A context menu item in the Application Panel Sidebar of the Application panel
   */

package/front_end/panels/application/BackForwardCacheView.ts

@@ -17,67 +17,67 @@ import backForwardCacheViewStyles from './backForwardCacheView.css.js';
 
 const UIStrings = {
   /**
-   * @description Title text in Back-Forward Cache view of the Application panel
+   * @description Title text in back/forward cache view of the Application panel
    */
   mainFrame: 'Main Frame',
   /**
-   * @description Title text in Back-Forward Cache view of the Application panel
+   * @description Title text in back/forward cache view of the Application panel
    */
-  backForwardCacheTitle: 'Back-Forward Cache',
+  backForwardCacheTitle: 'Back/forward cache',
   /**
    * @description Status text for the status of the main frame
    */
   unavailable: 'unavailable',
   /**
-   * @description Entry name text in the Back-Forward Cache view of the Application panel
+   * @description Entry name text in the back/forward cache view of the Application panel
    */
   url: 'URL',
   /**
-   * @description Entry name text in the Back-Forward Cache view of the Application panel
+   * @description Entry name text in the back/forward cache view of the Application panel
    */
-  bfcacheStatus: 'Back-Forward Cache Status',
+  bfcacheStatus: 'Back/forward cache Status',
   /**
-   * @description Status text for the status of the Back-Forward Cache status
+   * @description Status text for the status of the back/forward cache status
    */
   unknown: 'unknown',
   /**
-   * @description Status text for the status of the Back-Forward Cache status indicating that
-   * the Back-Forward Cache was not used and a normal navigation occured instead.
+   * @description Status text for the status of the back/forward cache status indicating that
+   * the back/forward cache was not used and a normal navigation occured instead.
    */
-  normalNavigation: 'Normal navigation (Not restored from Back-Forward Cache)',
+  normalNavigation: 'Normal navigation (Not restored from back/forward cache)',
   /**
-   * @description Status text for the status of the Back-Forward Cache status indicating that
-   * the Back-Forward Cache was used to restore the page instead of reloading it.
+   * @description Status text for the status of the back/forward cache status indicating that
+   * the back/forward cache was used to restore the page instead of reloading it.
    */
-  restoredFromBFCache: 'Restored from Back-Forward Cache',
+  restoredFromBFCache: 'Restored from back/forward cache',
   /**
    * @description Label for a list of reasons which prevent the page from being eligible for
-   * Back-Forward Cache. These reasons are actionable i.e. they can be cleaned up to make the
-   * page eligible for Back-Forward Cache.
+   * back/forward cache. These reasons are actionable i.e. they can be cleaned up to make the
+   * page eligible for back/forward cache.
    */
   pageSupportNeeded: 'Actionable',
   /**
    * @description Explanation for actionable items which prevent the page from being eligible
-   * for Back-Forward Cache.
+   * for back/forward cache.
    */
   pageSupportNeededExplanation:
-      'These reasons are actionable i.e. they can be cleaned up to make the page eligible for Back-Forward Cache.',
+      'These reasons are actionable i.e. they can be cleaned up to make the page eligible for back/forward cache.',
   /**
    * @description Label for a list of reasons which prevent the page from being eligible for
-   * Back-Forward Cache. These reasons are circumstantial / not actionable i.e. they cannot be
-   * cleaned up by developers to make the page eligible for Back-Forward Cache.
+   * back/forward cache. These reasons are circumstantial / not actionable i.e. they cannot be
+   * cleaned up by developers to make the page eligible for back/forward cache.
    */
   circumstantial: 'Not Actionable',
   /**
    * @description Explanation for circumstantial/non-actionable items which prevent the page from being eligible
-   * for Back-Forward Cache.
+   * for back/forward cache.
    */
   circumstantialExplanation:
       'These reasons are not actionable i.e. caching was prevented by something outside of the direct control of the page.',
   /**
    * @description Label for a list of reasons which prevent the page from being eligible for
-   * Back-Forward Cache. These reasons are pending support by chrome i.e. in a future version
-   * of chrome they will not prevent Back-Forward Cache usage anymore.
+   * back/forward cache. These reasons are pending support by chrome i.e. in a future version
+   * of chrome they will not prevent back/forward cache usage anymore.
    */
   supportPending: 'Pending Support',
   /**
@@ -86,10 +86,10 @@ const UIStrings = {
   runTest: 'Run Test',
   /**
    * @description Explanation for 'pending support' items which prevent the page from being eligible
-   * for Back-Forward Cache.
+   * for back/forward cache.
    */
   supportPendingExplanation:
-      'Chrome support for these reasons is pending i.e. they will not prevent the page from being eligible for Back-Forward Cache in a future version of Chrome.',
+      'Chrome support for these reasons is pending i.e. they will not prevent the page from being eligible for back/forward cache in a future version of Chrome.',
 };
 const str_ = i18n.i18n.registerUIStrings('panels/application/BackForwardCacheView.ts', UIStrings);
 const i18nString = i18n.i18n.getLocalizedString.bind(undefined, str_);
@@ -168,9 +168,9 @@ export class BackForwardCacheView extends UI.ThrottledWidget.ThrottledWidget {
 
       // We can know whether the current page can use BFCache
       // as the browser navigates to another unrelated page and goes back to the current page.
-      // We chose "chrome://version" because it must be cross-site.
+      // We chose "chrome://terms" because it must be cross-site.
       // Ideally, We want to have our own testing page like "chrome: //bfcache-test".
-      resourceTreeModel.navigate('chrome://version/');
+      resourceTreeModel.navigate('chrome://terms');
     }
   }
 

package/front_end/panels/application/application-meta.ts

@@ -6,7 +6,6 @@ import * as Common from '../../core/common/common.js';
 import * as SDK from '../../core/sdk/sdk.js';
 import * as UI from '../../ui/legacy/legacy.js';
 
-// eslint-disable-next-line rulesdir/es_modules_import
 import type * as Resources from './application.js';
 
 import * as i18n from '../../core/i18n/i18n.js';

package/front_end/panels/application/components/stackTraceRow.css

@@ -26,3 +26,11 @@
 .ignore-list-link {
   opacity: 60%;
 }
+
+.link,
+.devtools-link {
+  color: var(--color-link);
+  text-decoration: underline;
+  cursor: pointer;
+  padding: 2px 0; /* adjust focus ring size */
+}

package/front_end/panels/browser_debugger/browser_debugger-meta.ts

@@ -4,10 +4,9 @@
 
 import * as SDK from '../../core/sdk/sdk.js';
 import * as UI from '../../ui/legacy/legacy.js';
+import type * as Sources from '../sources/sources.js';
 
-// eslint-disable-next-line rulesdir/es_modules_import
 import type * as BrowserDebugger from './browser_debugger.js';
-import type * as Sources from '../sources/sources.js';
 
 import * as i18n from '../../core/i18n/i18n.js';
 const UIStrings = {

package/front_end/panels/changes/changes-meta.ts

@@ -7,7 +7,6 @@ import * as i18n from '../../core/i18n/i18n.js';
 import * as WorkspaceDiff from '../../models/workspace_diff/workspace_diff.js';
 import * as UI from '../../ui/legacy/legacy.js';
 
-// eslint-disable-next-line rulesdir/es_modules_import
 import type * as Changes from './changes.js';
 
 let loadedChangesModule: (typeof Changes|undefined);

package/front_end/panels/console/ConsolePinPane.ts

@@ -164,8 +164,6 @@ export class ConsolePinPane extends UI.ThrottledWidget.ThrottledWidget {
   }
 }
 
-let consolePinNumber = 0;
-
 export class ConsolePin {
   private readonly pinElement: Element;
   private readonly pinPreview: HTMLElement;
@@ -175,11 +173,9 @@ export class ConsolePin {
   private committedExpression: string;
   private hovered: boolean;
   private lastNode: SDK.RemoteObject.RemoteObject|null;
-  private consolePinNumber: number;
   private deletePinIcon: UI.UIUtils.DevToolsCloseButton;
 
   constructor(expression: string, private readonly pinPane: ConsolePinPane, private readonly focusOut: () => void) {
-    this.consolePinNumber = ++consolePinNumber;
     this.deletePinIcon = document.createElement('div', {is: 'dt-close-button'}) as UI.UIUtils.DevToolsCloseButton;
     this.deletePinIcon.gray = true;
     this.deletePinIcon.classList.add('close-button');
@@ -259,6 +255,7 @@ export class ConsolePin {
         ]),
         CodeMirror.EditorView.domEventHandlers({blur: (_e, view) => this.onBlur(view)}),
         TextEditor.Config.baseConfiguration(expression),
+        TextEditor.Config.closeBrackets,
         TextEditor.Config.autocompletion,
       ],
     }));
@@ -325,8 +322,8 @@ export class ConsolePin {
     const executionContext = UI.Context.Context.instance().flavor(SDK.RuntimeModel.ExecutionContext);
     const preprocessedExpression = ObjectUI.JavaScriptREPL.JavaScriptREPL.preprocessExpression(text);
     const {preview, result} = await ObjectUI.JavaScriptREPL.JavaScriptREPL.evaluateAndBuildPreview(
-        `${preprocessedExpression}\n//# sourceURL=watch-expression-${this.consolePinNumber}.devtools`,
-        throwOnSideEffect, false /* replMode */, timeout, !isEditing /* allowErrors */, 'console');
+        preprocessedExpression, throwOnSideEffect, false /* replMode */, timeout, !isEditing /* allowErrors */,
+        'console', true /* awaitPromise */);
     if (this.lastResult && this.lastExecutionContext) {
       this.lastExecutionContext.runtimeModel.releaseEvaluationResult(this.lastResult);
     }