chrome-devtools-frontend 1.0.950850 → 1.0.952284

package/front_end/panels/issues/CorsIssueDetailsView.ts

@@ -133,7 +133,7 @@ export class CorsIssueDetailsView extends AffectedResourcesView {
     this.affectedResourcesCountElement.classList.add('cors-issue-affected-resource-label');
   }
 
-  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');
@@ -149,8 +149,8 @@ export class CorsIssueDetailsView extends AffectedResourcesView {
     return i18nString(UIStrings.nRequests, {n: count});
   }
 
-  private appendDetails(
-      issueCode: IssuesManager.CorsIssue.IssueCode, issues: Iterable<IssuesManager.CorsIssue.CorsIssue>): void {
+  #appendDetails(issueCode: IssuesManager.CorsIssue.IssueCode, issues: Iterable<IssuesManager.CorsIssue.CorsIssue>):
+      void {
     const header = document.createElement('tr');
     this.appendColumnTitle(header, i18nString(UIStrings.request));
     this.appendColumnTitle(header, i18nString(UIStrings.status));
@@ -222,12 +222,12 @@ export class CorsIssueDetailsView extends AffectedResourcesView {
     let count = 0;
     for (const issue of issues) {
       count++;
-      this.appendDetail(issueCode, issue);
+      this.#appendDetail(issueCode, issue);
     }
     this.updateAffectedResourceCount(count);
   }
 
-  private appendSecureContextCell(element: HTMLElement, isSecureContext: boolean|undefined): void {
+  #appendSecureContextCell(element: HTMLElement, isSecureContext: boolean|undefined): void {
     if (isSecureContext === undefined) {
       this.appendIssueDetailCell(element, '');
       return;
@@ -292,7 +292,7 @@ export class CorsIssueDetailsView extends AffectedResourcesView {
     throw new Error('Invalid Argument');
   }
 
-  private appendDetail(issueCode: IssuesManager.CorsIssue.IssueCode, issue: IssuesManager.CorsIssue.CorsIssue): void {
+  #appendDetail(issueCode: IssuesManager.CorsIssue.IssueCode, issue: IssuesManager.CorsIssue.CorsIssue): void {
     const element = document.createElement('tr');
     element.classList.add('affected-resource-directive');
 
@@ -314,7 +314,7 @@ export class CorsIssueDetailsView extends AffectedResourcesView {
     switch (issueCode) {
       case IssuesManager.CorsIssue.IssueCode.InvalidHeaderValues:
         element.appendChild(this.createRequestCell(details.request, opts));
-        this.appendStatus(element, details.isWarning);
+        this.#appendStatus(element, details.isWarning);
         if (corsError.includes('Preflight')) {
           element.appendChild(
               this.createRequestCell(details.request, {...opts, linkToPreflight: true, highlightHeader}));
@@ -327,7 +327,7 @@ export class CorsIssueDetailsView extends AffectedResourcesView {
         break;
       case IssuesManager.CorsIssue.IssueCode.WildcardOriginNotAllowed:
         element.appendChild(this.createRequestCell(details.request, opts));
-        this.appendStatus(element, details.isWarning);
+        this.#appendStatus(element, details.isWarning);
         if (corsError.includes('Preflight')) {
           element.appendChild(
               this.createRequestCell(details.request, {...opts, linkToPreflight: true, highlightHeader}));
@@ -337,7 +337,7 @@ export class CorsIssueDetailsView extends AffectedResourcesView {
         break;
       case IssuesManager.CorsIssue.IssueCode.PreflightResponseInvalid: {
         element.appendChild(this.createRequestCell(details.request, opts));
-        this.appendStatus(element, details.isWarning);
+        this.#appendStatus(element, details.isWarning);
         const specialHighlightHeader = corsError === Protocol.Network.CorsError.PreflightInvalidStatus ?
             {
               section: NetworkForward.UIRequestLocation.UIHeaderSection.General,
@@ -351,7 +351,7 @@ export class CorsIssueDetailsView extends AffectedResourcesView {
       }
       case IssuesManager.CorsIssue.IssueCode.OriginMismatch:
         element.appendChild(this.createRequestCell(details.request, opts));
-        this.appendStatus(element, details.isWarning);
+        this.#appendStatus(element, details.isWarning);
         if (corsError.includes('Preflight')) {
           element.appendChild(
               this.createRequestCell(details.request, {...opts, linkToPreflight: true, highlightHeader}));
@@ -363,7 +363,7 @@ export class CorsIssueDetailsView extends AffectedResourcesView {
         break;
       case IssuesManager.CorsIssue.IssueCode.AllowCredentialsRequired:
         element.appendChild(this.createRequestCell(details.request, opts));
-        this.appendStatus(element, details.isWarning);
+        this.#appendStatus(element, details.isWarning);
         if (corsError.includes('Preflight')) {
           element.appendChild(
               this.createRequestCell(details.request, {...opts, linkToPreflight: true, highlightHeader}));
@@ -374,23 +374,23 @@ export class CorsIssueDetailsView extends AffectedResourcesView {
         break;
       case IssuesManager.CorsIssue.IssueCode.InsecurePrivateNetwork:
         element.appendChild(this.createRequestCell(details.request, opts));
-        this.appendStatus(element, details.isWarning);
+        this.#appendStatus(element, details.isWarning);
         this.appendIssueDetailCell(element, details.resourceIPAddressSpace ?? '');
         this.appendIssueDetailCell(element, details.clientSecurityState?.initiatorIPAddressSpace ?? '');
-        this.appendSecureContextCell(element, details.clientSecurityState?.initiatorIsSecureContext);
+        this.#appendSecureContextCell(element, details.clientSecurityState?.initiatorIsSecureContext);
         break;
       case IssuesManager.CorsIssue.IssueCode.PreflightAllowPrivateNetworkError: {
         element.appendChild(this.createRequestCell(details.request, opts));
-        this.appendStatus(element, details.isWarning);
+        this.#appendStatus(element, details.isWarning);
         element.appendChild(this.createRequestCell(details.request, {...opts, linkToPreflight: true, highlightHeader}));
         this.appendIssueDetailCell(element, details.corsErrorStatus.failedParameter, 'code-example');
         this.appendIssueDetailCell(element, details.clientSecurityState?.initiatorIPAddressSpace ?? '');
-        this.appendSecureContextCell(element, details.clientSecurityState?.initiatorIsSecureContext);
+        this.#appendSecureContextCell(element, details.clientSecurityState?.initiatorIsSecureContext);
         break;
       }
       case IssuesManager.CorsIssue.IssueCode.MethodDisallowedByPreflightResponse:
         element.appendChild(this.createRequestCell(details.request, opts));
-        this.appendStatus(element, details.isWarning);
+        this.#appendStatus(element, details.isWarning);
         element.appendChild(this.createRequestCell(details.request, {...opts, linkToPreflight: true, highlightHeader}));
         this.appendIssueDetailCell(element, details.corsErrorStatus.failedParameter, 'code-example');
         break;
@@ -402,7 +402,7 @@ export class CorsIssueDetailsView extends AffectedResourcesView {
             name: corsErrorStatus.failedParameter,
           },
         }));
-        this.appendStatus(element, details.isWarning);
+        this.#appendStatus(element, details.isWarning);
         element.appendChild(this.createRequestCell(details.request, {
           ...opts,
           linkToPreflight: true,
@@ -421,11 +421,11 @@ export class CorsIssueDetailsView extends AffectedResourcesView {
             name: CorsIssueDetailsView.getHeaderFromError(corsError),
           },
         }));
-        this.appendStatus(element, details.isWarning);
+        this.#appendStatus(element, details.isWarning);
         break;
       case IssuesManager.CorsIssue.IssueCode.DisallowedByMode:
         element.appendChild(this.createRequestCell(details.request, opts));
-        this.appendStatus(element, details.isWarning);
+        this.#appendStatus(element, details.isWarning);
         this.appendIssueDetailCell(element, details.initiatorOrigin ?? '', 'code-example');
         this.appendSourceLocation(element, details.location, issue.model()?.getTargetIfNotDisposed());
         break;
@@ -437,19 +437,19 @@ export class CorsIssueDetailsView extends AffectedResourcesView {
             name: CorsIssueDetailsView.getHeaderFromError(corsError),
           },
         }));
-        this.appendStatus(element, details.isWarning);
+        this.#appendStatus(element, details.isWarning);
         this.appendIssueDetailCell(element, details.initiatorOrigin ?? '', 'code-example');
         this.appendSourceLocation(element, details.location, issue.model()?.getTargetIfNotDisposed());
         this.appendIssueDetailCell(element, details.corsErrorStatus.failedParameter ?? '', 'code-example');
         break;
       case IssuesManager.CorsIssue.IssueCode.NoCorsRedirectModeNotFollow:
         element.appendChild(this.createRequestCell(details.request, opts));
-        this.appendStatus(element, details.isWarning);
+        this.#appendStatus(element, details.isWarning);
         this.appendSourceLocation(element, details.location, issue.model()?.getTargetIfNotDisposed());
         break;
       default:
         element.appendChild(this.createRequestCell(details.request, opts));
-        this.appendStatus(element, details.isWarning);
+        this.#appendStatus(element, details.isWarning);
         Platform.assertUnhandled<IssuesManager.CorsIssue.IssueCode.PreflightMissingAllowExternal|
                                  IssuesManager.CorsIssue.IssueCode.PreflightInvalidAllowExternal|
                                  IssuesManager.CorsIssue.IssueCode.InvalidPrivateNetworkAccess|
@@ -464,7 +464,7 @@ export class CorsIssueDetailsView extends AffectedResourcesView {
     this.clear();
     const issues = this.issue.getCorsIssues();
     if (issues.size > 0) {
-      this.appendDetails(issues.values().next().value.code(), issues);
+      this.#appendDetails(issues.values().next().value.code(), issues);
     } else {
       this.updateAffectedResourceCount(0);
     }

package/front_end/panels/issues/GenericIssueDetailsView.ts

@@ -27,7 +27,7 @@ export class GenericIssueDetailsView extends AffectedResourcesView {
     return i18nString(UIStrings.nResources, {n: count});
   }
 
-  private appendDetails(genericIssues: ReadonlySet<IssuesManager.GenericIssue.GenericIssue>): void {
+  #appendDetails(genericIssues: ReadonlySet<IssuesManager.GenericIssue.GenericIssue>): void {
     const header = document.createElement('tr');
 
     const sampleIssueDetails = genericIssues.values().next().value.details();
@@ -39,12 +39,12 @@ export class GenericIssueDetailsView extends AffectedResourcesView {
     let count = 0;
     for (const genericIssue of genericIssues) {
       count++;
-      this.appendDetail(genericIssue);
+      this.#appendDetail(genericIssue);
     }
     this.updateAffectedResourceCount(count);
   }
 
-  private appendDetail(genericIssue: IssuesManager.GenericIssue.GenericIssue): void {
+  #appendDetail(genericIssue: IssuesManager.GenericIssue.GenericIssue): void {
     const element = document.createElement('tr');
     element.classList.add('affected-resource-directive');
 
@@ -60,7 +60,7 @@ export class GenericIssueDetailsView extends AffectedResourcesView {
     this.clear();
     const issues = this.issue.getGenericIssues();
     if (issues.size > 0) {
-      this.appendDetails(issues);
+      this.#appendDetails(issues);
     } else {
       this.updateAffectedResourceCount(0);
     }

package/front_end/panels/issues/HiddenIssuesRow.ts

@@ -22,18 +22,18 @@ const str_ = i18n.i18n.registerUIStrings('panels/issues/HiddenIssuesRow.ts', UIS
 const i18nString = i18n.i18n.getLocalizedString.bind(undefined, str_);
 
 export class HiddenIssuesRow extends UI.TreeOutline.TreeElement {
-  private numHiddenAggregatedIssues: HTMLElement;
+  #numHiddenAggregatedIssues: HTMLElement;
 
   constructor() {
     super(undefined, true);
-    this.numHiddenAggregatedIssues = document.createElement('span');
+    this.#numHiddenAggregatedIssues = document.createElement('span');
     this.toggleOnClick = true;
     this.listItemElement.classList.add('issue-category', 'hidden-issues');
     this.childrenListElement.classList.add('hidden-issues-body');
-    this.appendHeader();
+    this.#appendHeader();
   }
 
-  private appendHeader(): void {
+  #appendHeader(): void {
     const unhideAllIssuesBtn = UI.UIUtils.createTextButton(
         i18nString(UIStrings.unhideAll),
         (): void => IssuesManager.IssuesManager.IssuesManager.instance().unhideAllIssues(),
@@ -42,10 +42,10 @@ export class HiddenIssuesRow extends UI.TreeOutline.TreeElement {
     const countAdorner = new Adorners.Adorner.Adorner();
     countAdorner.data = {
       name: 'countWrapper',
-      content: this.numHiddenAggregatedIssues,
+      content: this.#numHiddenAggregatedIssues,
     };
     countAdorner.classList.add('aggregated-issues-count');
-    this.numHiddenAggregatedIssues.textContent = '0';
+    this.#numHiddenAggregatedIssues.textContent = '0';
     const header = document.createElement('div');
     const title = document.createElement('div');
     header.classList.add('header');
@@ -58,6 +58,6 @@ export class HiddenIssuesRow extends UI.TreeOutline.TreeElement {
   }
 
   update(count: number): void {
-    this.numHiddenAggregatedIssues.textContent = `${count}`;
+    this.#numHiddenAggregatedIssues.textContent = `${count}`;
   }
 }

package/front_end/panels/issues/IssueAggregator.ts

@@ -25,34 +25,34 @@ export type AggregationKey = {
  * of all resources that are affected by the aggregated issues.
  */
 export class AggregatedIssue extends IssuesManager.Issue.Issue {
-  private affectedCookies = new Map<string, {
+  #affectedCookies = new Map<string, {
     cookie: Protocol.Audits.AffectedCookie,
     hasRequest: boolean,
   }>();
-  private affectedRawCookieLines = new Map<string, {rawCookieLine: string, hasRequest: boolean}>();
-  private affectedRequests = new Map<string, Protocol.Audits.AffectedRequest>();
-  private affectedLocations = new Map<string, Protocol.Audits.SourceCodeLocation>();
-  private heavyAdIssues = new Set<IssuesManager.HeavyAdIssue.HeavyAdIssue>();
-  private blockedByResponseDetails = new Map<string, Protocol.Audits.BlockedByResponseIssueDetails>();
-  private corsIssues = new Set<IssuesManager.CorsIssue.CorsIssue>();
-  private cspIssues = new Set<IssuesManager.ContentSecurityPolicyIssue.ContentSecurityPolicyIssue>();
-  private issueKind = IssuesManager.Issue.IssueKind.Improvement;
-  private lowContrastIssues = new Set<IssuesManager.LowTextContrastIssue.LowTextContrastIssue>();
-  private mixedContentIssues = new Set<IssuesManager.MixedContentIssue.MixedContentIssue>();
-  private sharedArrayBufferIssues = new Set<IssuesManager.SharedArrayBufferIssue.SharedArrayBufferIssue>();
-  private trustedWebActivityIssues = new Set<IssuesManager.TrustedWebActivityIssue.TrustedWebActivityIssue>();
-  private quirksModeIssues = new Set<IssuesManager.QuirksModeIssue.QuirksModeIssue>();
-  private attributionReportingIssues = new Set<IssuesManager.AttributionReportingIssue.AttributionReportingIssue>();
-  private wasmCrossOriginModuleSharingIssues =
+  #affectedRawCookieLines = new Map<string, {rawCookieLine: string, hasRequest: boolean}>();
+  #affectedRequests = new Map<string, Protocol.Audits.AffectedRequest>();
+  #affectedLocations = new Map<string, Protocol.Audits.SourceCodeLocation>();
+  #heavyAdIssues = new Set<IssuesManager.HeavyAdIssue.HeavyAdIssue>();
+  #blockedByResponseDetails = new Map<string, Protocol.Audits.BlockedByResponseIssueDetails>();
+  #corsIssues = new Set<IssuesManager.CorsIssue.CorsIssue>();
+  #cspIssues = new Set<IssuesManager.ContentSecurityPolicyIssue.ContentSecurityPolicyIssue>();
+  #issueKind = IssuesManager.Issue.IssueKind.Improvement;
+  #lowContrastIssues = new Set<IssuesManager.LowTextContrastIssue.LowTextContrastIssue>();
+  #mixedContentIssues = new Set<IssuesManager.MixedContentIssue.MixedContentIssue>();
+  #sharedArrayBufferIssues = new Set<IssuesManager.SharedArrayBufferIssue.SharedArrayBufferIssue>();
+  #trustedWebActivityIssues = new Set<IssuesManager.TrustedWebActivityIssue.TrustedWebActivityIssue>();
+  #quirksModeIssues = new Set<IssuesManager.QuirksModeIssue.QuirksModeIssue>();
+  #attributionReportingIssues = new Set<IssuesManager.AttributionReportingIssue.AttributionReportingIssue>();
+  #wasmCrossOriginModuleSharingIssues =
       new Set<IssuesManager.WasmCrossOriginModuleSharingIssue.WasmCrossOriginModuleSharingIssue>();
-  private genericIssues = new Set<IssuesManager.GenericIssue.GenericIssue>();
-  private representative?: IssuesManager.Issue.Issue;
-  private aggregatedIssuesCount = 0;
-  private key: AggregationKey;
+  #genericIssues = new Set<IssuesManager.GenericIssue.GenericIssue>();
+  #representative?: IssuesManager.Issue.Issue;
+  #aggregatedIssuesCount = 0;
+  #key: AggregationKey;
 
   constructor(code: string, aggregationKey: AggregationKey) {
     super(code);
-    this.key = aggregationKey;
+    this.#key = aggregationKey;
   }
 
   override primaryKey(): string {
@@ -60,183 +60,183 @@ export class AggregatedIssue extends IssuesManager.Issue.Issue {
   }
 
   aggregationKey(): AggregationKey {
-    return this.key;
+    return this.#key;
   }
 
   getBlockedByResponseDetails(): Iterable<Protocol.Audits.BlockedByResponseIssueDetails> {
-    return this.blockedByResponseDetails.values();
+    return this.#blockedByResponseDetails.values();
   }
 
   cookies(): Iterable<Protocol.Audits.AffectedCookie> {
-    return Array.from(this.affectedCookies.values()).map(x => x.cookie);
+    return Array.from(this.#affectedCookies.values()).map(x => x.cookie);
   }
 
   getRawCookieLines(): Iterable<{rawCookieLine: string, hasRequest: boolean}> {
-    return this.affectedRawCookieLines.values();
+    return this.#affectedRawCookieLines.values();
   }
 
   sources(): Iterable<Protocol.Audits.SourceCodeLocation> {
-    return this.affectedLocations.values();
+    return this.#affectedLocations.values();
   }
 
   cookiesWithRequestIndicator(): Iterable<{
     cookie: Protocol.Audits.AffectedCookie,
     hasRequest: boolean,
   }> {
-    return this.affectedCookies.values();
+    return this.#affectedCookies.values();
   }
 
   getHeavyAdIssues(): Iterable<IssuesManager.HeavyAdIssue.HeavyAdIssue> {
-    return this.heavyAdIssues;
+    return this.#heavyAdIssues;
   }
 
   getMixedContentIssues(): Iterable<IssuesManager.MixedContentIssue.MixedContentIssue> {
-    return this.mixedContentIssues;
+    return this.#mixedContentIssues;
   }
 
   getTrustedWebActivityIssues(): Iterable<IssuesManager.TrustedWebActivityIssue.TrustedWebActivityIssue> {
-    return this.trustedWebActivityIssues;
+    return this.#trustedWebActivityIssues;
   }
 
   getCorsIssues(): Set<IssuesManager.CorsIssue.CorsIssue> {
-    return this.corsIssues;
+    return this.#corsIssues;
   }
 
   getCspIssues(): Iterable<IssuesManager.ContentSecurityPolicyIssue.ContentSecurityPolicyIssue> {
-    return this.cspIssues;
+    return this.#cspIssues;
   }
 
   getLowContrastIssues(): Iterable<IssuesManager.LowTextContrastIssue.LowTextContrastIssue> {
-    return this.lowContrastIssues;
+    return this.#lowContrastIssues;
   }
 
   requests(): Iterable<Protocol.Audits.AffectedRequest> {
-    return this.affectedRequests.values();
+    return this.#affectedRequests.values();
   }
 
   getSharedArrayBufferIssues(): Iterable<IssuesManager.SharedArrayBufferIssue.SharedArrayBufferIssue> {
-    return this.sharedArrayBufferIssues;
+    return this.#sharedArrayBufferIssues;
   }
 
   getQuirksModeIssues(): Iterable<IssuesManager.QuirksModeIssue.QuirksModeIssue> {
-    return this.quirksModeIssues;
+    return this.#quirksModeIssues;
   }
 
   getAttributionReportingIssues(): ReadonlySet<IssuesManager.AttributionReportingIssue.AttributionReportingIssue> {
-    return this.attributionReportingIssues;
+    return this.#attributionReportingIssues;
   }
 
   getWasmCrossOriginModuleSharingIssue():
       ReadonlySet<IssuesManager.WasmCrossOriginModuleSharingIssue.WasmCrossOriginModuleSharingIssue> {
-    return this.wasmCrossOriginModuleSharingIssues;
+    return this.#wasmCrossOriginModuleSharingIssues;
   }
 
   getGenericIssues(): ReadonlySet<IssuesManager.GenericIssue.GenericIssue> {
-    return this.genericIssues;
+    return this.#genericIssues;
   }
 
   getDescription(): IssuesManager.MarkdownIssueDescription.MarkdownIssueDescription|null {
-    if (this.representative) {
-      return this.representative.getDescription();
+    if (this.#representative) {
+      return this.#representative.getDescription();
     }
     return null;
   }
 
   getCategory(): IssuesManager.Issue.IssueCategory {
-    if (this.representative) {
-      return this.representative.getCategory();
+    if (this.#representative) {
+      return this.#representative.getCategory();
     }
     return IssuesManager.Issue.IssueCategory.Other;
   }
 
   getAggregatedIssuesCount(): number {
-    return this.aggregatedIssuesCount;
+    return this.#aggregatedIssuesCount;
   }
 
   /**
    * Produces a primary key for a cookie. Use this instead of `JSON.stringify` in
    * case new fields are added to `AffectedCookie`.
    */
-  private keyForCookie(cookie: Protocol.Audits.AffectedCookie): string {
+  #keyForCookie(cookie: Protocol.Audits.AffectedCookie): string {
     const {domain, path, name} = cookie;
     return `${domain};${path};${name}`;
   }
 
   addInstance(issue: IssuesManager.Issue.Issue): void {
-    this.aggregatedIssuesCount++;
-    if (!this.representative) {
-      this.representative = issue;
+    this.#aggregatedIssuesCount++;
+    if (!this.#representative) {
+      this.#representative = issue;
     }
-    this.issueKind = IssuesManager.Issue.unionIssueKind(this.issueKind, issue.getKind());
+    this.#issueKind = IssuesManager.Issue.unionIssueKind(this.#issueKind, issue.getKind());
     let hasRequest = false;
     for (const request of issue.requests()) {
       hasRequest = true;
-      if (!this.affectedRequests.has(request.requestId)) {
-        this.affectedRequests.set(request.requestId, request);
+      if (!this.#affectedRequests.has(request.requestId)) {
+        this.#affectedRequests.set(request.requestId, request);
       }
     }
     for (const cookie of issue.cookies()) {
-      const key = this.keyForCookie(cookie);
-      if (!this.affectedCookies.has(key)) {
-        this.affectedCookies.set(key, {cookie, hasRequest});
+      const key = this.#keyForCookie(cookie);
+      if (!this.#affectedCookies.has(key)) {
+        this.#affectedCookies.set(key, {cookie, hasRequest});
       }
     }
     for (const rawCookieLine of issue.rawCookieLines()) {
-      if (!this.affectedRawCookieLines.has(rawCookieLine)) {
-        this.affectedRawCookieLines.set(rawCookieLine, {rawCookieLine, hasRequest});
+      if (!this.#affectedRawCookieLines.has(rawCookieLine)) {
+        this.#affectedRawCookieLines.set(rawCookieLine, {rawCookieLine, hasRequest});
       }
     }
     for (const location of issue.sources()) {
       const key = JSON.stringify(location);
-      if (!this.affectedLocations.has(key)) {
-        this.affectedLocations.set(key, location);
+      if (!this.#affectedLocations.has(key)) {
+        this.#affectedLocations.set(key, location);
       }
     }
     if (issue instanceof IssuesManager.MixedContentIssue.MixedContentIssue) {
-      this.mixedContentIssues.add(issue);
+      this.#mixedContentIssues.add(issue);
     }
     if (issue instanceof IssuesManager.HeavyAdIssue.HeavyAdIssue) {
-      this.heavyAdIssues.add(issue);
+      this.#heavyAdIssues.add(issue);
     }
     for (const details of issue.getBlockedByResponseDetails()) {
       const key = JSON.stringify(details, ['parentFrame', 'blockedFrame', 'requestId', 'frameId', 'reason', 'request']);
-      this.blockedByResponseDetails.set(key, details);
+      this.#blockedByResponseDetails.set(key, details);
     }
     if (issue instanceof IssuesManager.TrustedWebActivityIssue.TrustedWebActivityIssue) {
-      this.trustedWebActivityIssues.add(issue);
+      this.#trustedWebActivityIssues.add(issue);
     }
     if (issue instanceof IssuesManager.ContentSecurityPolicyIssue.ContentSecurityPolicyIssue) {
-      this.cspIssues.add(issue);
+      this.#cspIssues.add(issue);
     }
     if (issue instanceof IssuesManager.SharedArrayBufferIssue.SharedArrayBufferIssue) {
-      this.sharedArrayBufferIssues.add(issue);
+      this.#sharedArrayBufferIssues.add(issue);
     }
     if (issue instanceof IssuesManager.LowTextContrastIssue.LowTextContrastIssue) {
-      this.lowContrastIssues.add(issue);
+      this.#lowContrastIssues.add(issue);
     }
     if (issue instanceof IssuesManager.CorsIssue.CorsIssue) {
-      this.corsIssues.add(issue);
+      this.#corsIssues.add(issue);
     }
     if (issue instanceof IssuesManager.QuirksModeIssue.QuirksModeIssue) {
-      this.quirksModeIssues.add(issue);
+      this.#quirksModeIssues.add(issue);
     }
     if (issue instanceof IssuesManager.AttributionReportingIssue.AttributionReportingIssue) {
-      this.attributionReportingIssues.add(issue);
+      this.#attributionReportingIssues.add(issue);
     }
     if (issue instanceof IssuesManager.WasmCrossOriginModuleSharingIssue.WasmCrossOriginModuleSharingIssue) {
-      this.wasmCrossOriginModuleSharingIssues.add(issue);
+      this.#wasmCrossOriginModuleSharingIssues.add(issue);
     }
     if (issue instanceof IssuesManager.GenericIssue.GenericIssue) {
-      this.genericIssues.add(issue);
+      this.#genericIssues.add(issue);
     }
   }
 
   getKind(): IssuesManager.Issue.IssueKind {
-    return this.issueKind;
+    return this.#issueKind;
   }
 
   isHidden(): boolean {
-    return this.representative?.isHidden() || false;
+    return this.#representative?.isHidden() || false;
   }
 
   setHidden(_value: boolean): void {
@@ -245,40 +245,40 @@ export class AggregatedIssue extends IssuesManager.Issue.Issue {
 }
 
 export class IssueAggregator extends Common.ObjectWrapper.ObjectWrapper<EventTypes> {
-  private readonly aggregatedIssuesByKey = new Map<AggregationKey, AggregatedIssue>();
-  private readonly hiddenAggregatedIssuesByKey = new Map<AggregationKey, AggregatedIssue>();
+  readonly #aggregatedIssuesByKey = new Map<AggregationKey, AggregatedIssue>();
+  readonly #hiddenAggregatedIssuesByKey = new Map<AggregationKey, AggregatedIssue>();
   constructor(private readonly issuesManager: IssuesManager.IssuesManager.IssuesManager) {
     super();
-    this.issuesManager.addEventListener(IssuesManager.IssuesManager.Events.IssueAdded, this.onIssueAdded, this);
+    this.issuesManager.addEventListener(IssuesManager.IssuesManager.Events.IssueAdded, this.#onIssueAdded, this);
     this.issuesManager.addEventListener(
-        IssuesManager.IssuesManager.Events.FullUpdateRequired, this.onFullUpdateRequired, this);
+        IssuesManager.IssuesManager.Events.FullUpdateRequired, this.#onFullUpdateRequired, this);
     for (const issue of this.issuesManager.issues()) {
-      this.aggregateIssue(issue);
+      this.#aggregateIssue(issue);
     }
   }
 
-  private onIssueAdded(event: Common.EventTarget.EventTargetEvent<IssuesManager.IssuesManager.IssueAddedEvent>): void {
-    this.aggregateIssue(event.data.issue);
+  #onIssueAdded(event: Common.EventTarget.EventTargetEvent<IssuesManager.IssuesManager.IssueAddedEvent>): void {
+    this.#aggregateIssue(event.data.issue);
   }
 
-  private onFullUpdateRequired(): void {
-    this.aggregatedIssuesByKey.clear();
-    this.hiddenAggregatedIssuesByKey.clear();
+  #onFullUpdateRequired(): void {
+    this.#aggregatedIssuesByKey.clear();
+    this.#hiddenAggregatedIssuesByKey.clear();
     for (const issue of this.issuesManager.issues()) {
-      this.aggregateIssue(issue);
+      this.#aggregateIssue(issue);
     }
     this.dispatchEventToListeners(Events.FullUpdateRequired);
   }
 
-  private aggregateIssue(issue: IssuesManager.Issue.Issue): AggregatedIssue {
-    const map = issue.isHidden() ? this.hiddenAggregatedIssuesByKey : this.aggregatedIssuesByKey;
-    const aggregatedIssue = this.aggregateIssueByStatus(map, issue);
+  #aggregateIssue(issue: IssuesManager.Issue.Issue): AggregatedIssue {
+    const map = issue.isHidden() ? this.#hiddenAggregatedIssuesByKey : this.#aggregatedIssuesByKey;
+    const aggregatedIssue = this.#aggregateIssueByStatus(map, issue);
     this.dispatchEventToListeners(Events.AggregatedIssueUpdated, aggregatedIssue);
     return aggregatedIssue;
   }
 
-  private aggregateIssueByStatus(
-      aggregatedIssuesMap: Map<AggregationKey, AggregatedIssue>, issue: IssuesManager.Issue.Issue): AggregatedIssue {
+  #aggregateIssueByStatus(aggregatedIssuesMap: Map<AggregationKey, AggregatedIssue>, issue: IssuesManager.Issue.Issue):
+      AggregatedIssue {
     const key = issue.code() as unknown as AggregationKey;
     let aggregatedIssue = aggregatedIssuesMap.get(key);
     if (!aggregatedIssue) {
@@ -290,20 +290,20 @@ export class IssueAggregator extends Common.ObjectWrapper.ObjectWrapper<EventTyp
   }
 
   aggregatedIssues(): Iterable<AggregatedIssue> {
-    return [...this.aggregatedIssuesByKey.values(), ...this.hiddenAggregatedIssuesByKey.values()];
+    return [...this.#aggregatedIssuesByKey.values(), ...this.#hiddenAggregatedIssuesByKey.values()];
   }
 
   hiddenAggregatedIssues(): Iterable<AggregatedIssue> {
-    return this.hiddenAggregatedIssuesByKey.values();
+    return this.#hiddenAggregatedIssuesByKey.values();
   }
 
   aggregatedIssueCodes(): Set<AggregationKey> {
-    return new Set([...this.aggregatedIssuesByKey.keys(), ...this.hiddenAggregatedIssuesByKey.keys()]);
+    return new Set([...this.#aggregatedIssuesByKey.keys(), ...this.#hiddenAggregatedIssuesByKey.keys()]);
   }
 
   aggregatedIssueCategories(): Set<IssuesManager.Issue.IssueCategory> {
     const result = new Set<IssuesManager.Issue.IssueCategory>();
-    for (const issue of this.aggregatedIssuesByKey.values()) {
+    for (const issue of this.#aggregatedIssuesByKey.values()) {
       result.add(issue.getCategory());
     }
     return result;
@@ -311,18 +311,18 @@ export class IssueAggregator extends Common.ObjectWrapper.ObjectWrapper<EventTyp
 
   aggregatedIssueKinds(): Set<IssuesManager.Issue.IssueKind> {
     const result = new Set<IssuesManager.Issue.IssueKind>();
-    for (const issue of this.aggregatedIssuesByKey.values()) {
+    for (const issue of this.#aggregatedIssuesByKey.values()) {
       result.add(issue.getKind());
     }
     return result;
   }
 
   numberOfAggregatedIssues(): number {
-    return this.aggregatedIssuesByKey.size;
+    return this.#aggregatedIssuesByKey.size;
   }
 
   numberOfHiddenAggregatedIssues(): number {
-    return this.hiddenAggregatedIssuesByKey.size;
+    return this.#hiddenAggregatedIssuesByKey.size;
   }
 
   keyForIssue(issue: IssuesManager.Issue.Issue<string>): AggregationKey {