commit 8d4265479e2c0d408cd37959ca8f36d544b73a1c
author Anri Sidorov <ansid@google.com> Thu Sep 25 13:35:54 2025 +0200
committer SkCQ <skcq-be@skia-corp.google.com.iam.gserviceaccount.com> Thu Sep 25 04:48:27 2025 -0700
tree 44c2a0732305dc91ed0a3e6f10a0c2bff07a82e7
parent e41bc5d5fc0e2fa0a8692b9cdcf22ebbf7e181b7

[perfUI] Adjust host path comparison for skipping preGenerateMultiGraphUrl

Make the comparison work with both formats, with and without trailing '/'.

Bug: 447103073
Change-Id: I9a6ecd062dd8f465696c0a3bd09430a80a2e6c26
Reviewed-on: https://skia-review.googlesource.com/c/buildbot/+/1061876
Reviewed-by: Marcin Mordecki <mordeckimarcin@google.com>
Reviewed-by: Sergei Rudenkov <sergeirudenkov@google.com>
Commit-Queue: Anri Sidorov <ansid@google.com>

perf/modules/anomalies-table-sk/anomalies-table-sk.ts

diff --git a/perf/modules/anomalies-table-sk/anomalies-table-sk.ts b/perf/modules/anomalies-table-sk/anomalies-table-sk.ts
index 6130560..b639238 100644
--- a/perf/modules/anomalies-table-sk/anomalies-table-sk.ts
+++ b/perf/modules/anomalies-table-sk/anomalies-table-sk.ts
@@ -52,9 +52,9 @@
 
   multiChartUrlToAnomalyMap: Map<string, string> = new Map<string, string>();
 
-  private regressionsPageHost = '/a/';
+  private regressionsPageHost = '/a';
 
-  private reportPageHost = '/u/';
+  private reportPageHost = '/u';
 
   private isParentRow = false;
 
@@ -781,7 +781,11 @@
       msg.hidden = true;
       table.hidden = false;
       this.anomalyList = anomalyList;
-      if (![this.regressionsPageHost, this.reportPageHost].includes(window.location.pathname)) {
+      let currentPath = window.location.pathname;
+      if (currentPath.endsWith('/')) {
+        currentPath = currentPath.slice(0, -1);
+      }
+      if (![this.regressionsPageHost, this.reportPageHost].includes(currentPath)) {
         // I am not sure we need preGenerateMultiGraphUrl at all.
         // TODO(sergeirudenkov): I want to log metrics here, but don't know whether we
         // have mechanism for this.
@@ -914,7 +918,11 @@
     // when the user chooses 'V8 Javascript Perf' on the Regressions page.
     // It would significantly increase the page loading time if it pre-generates each row's url.
     // To prevent this, we will only pre-generate the URLs on the Report page.
-    if (window.location.pathname !== this.regressionsPageHost) {
+    let currentPath = window.location.pathname;
+    if (currentPath.endsWith('/')) {
+      currentPath = currentPath.slice(0, -1);
+    }
+    if (currentPath !== this.regressionsPageHost) {
       const url = this.multiChartUrlToAnomalyMap.get(anomaly.id);
       if (url) {
         return this.openAnomalyUrl(url);