chrome-devtools-frontend 1.0.950971 → 1.0.952403

package/front_end/panels/issues/AffectedResourcesView.ts

@@ -62,27 +62,27 @@ export interface CreateRequestCellOptions {
  * as well as machinery for resolving request and frame ids to SDK objects.
  */
 export abstract class AffectedResourcesView extends UI.TreeOutline.TreeElement {
-  private readonly parentView: IssueView;
+  readonly #parentView: IssueView;
   protected issue: AggregatedIssue;
   protected affectedResourcesCountElement: HTMLElement;
   protected affectedResources: HTMLElement;
-  private affectedResourcesCount: number;
-  private frameListeners: Common.EventTarget.EventDescriptor[];
-  private unresolvedFrameIds: Set<string>;
+  #affectedResourcesCount: number;
+  #frameListeners: Common.EventTarget.EventDescriptor[];
+  #unresolvedFrameIds: Set<string>;
   protected requestResolver: Logs.RequestResolver.RequestResolver;
 
   constructor(parent: IssueView, issue: AggregatedIssue) {
     super();
-    this.parentView = parent;
+    this.#parentView = parent;
     this.issue = issue;
     this.toggleOnClick = true;
     this.affectedResourcesCountElement = this.createAffectedResourcesCounter();
 
     this.affectedResources = this.createAffectedResources();
-    this.affectedResourcesCount = 0;
+    this.#affectedResourcesCount = 0;
     this.requestResolver = new Logs.RequestResolver.RequestResolver();
-    this.frameListeners = [];
-    this.unresolvedFrameIds = new Set();
+    this.#frameListeners = [];
+    this.#unresolvedFrameIds = new Set();
   }
 
   /**
@@ -113,14 +113,14 @@ export abstract class AffectedResourcesView extends UI.TreeOutline.TreeElement {
   protected abstract getResourceNameWithCount(count: number): string;
 
   protected updateAffectedResourceCount(count: number): void {
-    this.affectedResourcesCount = count;
+    this.#affectedResourcesCount = count;
     this.affectedResourcesCountElement.textContent = this.getResourceNameWithCount(count);
-    this.hidden = this.affectedResourcesCount === 0;
-    this.parentView.updateAffectedResourceVisibility();
+    this.hidden = this.#affectedResourcesCount === 0;
+    this.#parentView.updateAffectedResourceVisibility();
   }
 
   isEmpty(): boolean {
-    return this.affectedResourcesCount === 0;
+    return this.#affectedResourcesCount === 0;
   }
 
   clear(): void {
@@ -129,7 +129,7 @@ export abstract class AffectedResourcesView extends UI.TreeOutline.TreeElement {
   }
 
   expandIfOneResource(): void {
-    if (this.affectedResourcesCount === 1) {
+    if (this.#affectedResourcesCount === 1) {
       this.expand();
     }
   }
@@ -139,32 +139,31 @@ export abstract class AffectedResourcesView extends UI.TreeOutline.TreeElement {
    * a listener is installed that takes care of updating the view if the frame is added. This is useful if the issue is
    * added before the frame gets reported.
    */
-  private resolveFrameId(frameId: Protocol.Page.FrameId): SDK.ResourceTreeModel.ResourceTreeFrame|null {
+  #resolveFrameId(frameId: Protocol.Page.FrameId): SDK.ResourceTreeModel.ResourceTreeFrame|null {
     const frame = SDK.FrameManager.FrameManager.instance().getFrame(frameId);
     if (!frame || !frame.url) {
-      this.unresolvedFrameIds.add(frameId);
-      if (!this.frameListeners.length) {
+      this.#unresolvedFrameIds.add(frameId);
+      if (!this.#frameListeners.length) {
         const addListener = SDK.FrameManager.FrameManager.instance().addEventListener(
-            SDK.FrameManager.Events.FrameAddedToTarget, this.onFrameChanged, this);
+            SDK.FrameManager.Events.FrameAddedToTarget, this.#onFrameChanged, this);
         const navigateListener = SDK.FrameManager.FrameManager.instance().addEventListener(
-            SDK.FrameManager.Events.FrameNavigated, this.onFrameChanged, this);
-        this.frameListeners = [addListener, navigateListener];
+            SDK.FrameManager.Events.FrameNavigated, this.#onFrameChanged, this);
+        this.#frameListeners = [addListener, navigateListener];
       }
     }
     return frame;
   }
 
-  private onFrameChanged(event: Common.EventTarget.EventTargetEvent<{frame: SDK.ResourceTreeModel.ResourceTreeFrame}>):
-      void {
+  #onFrameChanged(event: Common.EventTarget.EventTargetEvent<{frame: SDK.ResourceTreeModel.ResourceTreeFrame}>): void {
     const frame = event.data.frame;
     if (!frame.url) {
       return;
     }
-    const frameWasUnresolved = this.unresolvedFrameIds.delete(frame.id);
-    if (this.unresolvedFrameIds.size === 0 && this.frameListeners.length) {
+    const frameWasUnresolved = this.#unresolvedFrameIds.delete(frame.id);
+    if (this.#unresolvedFrameIds.size === 0 && this.#frameListeners.length) {
       // Stop listening once all requests are resolved.
-      Common.EventTarget.removeEventListeners(this.frameListeners);
-      this.frameListeners = [];
+      Common.EventTarget.removeEventListeners(this.#frameListeners);
+      this.#frameListeners = [];
     }
     if (frameWasUnresolved) {
       this.update();
@@ -173,7 +172,7 @@ export abstract class AffectedResourcesView extends UI.TreeOutline.TreeElement {
 
   protected createFrameCell(frameId: Protocol.Page.FrameId, issueCategory: IssuesManager.Issue.IssueCategory):
       HTMLElement {
-    const frame = this.resolveFrameId(frameId);
+    const frame = this.#resolveFrameId(frameId);
     const url = frame && (frame.unreachableUrl() || frame.url) || i18nString(UIStrings.unknown);
 
     const frameCell = document.createElement('td');

package/front_end/panels/issues/AffectedSharedArrayBufferIssueDetailsView.ts

@@ -58,7 +58,7 @@ export class AffectedSharedArrayBufferIssueDetailsView extends AffectedResources
     return i18nString(UIStrings.nViolations, {n: count});
   }
 
-  private appendStatus(element: HTMLElement, isWarning: boolean): void {
+  #appendStatus(element: HTMLElement, isWarning: boolean): void {
     const status = document.createElement('td');
     if (isWarning) {
       status.classList.add('affected-resource-report-only-status');
@@ -70,7 +70,7 @@ export class AffectedSharedArrayBufferIssueDetailsView extends AffectedResources
     element.appendChild(status);
   }
 
-  private appendType(element: HTMLElement, type: Protocol.Audits.SharedArrayBufferIssueType): void {
+  #appendType(element: HTMLElement, type: Protocol.Audits.SharedArrayBufferIssueType): void {
     const status = document.createElement('td');
     switch (type) {
       case Protocol.Audits.SharedArrayBufferIssueType.CreationIssue:
@@ -85,7 +85,7 @@ export class AffectedSharedArrayBufferIssueDetailsView extends AffectedResources
     element.appendChild(status);
   }
 
-  private appendDetails(sabIssues: Iterable<IssuesManager.SharedArrayBufferIssue.SharedArrayBufferIssue>): void {
+  #appendDetails(sabIssues: Iterable<IssuesManager.SharedArrayBufferIssue.SharedArrayBufferIssue>): void {
     const header = document.createElement('tr');
     this.appendColumnTitle(header, i18nString(UIStrings.sourceLocation));
     this.appendColumnTitle(header, i18nString(UIStrings.trigger));
@@ -95,26 +95,26 @@ export class AffectedSharedArrayBufferIssueDetailsView extends AffectedResources
     let count = 0;
     for (const sabIssue of sabIssues) {
       count++;
-      this.appendDetail(sabIssue);
+      this.#appendDetail(sabIssue);
     }
     this.updateAffectedResourceCount(count);
   }
 
-  private appendDetail(sabIssue: IssuesManager.SharedArrayBufferIssue.SharedArrayBufferIssue): void {
+  #appendDetail(sabIssue: IssuesManager.SharedArrayBufferIssue.SharedArrayBufferIssue): void {
     const element = document.createElement('tr');
     element.classList.add('affected-resource-directive');
 
     const sabIssueDetails = sabIssue.details();
     const location = IssuesManager.Issue.toZeroBasedLocation(sabIssueDetails.sourceCodeLocation);
     this.appendSourceLocation(element, location, sabIssue.model()?.getTargetIfNotDisposed());
-    this.appendType(element, sabIssueDetails.type);
-    this.appendStatus(element, sabIssueDetails.isWarning);
+    this.#appendType(element, sabIssueDetails.type);
+    this.#appendStatus(element, sabIssueDetails.isWarning);
 
     this.affectedResources.appendChild(element);
   }
 
   update(): void {
     this.clear();
-    this.appendDetails(this.issue.getSharedArrayBufferIssues());
+    this.#appendDetails(this.issue.getSharedArrayBufferIssues());
   }
 }

package/front_end/panels/issues/AffectedSourcesView.ts

@@ -19,10 +19,10 @@ const str_ = i18n.i18n.registerUIStrings('panels/issues/AffectedSourcesView.ts',
 const i18nString = i18n.i18n.getLocalizedString.bind(undefined, str_);
 
 export class AffectedSourcesView extends AffectedResourcesView {
-  private appendAffectedSources(affectedSources: Iterable<Protocol.Audits.SourceCodeLocation>): void {
+  #appendAffectedSources(affectedSources: Iterable<Protocol.Audits.SourceCodeLocation>): void {
     let count = 0;
     for (const source of affectedSources) {
-      this.appendAffectedSource(source);
+      this.#appendAffectedSource(source);
       count++;
     }
     this.updateAffectedResourceCount(count);
@@ -32,7 +32,7 @@ export class AffectedSourcesView extends AffectedResourcesView {
     return i18nString(UIStrings.nSources, {n: count});
   }
 
-  private appendAffectedSource({url, lineNumber, columnNumber}: Protocol.Audits.SourceCodeLocation): void {
+  #appendAffectedSource({url, lineNumber, columnNumber}: Protocol.Audits.SourceCodeLocation): void {
     const cellElement = document.createElement('td');
     // TODO(chromium:1072331): Check feasibility of plumping through scriptId for `linkifyScriptLocation`
     //                         to support source maps and formatted scripts.
@@ -52,6 +52,6 @@ export class AffectedSourcesView extends AffectedResourcesView {
 
   update(): void {
     this.clear();
-    this.appendAffectedSources(this.issue.sources());
+    this.#appendAffectedSources(this.issue.sources());
   }
 }

package/front_end/panels/issues/AffectedTrustedWebActivityIssueDetailsView.ts

@@ -37,7 +37,7 @@ export class AffectedTrustedWebActivityIssueDetailsView extends AffectedResource
     return i18nString(UIStrings.nResources, {n: count});
   }
 
-  private appendDetail(twaIssue: IssuesManager.TrustedWebActivityIssue.TrustedWebActivityIssue): void {
+  #appendDetail(twaIssue: IssuesManager.TrustedWebActivityIssue.TrustedWebActivityIssue): void {
     const element = document.createElement('tr');
     element.classList.add('affected-resource-row');
 
@@ -56,7 +56,7 @@ export class AffectedTrustedWebActivityIssueDetailsView extends AffectedResource
     this.affectedResources.appendChild(element);
   }
 
-  private appendDetails(twaIssues: Iterable<IssuesManager.TrustedWebActivityIssue.TrustedWebActivityIssue>): void {
+  #appendDetails(twaIssues: Iterable<IssuesManager.TrustedWebActivityIssue.TrustedWebActivityIssue>): void {
     const header = document.createElement('tr');
     if (this.issue.code() === IssuesManager.TrustedWebActivityIssue.httpViolationCode) {
       this.appendColumnTitle(header, i18nString(UIStrings.statusCode));
@@ -72,7 +72,7 @@ export class AffectedTrustedWebActivityIssueDetailsView extends AffectedResource
 
     let count = 0;
     for (const twaIssue of twaIssues) {
-      this.appendDetail(twaIssue);
+      this.#appendDetail(twaIssue);
       count++;
     }
     this.updateAffectedResourceCount(count);
@@ -80,6 +80,6 @@ export class AffectedTrustedWebActivityIssueDetailsView extends AffectedResource
 
   update(): void {
     this.clear();
-    this.appendDetails(this.issue.getTrustedWebActivityIssues());
+    this.#appendDetails(this.issue.getTrustedWebActivityIssues());
   }
 }

package/front_end/panels/issues/AttributionReportingIssueDetailsView.ts

@@ -80,13 +80,13 @@ export class AttributionReportingIssueDetailsView extends AffectedResourcesView
     this.clear();
     const issues = this.issue.getAttributionReportingIssues();
     if (issues.size > 0) {
-      this.appendDetails(issues.values().next().value.code(), issues);
+      this.#appendDetails(issues.values().next().value.code(), issues);
     } else {
       this.updateAffectedResourceCount(0);
     }
   }
 
-  private appendDetails(
+  #appendDetails(
       issueCode: IssuesManager.AttributionReportingIssue.IssueCode,
       issues: Iterable<IssuesManager.AttributionReportingIssue.AttributionReportingIssue>): void {
     const header = document.createElement('tr');
@@ -156,12 +156,12 @@ export class AttributionReportingIssueDetailsView extends AffectedResourcesView
     let count = 0;
     for (const issue of issues) {
       count++;
-      this.appendDetail(issueCode, issue);
+      this.#appendDetail(issueCode, issue);
     }
     this.updateAffectedResourceCount(count);
   }
 
-  private async appendDetail(
+  async #appendDetail(
       issueCode: IssuesManager.AttributionReportingIssue.IssueCode,
       issue: IssuesManager.AttributionReportingIssue.AttributionReportingIssue): Promise<void> {
     const element = document.createElement('tr');
@@ -171,12 +171,12 @@ export class AttributionReportingIssueDetailsView extends AffectedResourcesView
 
     switch (issueCode) {
       case IssuesManager.AttributionReportingIssue.IssueCode.AttributionUntrustworthyFrameOrigin:
-        this.appendFrameOrEmptyCell(element, issue);
-        this.appendRequestOrEmptyCell(element, details.request);
+        this.#appendFrameOrEmptyCell(element, issue);
+        this.#appendRequestOrEmptyCell(element, details.request);
         this.appendIssueDetailCell(element, details.invalidParameter || '');
         break;
       case IssuesManager.AttributionReportingIssue.IssueCode.AttributionSourceUntrustworthyOrigin:
-        await this.appendElementOrEmptyCell(element, issue);
+        await this.#appendElementOrEmptyCell(element, issue);
         this.appendIssueDetailCell(element, details.invalidParameter || '');
         break;
       case IssuesManager.AttributionReportingIssue.IssueCode.AttributionTriggerDataTooLarge:
@@ -186,31 +186,31 @@ export class AttributionReportingIssueDetailsView extends AffectedResourcesView
       case IssuesManager.AttributionReportingIssue.IssueCode.InvalidEventSourceTriggerData:
       case IssuesManager.AttributionReportingIssue.IssueCode.InvalidTriggerPriority:
       case IssuesManager.AttributionReportingIssue.IssueCode.InvalidTriggerDedupKey:
-        this.appendRequestOrEmptyCell(element, details.request);
+        this.#appendRequestOrEmptyCell(element, details.request);
         this.appendIssueDetailCell(element, details.invalidParameter || '');
         break;
       case IssuesManager.AttributionReportingIssue.IssueCode.AttributionSourceUntrustworthyFrameOrigin:
       case IssuesManager.AttributionReportingIssue.IssueCode.InvalidAttributionSourceEventId:
       case IssuesManager.AttributionReportingIssue.IssueCode.InvalidAttributionSourceExpiry:
       case IssuesManager.AttributionReportingIssue.IssueCode.InvalidAttributionSourcePriority:
-        this.appendFrameOrEmptyCell(element, issue);
-        await this.appendElementOrEmptyCell(element, issue);
+        this.#appendFrameOrEmptyCell(element, issue);
+        await this.#appendElementOrEmptyCell(element, issue);
         this.appendIssueDetailCell(element, details.invalidParameter || '');
         break;
       case IssuesManager.AttributionReportingIssue.IssueCode.MissingAttributionData:
-        this.appendRequestOrEmptyCell(element, details.request);
+        this.#appendRequestOrEmptyCell(element, details.request);
         break;
       case IssuesManager.AttributionReportingIssue.IssueCode.PermissionPolicyDisabled:
-        this.appendFrameOrEmptyCell(element, issue);
-        await this.appendElementOrEmptyCell(element, issue);
-        this.appendRequestOrEmptyCell(element, details.request);
+        this.#appendFrameOrEmptyCell(element, issue);
+        await this.#appendElementOrEmptyCell(element, issue);
+        this.#appendRequestOrEmptyCell(element, details.request);
         break;
     }
 
     this.affectedResources.appendChild(element);
   }
 
-  private appendFrameOrEmptyCell(
+  #appendFrameOrEmptyCell(
       parent: HTMLElement, issue: IssuesManager.AttributionReportingIssue.AttributionReportingIssue): void {
     const details = issue.issueDetails;
     if (details.frame) {
@@ -220,7 +220,7 @@ export class AttributionReportingIssueDetailsView extends AffectedResourcesView
     }
   }
 
-  private async appendElementOrEmptyCell(
+  async #appendElementOrEmptyCell(
       parent: HTMLElement, issue: IssuesManager.AttributionReportingIssue.AttributionReportingIssue): Promise<void> {
     const details = issue.issueDetails;
     if (details.violatingNodeId !== undefined) {
@@ -233,7 +233,7 @@ export class AttributionReportingIssueDetailsView extends AffectedResourcesView
     }
   }
 
-  private appendRequestOrEmptyCell(parent: HTMLElement, request?: Protocol.Audits.AffectedRequest): void {
+  #appendRequestOrEmptyCell(parent: HTMLElement, request?: Protocol.Audits.AffectedRequest): void {
     if (!request) {
       this.appendIssueDetailCell(parent, '');
       return;

package/front_end/panels/issues/CSPViolationsListView.ts

@@ -11,15 +11,15 @@ import * as LitHtml from '../../ui/lit-html/lit-html.js';
 import cspViolationsListViewStyles from './cspViolationsListView.css.js';
 
 export class CSPViolationsListView extends UI.Widget.VBox {
-  private table = new DataGrid.DataGridController.DataGridController();
-  private categoryFilter = new Set<string>();
-  private issueRows =
+  #table = new DataGrid.DataGridController.DataGridController();
+  #categoryFilter = new Set<string>();
+  #issueRows =
       new Map<IssuesManager.ContentSecurityPolicyIssue.ContentSecurityPolicyIssue, DataGrid.DataGridUtils.Row>();
 
   constructor() {
     super(true);
 
-    this.table.data = {
+    this.#table.data = {
       columns: [
         {id: 'sourceCode', title: 'Source Code', sortable: false, widthWeighting: 1, visible: true, hideable: false},
         {
@@ -35,34 +35,33 @@ export class CSPViolationsListView extends UI.Widget.VBox {
       ],
       rows: [],
     };
-    this.contentElement.appendChild(this.table);
+    this.contentElement.appendChild(this.#table);
   }
 
   updateTextFilter(filter: string): void {
     if (filter.length === 0) {
-      this.table.data = {...this.table.data, filters: []};
+      this.#table.data = {...this.#table.data, filters: []};
     } else {
-      this.table.data = {
-        ...this.table.data,
+      this.#table.data = {
+        ...this.#table.data,
         filters: [{text: filter, key: undefined, regex: undefined, negative: false}],
       };
     }
   }
 
   updateCategoryFilter(categories: Set<string>): void {
-    this.categoryFilter = categories;
+    this.#categoryFilter = categories;
     const rows = [];
-    for (const [issue, row] of this.issueRows.entries()) {
-      if (this.isIssueInFilterCategories(issue)) {
+    for (const [issue, row] of this.#issueRows.entries()) {
+      if (this.#isIssueInFilterCategories(issue)) {
         rows.push(row);
       }
     }
-    this.table.data = {...this.table.data, rows: rows};
+    this.#table.data = {...this.#table.data, rows: rows};
   }
 
-  private isIssueInFilterCategories(issue: IssuesManager.ContentSecurityPolicyIssue.ContentSecurityPolicyIssue):
-      boolean {
-    return (this.categoryFilter.has(issue.code()) || this.categoryFilter.size === 0);
+  #isIssueInFilterCategories(issue: IssuesManager.ContentSecurityPolicyIssue.ContentSecurityPolicyIssue): boolean {
+    return (this.#categoryFilter.has(issue.code()) || this.#categoryFilter.size === 0);
   }
 
   addIssue(issue: IssuesManager.ContentSecurityPolicyIssue.ContentSecurityPolicyIssue): void {
@@ -71,7 +70,7 @@ export class CSPViolationsListView extends UI.Widget.VBox {
       return;
     }
     const status = issue.details().isReportOnly ? 'report-only' : 'blocked';
-    const category = this.issueViolationCodeToCategoryName(issue.code());
+    const category = this.#issueViolationCodeToCategoryName(issue.code());
     const newIssue = {
       cells: [
         {
@@ -87,20 +86,20 @@ export class CSPViolationsListView extends UI.Widget.VBox {
         {columnId: 'status', value: status},
       ],
     };
-    this.issueRows.set(issue, newIssue);
+    this.#issueRows.set(issue, newIssue);
 
-    if (this.isIssueInFilterCategories(issue)) {
-      this.table.data.rows.push(newIssue);
-      this.table.data = {...this.table.data};
+    if (this.#isIssueInFilterCategories(issue)) {
+      this.#table.data.rows.push(newIssue);
+      this.#table.data = {...this.#table.data};
     }
   }
 
   clearIssues(): void {
-    this.issueRows.clear();
-    this.table.data = {...this.table.data, rows: []};
+    this.#issueRows.clear();
+    this.#table.data = {...this.#table.data, rows: []};
   }
 
-  private issueViolationCodeToCategoryName(code: string): string {
+  #issueViolationCodeToCategoryName(code: string): string {
     if (code === IssuesManager.ContentSecurityPolicyIssue.inlineViolationCode) {
       return 'Inline Violation';
     }

package/front_end/panels/issues/CSPViolationsView.ts

@@ -23,8 +23,8 @@ const str_ = i18n.i18n.registerUIStrings('panels/issues/CSPViolationsView.ts', U
 const i18nString = i18n.i18n.getLocalizedString.bind(undefined, str_);
 let cspViolationsViewInstance: CSPViolationsView;
 export class CSPViolationsView extends UI.Widget.VBox {
-  private listView = new CSPViolationsListView();
-  private issuesManager = IssuesManager.IssuesManager.IssuesManager.instance();
+  #listView = new CSPViolationsListView();
+  #issuesManager = IssuesManager.IssuesManager.IssuesManager.instance();
 
   /**
    * @private
@@ -37,7 +37,7 @@ export class CSPViolationsView extends UI.Widget.VBox {
     const topToolbar = new UI.Toolbar.Toolbar('csp-violations-toolbar', this.contentElement);
     const textFilterUI = new UI.Toolbar.ToolbarInput(i18nString(UIStrings.filter), '', 1, .2, '');
     textFilterUI.addEventListener(UI.Toolbar.ToolbarInput.Event.TextChanged, () => {
-      this.listView.updateTextFilter(textFilterUI.value());
+      this.#listView.updateTextFilter(textFilterUI.value());
     });
     topToolbar.appendToolbarItem(textFilterUI);
 
@@ -55,16 +55,16 @@ export class CSPViolationsView extends UI.Widget.VBox {
     });
     levelMenuButton.setOnOptionClicked(() => {
       const categories = new Set(levelMenuButton.getOptions().filter(x => x.enabled).map(x => x.value));
-      this.listView.updateCategoryFilter(categories);
+      this.#listView.updateCategoryFilter(categories);
     });
     topToolbar.appendToolbarItem(levelMenuButton);
-    this.listView.show(this.contentElement);
+    this.#listView.show(this.contentElement);
 
-    this.issuesManager.addEventListener(IssuesManager.IssuesManager.Events.IssueAdded, this.onIssueAdded, this);
-    this.issuesManager.addEventListener(
-        IssuesManager.IssuesManager.Events.FullUpdateRequired, this.onFullUpdateRequired, this);
+    this.#issuesManager.addEventListener(IssuesManager.IssuesManager.Events.IssueAdded, this.#onIssueAdded, this);
+    this.#issuesManager.addEventListener(
+        IssuesManager.IssuesManager.Events.FullUpdateRequired, this.#onFullUpdateRequired, this);
 
-    this.addAllIssues();
+    this.#addAllIssues();
   }
 
   static instance(opts = {forceNew: null}): CSPViolationsView {
@@ -76,22 +76,22 @@ export class CSPViolationsView extends UI.Widget.VBox {
     return cspViolationsViewInstance;
   }
 
-  private onIssueAdded(event: Common.EventTarget.EventTargetEvent<IssuesManager.IssuesManager.IssueAddedEvent>): void {
+  #onIssueAdded(event: Common.EventTarget.EventTargetEvent<IssuesManager.IssuesManager.IssueAddedEvent>): void {
     const {issue} = event.data;
     if (issue instanceof IssuesManager.ContentSecurityPolicyIssue.ContentSecurityPolicyIssue) {
-      this.listView.addIssue(issue);
+      this.#listView.addIssue(issue);
     }
   }
 
-  private onFullUpdateRequired(): void {
-    this.listView.clearIssues();
-    this.addAllIssues();
+  #onFullUpdateRequired(): void {
+    this.#listView.clearIssues();
+    this.#addAllIssues();
   }
 
-  private addAllIssues(): void {
-    for (const issue of this.issuesManager.issues()) {
+  #addAllIssues(): void {
+    for (const issue of this.#issuesManager.issues()) {
       if (issue instanceof IssuesManager.ContentSecurityPolicyIssue.ContentSecurityPolicyIssue) {
-        this.listView.addIssue(issue);
+        this.#listView.addIssue(issue);
       }
     }
   }

package/front_end/panels/issues/ComboBoxOfCheckBoxes.ts

@@ -6,52 +6,52 @@ import type * as Common from '../../core/common/common.js';
 import * as UI from '../../ui/legacy/legacy.js';
 
 export class ComboBoxOfCheckBoxes extends UI.Toolbar.ToolbarButton {
-  private options = new Array<MenuOption>();
-  private headers = new Array<MenuHeader>();
-  private onOptionClicked = (): void => {};
+  #options = new Array<MenuOption>();
+  #headers = new Array<MenuHeader>();
+  #onOptionClicked = (): void => {};
   constructor(title: string) {
     super(title);
     this.turnIntoSelect();
-    this.addEventListener(UI.Toolbar.ToolbarButton.Events.Click, this.showLevelContextMenu.bind(this));
+    this.addEventListener(UI.Toolbar.ToolbarButton.Events.Click, this.#showLevelContextMenu.bind(this));
     UI.ARIAUtils.markAsMenuButton(this.element);
   }
 
   addOption(option: string, value: string, defaultEnabled: boolean): void {
-    this.options.push({'title': option, 'value': value, default: defaultEnabled, 'enabled': defaultEnabled});
+    this.#options.push({'title': option, 'value': value, default: defaultEnabled, 'enabled': defaultEnabled});
   }
 
   setOptionEnabled(index: number, enabled: boolean): void {
-    const option = this.options[index];
+    const option = this.#options[index];
     if (!option) {
       return;
     }
     option.enabled = enabled;
-    this.onOptionClicked();
+    this.#onOptionClicked();
   }
 
   addHeader(headerName: string, callback: (() => void)): void {
-    this.headers.push({title: headerName, callback: callback});
+    this.#headers.push({title: headerName, callback: callback});
   }
 
   setOnOptionClicked(onOptionClicked: (() => void)): void {
-    this.onOptionClicked = onOptionClicked;
+    this.#onOptionClicked = onOptionClicked;
   }
 
   getOptions(): Array<MenuOption> {
-    return this.options;
+    return this.#options;
   }
 
-  private showLevelContextMenu({data: mouseEvent}: Common.EventTarget.EventTargetEvent<Event>): void {
+  #showLevelContextMenu({data: mouseEvent}: Common.EventTarget.EventTargetEvent<Event>): void {
     const contextMenu = new UI.ContextMenu.ContextMenu(mouseEvent, {
       useSoftMenu: true,
       x: this.element.totalOffsetLeft(),
       y: this.element.totalOffsetTop() + this.element.offsetHeight,
     });
 
-    for (const {title, callback} of this.headers) {
+    for (const {title, callback} of this.#headers) {
       contextMenu.headerSection().appendCheckboxItem(title, () => callback());
     }
-    for (const [index, {title, enabled}] of this.options.entries()) {
+    for (const [index, {title, enabled}] of this.#options.entries()) {
       contextMenu.defaultSection().appendCheckboxItem(title, () => {
         this.setOptionEnabled(index, !enabled);
       }, enabled);