Fix MiniStats VRAM subcategories not hiding on collapse (#8507)

Refactor duplicated sub-graph cleanup into a shared clearSubGraphs helper
and add the missing VRAM cleanup when collapsing below the detail threshold.

Made-with: Cursor

Co-authored-by: Martin Valigursky <mvaligursky@snapchat.com>

Author: mvaligursky

src/extras/mini-stats/mini-stats.js

@@ -351,38 +351,15 @@ class MiniStats {
         // update opacity based on size (larger sizes have higher default opacity)
         this.opacity = value > 0 ? 0.85 : 0.7;
 
-        // delete GPU pass graphs when switching below threshold
+        // delete sub-stat graphs when switching below their thresholds
         if (value < this.gpuTimingMinSize && this.gpuPassGraphs) {
-            for (const passData of this.gpuPassGraphs.values()) {
-                const index = this.graphs.indexOf(passData.graph);
-                if (index !== -1) {
-                    this.graphs.splice(index, 1);
-                }
-                this.freeRow(passData.graph);
-                passData.graph.destroy();
-            }
-            this.gpuPassGraphs.clear();
-
-            // keep main GPU graph in GPU color group
-            const gpuGraph = this.graphs.find(g => g.name === 'GPU');
-            if (gpuGraph) gpuGraph.graphType = 0.33;
+            this.clearSubGraphs(this.gpuPassGraphs, 'GPU', 0.33);
         }
-
-        // delete CPU sub-timing graphs when switching below threshold
         if (value < this.cpuTimingMinSize && this.cpuGraphs) {
-            for (const statData of this.cpuGraphs.values()) {
-                const index = this.graphs.indexOf(statData.graph);
-                if (index !== -1) {
-                    this.graphs.splice(index, 1);
-                }
-                this.freeRow(statData.graph);
-                statData.graph.destroy();
-            }
-            this.cpuGraphs.clear();
-
-            // keep main CPU graph in CPU color group
-            const cpuGraph = this.graphs.find(g => g.name === 'CPU');
-            if (cpuGraph) cpuGraph.graphType = 0.66;
+            this.clearSubGraphs(this.cpuGraphs, 'CPU', 0.66);
+        }
+        if (value < this.vramTimingMinSize && this.vramGraphs) {
+            this.clearSubGraphs(this.vramGraphs);
         }
     }
 
@@ -777,6 +754,31 @@ class MiniStats {
         }
     }
 
+    /**
+     * Remove all sub-stat graphs from a tracking map when collapsing below a size threshold.
+     *
+     * @param {Map} subGraphs - The sub-graph map to clear.
+     * @param {string} [mainGraphName] - If provided, reset the main graph's graphType.
+     * @param {number} [graphType] - The graphType value to restore on the main graph.
+     * @private
+     */
+    clearSubGraphs(subGraphs, mainGraphName, graphType) {
+        for (const statData of subGraphs.values()) {
+            const index = this.graphs.indexOf(statData.graph);
+            if (index !== -1) {
+                this.graphs.splice(index, 1);
+            }
+            this.freeRow(statData.graph);
+            statData.graph.destroy();
+        }
+        subGraphs.clear();
+
+        if (mainGraphName) {
+            const mainGraph = this.graphs.find(g => g.name === mainGraphName);
+            if (mainGraph) mainGraph.graphType = graphType;
+        }
+    }
+
     /**
      * Ensures the texture has enough rows. Only grows, never shrinks.
      *