commit ab0818b28eb4ca1a1f441a09d4557a137c4dc4bf
author Joe Gregorio <jcgregorio@google.com> Mon Jul 11 13:00:14 2022 -0400
committer SkCQ <skcq-be@skia-corp.google.com.iam.gserviceaccount.com> Mon Jul 11 17:23:49 2022 +0000
tree ebef6727a3221805ad769ecc52152145824a2a86
parent a4c4a5561da3b0054d04431600741147877d73bf

[perf] Allow the summary to be turned off.

Also set the default to off, since this doesn't seem to be
used by many people.

Change-Id: I2a69abc44a8a2be21ec8619fbbbdb4d8d324be75
Reviewed-on: https://skia-review.googlesource.com/c/buildbot/+/557394
Commit-Queue: Joe Gregorio <jcgregorio@google.com>
Reviewed-by: Ravi Mistry <rmistry@google.com>

perf/modules/explore-sk/explore-sk.ts

diff --git a/perf/modules/explore-sk/explore-sk.ts b/perf/modules/explore-sk/explore-sk.ts
index edffc49..d1b1186 100644
--- a/perf/modules/explore-sk/explore-sk.ts
+++ b/perf/modules/explore-sk/explore-sk.ts
@@ -142,6 +142,8 @@
   pivotRequest: pivot.Request = defaultPivotRequest();
 
   sort: string = '' // Pivot table sort order.
+
+  summary: boolean = false; // Whether to show the zoom/summary area.
 }
 
 // TODO(jcgregorio) Move to a 'key' module.
@@ -354,6 +356,13 @@
           title='Toggle the presence of the zero line.'>
         </checkbox-sk>
         <checkbox-sk
+          name=summary
+          @change=${ele.summaryChangeHandler}
+          ?checked=${ele.state.summary}
+          label='Summary'
+          title='Toggle the presence of the summary pane.'>
+        </checkbox-sk>
+        <checkbox-sk
           name=dots
           @change=${ele.toggleDotsHandler}
           ?checked=${ele.state.dots}
@@ -397,7 +406,7 @@
 
     <div id=spin-overlay>
       <plot-simple-sk
-        summary
+        .summary=${ele.state.summary}
         id=plot
         @trace_selected=${ele.traceSelected}
         @zoom=${ele.plotZoom}
@@ -1017,6 +1026,12 @@
     this.zeroChanged();
   }
 
+  private summaryChangeHandler(e: MouseEvent) {
+    this.state.summary = (e.target! as HTMLInputElement).checked;
+    this._stateHasChanged();
+    this._render();
+  }
+
   private toggleDotsHandler() {
     this.state.dots = !this.state.dots;
     this._stateHasChanged();