chrome-devtools-frontend 1.0.1512147 → 1.0.1513662

package/front_end/panels/settings/SettingsScreen.ts

@@ -250,6 +250,7 @@ export class GenericSettingsTab extends UI.Widget.VBox implements SettingsTab {
   private readonly settingToControl = new Map<Common.Settings.Setting<unknown>, HTMLElement>();
   private readonly containerElement: HTMLElement;
   #updateSyncSectionTimerId = -1;
+  #syncSectionUpdatePromise: Promise<void>|null = null;
 
   constructor() {
     super({jslog: `${VisualLogging.pane('preferences')}`});
@@ -336,24 +337,19 @@ export class GenericSettingsTab extends UI.Widget.VBox implements SettingsTab {
       this.#updateSyncSectionTimerId = -1;
     }
 
-    void Promise
-        .all([
-          new Promise<Host.InspectorFrontendHostAPI.SyncInformation>(
-              resolve => Host.InspectorFrontendHost.InspectorFrontendHostInstance.getSyncInformation(resolve)),
-          Root.Runtime.hostConfig.devToolsGdpProfiles?.enabled ? Host.GdpClient.GdpClient.instance().getProfile() :
-                                                                 Promise.resolve(undefined),
-        ])
-        .then(([syncInfo, gdpProfile]) => {
-          this.syncSection.data = {
-            syncInfo,
-            syncSetting: Common.Settings.moduleSetting('sync-preferences') as Common.Settings.Setting<boolean>,
-            receiveBadgesSetting: Common.Settings.Settings.instance().moduleSetting('receive-gdp-badges'),
-            gdpProfile: gdpProfile ?? undefined,
-          };
-          if (!syncInfo.isSyncActive || !syncInfo.arePreferencesSynced) {
-            this.#updateSyncSectionTimerId = window.setTimeout(this.updateSyncSection.bind(this), 500);
-          }
-        });
+    this.#syncSectionUpdatePromise =
+        new Promise<Host.InspectorFrontendHostAPI.SyncInformation>(
+            resolve => Host.InspectorFrontendHost.InspectorFrontendHostInstance.getSyncInformation(resolve))
+            .then(syncInfo => {
+              this.syncSection.data = {
+                syncInfo,
+                syncSetting: Common.Settings.moduleSetting('sync-preferences') as Common.Settings.Setting<boolean>,
+                receiveBadgesSetting: Common.Settings.Settings.instance().moduleSetting('receive-gdp-badges'),
+              };
+              if (!syncInfo.isSyncActive || !syncInfo.arePreferencesSynced) {
+                this.#updateSyncSectionTimerId = window.setTimeout(this.updateSyncSection.bind(this), 500);
+              }
+            });
   }
 
   private createExtensionSection(settings: Common.Settings.SettingRegistration[]): void {
@@ -405,6 +401,10 @@ export class GenericSettingsTab extends UI.Widget.VBox implements SettingsTab {
       const element = this.settingToControl.get(setting);
       if (element) {
         PanelUtils.highlightElement(element);
+      } else if (setting.name === 'receive-gdp-badges') {
+        void this.#syncSectionUpdatePromise?.then(() => {
+          void this.syncSection.highlightReceiveBadgesSetting();
+        });
       }
     }
   }

package/front_end/panels/settings/components/SyncSection.ts

@@ -16,6 +16,7 @@ import * as Buttons from '../../../ui/components/buttons/buttons.js';
 import * as ComponentHelpers from '../../../ui/components/helpers/helpers.js';
 import * as Lit from '../../../ui/lit/lit.js';
 import * as PanelCommon from '../../common/common.js';
+import * as PanelUtils from '../../utils/utils.js';
 
 import syncSectionStyles from './syncSection.css.js';
 
@@ -87,7 +88,7 @@ const str_ = i18n.i18n.registerUIStrings('panels/settings/components/SyncSection
 const i18nString = i18n.i18n.getLocalizedString.bind(undefined, str_);
 const lockedString = i18n.i18n.lockedString;
 
-const {html} = Lit;
+const {html, Directives: {ref, createRef}} = Lit;
 
 function getGdpSubscriptionText(profile: Host.GdpClient.Profile): Platform.UIString.LocalizedString {
   if (!profile.activeSubscription ||
@@ -110,14 +111,11 @@ function getGdpSubscriptionText(profile: Host.GdpClient.Profile): Platform.UIStr
 }
 
 const GDP_LOGO_IMAGE_URL = new URL('../../../Images/gdp-logo-standalone.svg', import.meta.url).toString();
-// TODO(crbug.com/441679275): Update once the API is enabled for prod.
-const GOOGLE_DEVELOPER_PROGRAM_PROFILE_LINK = 'https://developers.devsite.corp.google.com/profile/u/';
 
 export interface SyncSectionData {
   syncInfo: Host.InspectorFrontendHostAPI.SyncInformation;
   syncSetting: Common.Settings.Setting<boolean>;
   receiveBadgesSetting: Common.Settings.Setting<boolean>;
-  gdpProfile?: Host.GdpClient.Profile;
 }
 
 export class SyncSection extends HTMLElement {
@@ -126,16 +124,25 @@ export class SyncSection extends HTMLElement {
   #syncInfo: Host.InspectorFrontendHostAPI.SyncInformation = {isSyncActive: false};
   #syncSetting?: Common.Settings.Setting<boolean>;
   #receiveBadgesSetting?: Common.Settings.Setting<boolean>;
+  #receiveBadgesSettingContainerRef = createRef<HTMLElement>();
   #gdpProfile?: Host.GdpClient.Profile;
 
   set data(data: SyncSectionData) {
     this.#syncInfo = data.syncInfo;
     this.#syncSetting = data.syncSetting;
     this.#receiveBadgesSetting = data.receiveBadgesSetting;
-    this.#gdpProfile = data.gdpProfile;
+    void this.#updateGdpProfile();
     void ComponentHelpers.ScheduledRender.scheduleRender(this, this.#render);
   }
 
+  async highlightReceiveBadgesSetting(): Promise<void> {
+    await ComponentHelpers.ScheduledRender.scheduleRender(this, this.#render);
+    const element = this.#receiveBadgesSettingContainerRef.value;
+    if (element) {
+      PanelUtils.PanelUtils.highlightElement(element);
+    }
+  }
+
   #render(): void {
     if (!this.#syncSetting) {
       throw new Error('SyncSection is not properly initialized');
@@ -152,11 +159,20 @@ export class SyncSection extends HTMLElement {
       <fieldset>
         ${renderAccountInfo(this.#syncInfo)}
         ${renderSettingCheckboxIfNeeded(this.#syncInfo, this.#syncSetting)}
-        ${renderGdpSectionIfNeeded({ receiveBadgesSetting: this.#receiveBadgesSetting, gdpProfile: this.#gdpProfile })}
+        ${renderGdpSectionIfNeeded({
+          receiveBadgesSetting: this.#receiveBadgesSetting,
+          receiveBadgesSettingContainerRef: this.#receiveBadgesSettingContainerRef,
+          gdpProfile: this.#gdpProfile
+        })}
       </fieldset>
     `, this.#shadow, {host: this});
     // clang-format on
   }
+
+  async #updateGdpProfile(): Promise<void> {
+    this.#gdpProfile = await Host.GdpClient.GdpClient.instance().getProfile() ?? undefined;
+    void ComponentHelpers.ScheduledRender.scheduleRender(this, this.#render);
+  }
 }
 
 function renderSettingCheckboxIfNeeded(
@@ -230,8 +246,10 @@ function renderAccountInfo(syncInfo: Host.InspectorFrontendHostAPI.SyncInformati
 
 function renderGdpSectionIfNeeded({
   receiveBadgesSetting,
+  receiveBadgesSettingContainerRef,
   gdpProfile,
 }: {
+  receiveBadgesSettingContainerRef: Lit.Directives.Ref<HTMLElement>,
   receiveBadgesSetting?: Common.Settings.Setting<boolean>,
   gdpProfile?: Host.GdpClient.Profile,
 }): Lit.LitTemplate {
@@ -258,11 +276,11 @@ function renderGdpSectionIfNeeded({
           <div class="plan-details">
             ${getGdpSubscriptionText(gdpProfile)}
             &nbsp;·&nbsp;
-            <x-link class="link" href=${GOOGLE_DEVELOPER_PROGRAM_PROFILE_LINK}>
+            <x-link class="link" href=${Host.GdpClient.GOOGLE_DEVELOPER_PROGRAM_PROFILE_LINK}>
               ${i18nString(UIStrings.viewProfile)}
             </x-link></div>
             ${receiveBadgesSetting ? html`
-              <div class="setting-container">
+              <div class="setting-container"  ${ref(receiveBadgesSettingContainerRef)}>
                 <setting-checkbox class="setting-checkbox" .data=${{setting: receiveBadgesSetting}}></setting-checkbox>
                 <span>${i18nString(UIStrings.relevantDataDisclaimer)}</span>
               </div>` : Lit.nothing}

package/front_end/panels/sources/AiCodeCompletionPlugin.ts

@@ -121,6 +121,9 @@ export class AiCodeCompletionPlugin extends Plugin {
     const query = doc.toString();
     const cursor = selection.main.head;
     let prefix = query.substring(0, cursor);
+    if (prefix.trim().length === 0) {
+      return;
+    }
     let suffix = query.substring(cursor);
     if (prefix.length > AI_CODE_COMPLETION_CHARACTER_LIMIT) {
       prefix = prefix.substring(prefix.length - AI_CODE_COMPLETION_CHARACTER_LIMIT);

package/front_end/panels/sources/DebuggerPausedMessage.ts

@@ -117,21 +117,20 @@ const i18nString = i18n.i18n.getLocalizedString.bind(undefined, str_);
 const i18nLazyString = i18n.i18n.getLazilyComputedLocalizedString.bind(undefined, str_);
 
 export class DebuggerPausedMessage {
-  private readonly elementInternal: HTMLDivElement;
+  readonly #element: HTMLDivElement;
   private contentElement: HTMLElement;
   constructor() {
-    this.elementInternal = document.createElement('div');
-    this.elementInternal.classList.add('paused-message');
-    this.elementInternal.classList.add('flex-none');
-    this.elementInternal.setAttribute('jslog', `${VisualLogging.dialog('debugger-paused')}`);
-    const root =
-        UI.UIUtils.createShadowRootWithCoreStyles(this.elementInternal, {cssFile: debuggerPausedMessageStyles});
+    this.#element = document.createElement('div');
+    this.#element.classList.add('paused-message');
+    this.#element.classList.add('flex-none');
+    this.#element.setAttribute('jslog', `${VisualLogging.dialog('debugger-paused')}`);
+    const root = UI.UIUtils.createShadowRootWithCoreStyles(this.#element, {cssFile: debuggerPausedMessageStyles});
     this.contentElement = root.createChild('div');
-    UI.ARIAUtils.markAsPoliteLiveRegion(this.elementInternal, false);
+    UI.ARIAUtils.markAsPoliteLiveRegion(this.#element, false);
   }
 
   element(): Element {
-    return this.elementInternal;
+    return this.#element;
   }
 
   private static descriptionWithoutStack(description: string): string {

package/front_end/panels/sources/NavigatorView.ts

@@ -206,7 +206,7 @@ export class NavigatorView extends UI.Widget.VBox implements SDK.TargetManager.O
   private deployedNode?: NavigatorGroupTreeNode;
   private navigatorGroupByFolderSetting: Common.Settings.Setting<boolean>;
   private navigatorGroupByAuthoredExperiment?: string;
-  private workspaceInternal!: Workspace.Workspace.WorkspaceImpl;
+  #workspace!: Workspace.Workspace.WorkspaceImpl;
   private groupByFrame?: boolean;
   private groupByAuthored?: boolean;
   private groupByDomain?: boolean;
@@ -262,7 +262,7 @@ export class NavigatorView extends UI.Widget.VBox implements SDK.TargetManager.O
 
     SDK.TargetManager.TargetManager.instance().observeTargets(this);
     this.resetWorkspace(Workspace.Workspace.WorkspaceImpl.instance());
-    this.workspaceInternal.uiSourceCodes().forEach(this.addUISourceCode.bind(this));
+    this.#workspace.uiSourceCodes().forEach(this.addUISourceCode.bind(this));
     Bindings.NetworkProject.NetworkProjectManager.instance().addEventListener(
         Bindings.NetworkProject.Events.FRAME_ATTRIBUTION_ADDED, this.frameAttributionAdded, this);
     Bindings.NetworkProject.NetworkProjectManager.instance().addEventListener(
@@ -408,26 +408,23 @@ export class NavigatorView extends UI.Widget.VBox implements SDK.TargetManager.O
 
   private resetWorkspace(workspace: Workspace.Workspace.WorkspaceImpl): void {
     // Clear old event listeners first.
-    if (this.workspaceInternal) {
-      this.workspaceInternal.removeEventListener(
+    if (this.#workspace) {
+      this.#workspace.removeEventListener(
           Workspace.Workspace.Events.UISourceCodeAdded, this.uiSourceCodeAddedCallback, this);
-      this.workspaceInternal.removeEventListener(
+      this.#workspace.removeEventListener(
           Workspace.Workspace.Events.UISourceCodeRemoved, this.uiSourceCodeRemovedCallback, this);
-      this.workspaceInternal.removeEventListener(
-          Workspace.Workspace.Events.ProjectAdded, this.projectAddedCallback, this);
-      this.workspaceInternal.removeEventListener(
-          Workspace.Workspace.Events.ProjectRemoved, this.projectRemovedCallback, this);
+      this.#workspace.removeEventListener(Workspace.Workspace.Events.ProjectAdded, this.projectAddedCallback, this);
+      this.#workspace.removeEventListener(Workspace.Workspace.Events.ProjectRemoved, this.projectRemovedCallback, this);
     }
 
-    this.workspaceInternal = workspace;
-    this.workspaceInternal.addEventListener(
+    this.#workspace = workspace;
+    this.#workspace.addEventListener(
         Workspace.Workspace.Events.UISourceCodeAdded, this.uiSourceCodeAddedCallback, this);
-    this.workspaceInternal.addEventListener(
+    this.#workspace.addEventListener(
         Workspace.Workspace.Events.UISourceCodeRemoved, this.uiSourceCodeRemovedCallback, this);
-    this.workspaceInternal.addEventListener(Workspace.Workspace.Events.ProjectAdded, this.projectAddedCallback, this);
-    this.workspaceInternal.addEventListener(
-        Workspace.Workspace.Events.ProjectRemoved, this.projectRemovedCallback, this);
-    this.workspaceInternal.projects().forEach(this.projectAdded.bind(this));
+    this.#workspace.addEventListener(Workspace.Workspace.Events.ProjectAdded, this.projectAddedCallback, this);
+    this.#workspace.addEventListener(Workspace.Workspace.Events.ProjectRemoved, this.projectRemovedCallback, this);
+    this.#workspace.projects().forEach(this.projectAdded.bind(this));
     this.computeUniqueFileSystemProjectNames();
   }
 
@@ -448,7 +445,7 @@ export class NavigatorView extends UI.Widget.VBox implements SDK.TargetManager.O
   }
 
   workspace(): Workspace.Workspace.WorkspaceImpl {
-    return this.workspaceInternal;
+    return this.#workspace;
   }
 
   acceptProject(project: Workspace.Workspace.Project): boolean {
@@ -587,7 +584,7 @@ export class NavigatorView extends UI.Widget.VBox implements SDK.TargetManager.O
   }
 
   private computeUniqueFileSystemProjectNames(): void {
-    const fileSystemProjects = this.workspaceInternal.projectsForType(Workspace.Workspace.projectTypes.FileSystem);
+    const fileSystemProjects = this.#workspace.projectsForType(Workspace.Workspace.projectTypes.FileSystem);
     if (!fileSystemProjects.length) {
       return;
     }
@@ -1213,7 +1210,7 @@ export class NavigatorView extends UI.Widget.VBox implements SDK.TargetManager.O
     this.initGrouping();
     // Reset the workspace to repopulate filesystem folders.
     this.resetWorkspace(Workspace.Workspace.WorkspaceImpl.instance());
-    this.workspaceInternal.uiSourceCodes().forEach(this.addUISourceCode.bind(this));
+    this.#workspace.uiSourceCodes().forEach(this.addUISourceCode.bind(this));
   }
 
   private ignoreListChanged(): void {
@@ -1237,7 +1234,7 @@ export class NavigatorView extends UI.Widget.VBox implements SDK.TargetManager.O
 
   protected resetForTest(): void {
     this.reset();
-    this.workspaceInternal.uiSourceCodes().forEach(this.addUISourceCode.bind(this));
+    this.#workspace.uiSourceCodes().forEach(this.addUISourceCode.bind(this));
   }
 
   private discardFrame(frame: SDK.ResourceTreeModel.ResourceTreeFrame, isAuthored: boolean): void {
@@ -1414,7 +1411,7 @@ export class NavigatorSourceTreeElement extends UI.TreeOutline.TreeElement {
   readonly nodeType: string;
   readonly node: NavigatorUISourceCodeTreeNode;
   private readonly navigatorView: NavigatorView;
-  uiSourceCodeInternal: Workspace.UISourceCode.UISourceCode;
+  #uiSourceCode: Workspace.UISourceCode.UISourceCode;
   private aiButtonContainer?: HTMLElement;
 
   constructor(
@@ -1430,18 +1427,18 @@ export class NavigatorSourceTreeElement extends UI.TreeOutline.TreeElement {
     UI.ARIAUtils.setLabel(this.listItemElement, `${uiSourceCode.name()}, ${this.nodeType}`);
     Common.EventTarget.fireEvent('source-tree-file-added', uiSourceCode.fullDisplayName());
     this.navigatorView = navigatorView;
-    this.uiSourceCodeInternal = uiSourceCode;
+    this.#uiSourceCode = uiSourceCode;
     this.updateIcon();
     (this.titleElement as HTMLElement).setAttribute('jslog', `${VisualLogging.value('title').track({change: true})}`);
   }
 
   updateIcon(): void {
-    const icon = PanelUtils.getIconForSourceFile(this.uiSourceCodeInternal);
+    const icon = PanelUtils.getIconForSourceFile(this.#uiSourceCode);
     this.setLeadingIcons([icon]);
   }
 
   updateAccessibleName(): void {
-    UI.ARIAUtils.setLabel(this.listItemElement, `${this.uiSourceCodeInternal.name()}, ${this.nodeType}`);
+    UI.ARIAUtils.setLabel(this.listItemElement, `${this.#uiSourceCode.name()}, ${this.nodeType}`);
   }
 
   private createAiButton(): void {
@@ -1469,7 +1466,7 @@ export class NavigatorSourceTreeElement extends UI.TreeOutline.TreeElement {
   }
 
   get uiSourceCode(): Workspace.UISourceCode.UISourceCode {
-    return this.uiSourceCodeInternal;
+    return this.#uiSourceCode;
   }
 
   override onattach(): void {
@@ -1481,7 +1478,7 @@ export class NavigatorSourceTreeElement extends UI.TreeOutline.TreeElement {
   }
 
   private shouldRenameOnMouseDown(): boolean {
-    if (!this.uiSourceCodeInternal.canRename()) {
+    if (!this.#uiSourceCode.canRename()) {
       return false;
     }
     if (!this.treeOutline) {
@@ -1509,7 +1506,7 @@ export class NavigatorSourceTreeElement extends UI.TreeOutline.TreeElement {
     if (!event.dataTransfer) {
       return;
     }
-    event.dataTransfer.setData('text/plain', this.uiSourceCodeInternal.url());
+    event.dataTransfer.setData('text/plain', this.#uiSourceCode.url());
     event.dataTransfer.effectAllowed = 'copy';
   }
 
@@ -1712,18 +1709,18 @@ export class NavigatorRootTreeNode extends NavigatorTreeNode {
 }
 
 export class NavigatorUISourceCodeTreeNode extends NavigatorTreeNode {
-  uiSourceCodeInternal: Workspace.UISourceCode.UISourceCode;
+  #uiSourceCode: Workspace.UISourceCode.UISourceCode;
   treeElement: NavigatorSourceTreeElement|null;
   private eventListeners: Common.EventTarget.EventDescriptor[];
-  private readonly frameInternal: SDK.ResourceTreeModel.ResourceTreeFrame|null;
+  readonly #frame: SDK.ResourceTreeModel.ResourceTreeFrame|null;
   constructor(
       navigatorView: NavigatorView, uiSourceCode: Workspace.UISourceCode.UISourceCode,
       frame: SDK.ResourceTreeModel.ResourceTreeFrame|null) {
     super(navigatorView, 'UISourceCode:' + uiSourceCode.canonicalScriptId(), Types.File);
-    this.uiSourceCodeInternal = uiSourceCode;
+    this.#uiSourceCode = uiSourceCode;
     this.treeElement = null;
     this.eventListeners = [];
-    this.frameInternal = frame;
+    this.#frame = frame;
     this.recursiveProperties.exclusivelySourceMapped = uiSourceCode.contentType().isFromSourceMap();
     if (uiSourceCode.contentType().isScript()) {
       // These properties affect ignore-listing menus and only matter when the UISourceCode is a script
@@ -1734,11 +1731,11 @@ export class NavigatorUISourceCodeTreeNode extends NavigatorTreeNode {
   }
 
   frame(): SDK.ResourceTreeModel.ResourceTreeFrame|null {
-    return this.frameInternal;
+    return this.#frame;
   }
 
   uiSourceCode(): Workspace.UISourceCode.UISourceCode {
-    return this.uiSourceCodeInternal;
+    return this.#uiSourceCode;
   }
 
   override treeNode(): UI.TreeOutline.TreeElement {
@@ -1746,14 +1743,14 @@ export class NavigatorUISourceCodeTreeNode extends NavigatorTreeNode {
       return this.treeElement;
     }
 
-    this.treeElement = new NavigatorSourceTreeElement(this.navigatorView, this.uiSourceCodeInternal, '', this);
+    this.treeElement = new NavigatorSourceTreeElement(this.navigatorView, this.#uiSourceCode, '', this);
     this.updateTitle();
 
     const updateTitleBound = this.updateTitle.bind(this, undefined);
     this.eventListeners = [
-      this.uiSourceCodeInternal.addEventListener(Workspace.UISourceCode.Events.TitleChanged, updateTitleBound),
-      this.uiSourceCodeInternal.addEventListener(Workspace.UISourceCode.Events.WorkingCopyChanged, updateTitleBound),
-      this.uiSourceCodeInternal.addEventListener(Workspace.UISourceCode.Events.WorkingCopyCommitted, updateTitleBound),
+      this.#uiSourceCode.addEventListener(Workspace.UISourceCode.Events.TitleChanged, updateTitleBound),
+      this.#uiSourceCode.addEventListener(Workspace.UISourceCode.Events.WorkingCopyChanged, updateTitleBound),
+      this.#uiSourceCode.addEventListener(Workspace.UISourceCode.Events.WorkingCopyCommitted, updateTitleBound),
     ];
     return this.treeElement;
   }
@@ -1761,8 +1758,8 @@ export class NavigatorUISourceCodeTreeNode extends NavigatorTreeNode {
   override updateTitle(ignoreIsDirty?: boolean): void {
     const isIgnoreListed =
         Workspace.IgnoreListManager.IgnoreListManager.instance().isUserOrSourceMapIgnoreListedUISourceCode(
-            this.uiSourceCodeInternal);
-    if (this.uiSourceCodeInternal.contentType().isScript() || isIgnoreListed) {
+            this.#uiSourceCode);
+    if (this.#uiSourceCode.contentType().isScript() || isIgnoreListed) {
       this.recursiveProperties.exclusivelyIgnored = isIgnoreListed;
     }
 
@@ -1770,8 +1767,8 @@ export class NavigatorUISourceCodeTreeNode extends NavigatorTreeNode {
       return;
     }
 
-    let titleText = this.uiSourceCodeInternal.displayName();
-    if (!ignoreIsDirty && this.uiSourceCodeInternal.isDirty()) {
+    let titleText = this.#uiSourceCode.displayName();
+    if (!ignoreIsDirty && this.#uiSourceCode.isDirty()) {
       titleText = '*' + titleText;
     }
 
@@ -1780,9 +1777,9 @@ export class NavigatorUISourceCodeTreeNode extends NavigatorTreeNode {
 
     this.treeElement.listItemElement.classList.toggle('is-ignore-listed', isIgnoreListed);
 
-    let tooltip: string = this.uiSourceCodeInternal.url();
-    if (this.uiSourceCodeInternal.contentType().isFromSourceMap()) {
-      tooltip = i18nString(UIStrings.sFromSourceMap, {PH1: this.uiSourceCodeInternal.displayName()});
+    let tooltip: string = this.#uiSourceCode.url();
+    if (this.#uiSourceCode.contentType().isFromSourceMap()) {
+      tooltip = i18nString(UIStrings.sFromSourceMap, {PH1: this.#uiSourceCode.displayName()});
     }
     if (isIgnoreListed) {
       tooltip = i18nString(UIStrings.sIgnoreListed, {PH1: tooltip});
@@ -1791,7 +1788,7 @@ export class NavigatorUISourceCodeTreeNode extends NavigatorTreeNode {
     this.treeElement.tooltip = tooltip;
     this.treeElement.updateAccessibleName();
 
-    this.updateId('UISourceCode:' + this.uiSourceCodeInternal.canonicalScriptId());
+    this.updateId('UISourceCode:' + this.#uiSourceCode.canonicalScriptId());
   }
 
   override hasChildren(): boolean {
@@ -1836,7 +1833,7 @@ export class NavigatorUISourceCodeTreeNode extends NavigatorTreeNode {
           this.treeElement.title = newTitle;
         }
         // necessary cast to RawPathString as alternative would be altering type of Config<T>
-        void this.uiSourceCodeInternal.rename(newTitle as Platform.DevToolsPath.RawPathString).then(renameCallback);
+        void this.#uiSourceCode.rename(newTitle as Platform.DevToolsPath.RawPathString).then(renameCallback);
         return;
       }
       afterEditing(true);