Merge pull request #2882 from moonmoonCL/fix/2.38.3-memoryleak

SudoUserReal · web-flow · commit 2574af20ba2a · 2025-06-30T15:52:47.000+08:00
fix: cleartimeout memory leak (#1785)
diff --git a/packages/semi-foundation/navigation/subNavFoundation.ts b/packages/semi-foundation/navigation/subNavFoundation.ts
@@ -52,7 +52,9 @@ export default class SubNavFoundation<P = Record<string, any>, S = Record<string
         this._timer = null;
     }
 
-    destroy() {} // eslint-disable-line
+    destroy() {
+        this.clearDelayTimer();
+    }
 
     clearDelayTimer() {
         if (this._timer) {
diff --git a/packages/semi-foundation/tooltip/foundation.ts b/packages/semi-foundation/tooltip/foundation.ts
@@ -115,6 +115,7 @@ export default class Tooltip<P = Record<string, any>, S = Record<string, any>> e
         this._adapter.unregisterClickOutsideHandler();
         this.unBindResizeEvent();
         this.unBindScrollEvent();
+        clearTimeout(this._timer);
     }
 
     _bindTriggerEvent(triggerEventSet: Record<string, any>) {

Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,9 @@ export default class SubNavFoundation<P = Record<string, any>, S = Record<string`
`52`	`52`	`this._timer = null;`
`53`	`53`	`}`
`54`	`54`
`55`		`- destroy() {} // eslint-disable-line`
	`55`	`+ destroy() {`
	`56`	`+ this.clearDelayTimer();`
	`57`	`+ }`
`56`	`58`
`57`	`59`	`clearDelayTimer() {`
`58`	`60`	`if (this._timer) {`
Original file line number	Diff line number	Diff line change
`@@ -115,6 +115,7 @@ export default class Tooltip<P = Record<string, any>, S = Record<string, any>> e`
`115`	`115`	`this._adapter.unregisterClickOutsideHandler();`
`116`	`116`	`this.unBindResizeEvent();`
`117`	`117`	`this.unBindScrollEvent();`
	`118`	`+ clearTimeout(this._timer);`
`118`	`119`	`}`
`119`	`120`
`120`	`121`	`_bindTriggerEvent(triggerEventSet: Record<string, any>) {`