fix: performance issue caused by excessive use of clearTimeout/Interval

CodeForked · Sep 21, 2021 · 22aea50 · 22aea50
1 parent 8f22191
commit 22aea50
Showing 1 changed file with 20 additions and 5 deletions.
diff --git a/src/index.js b/src/index.js
@@ -587,10 +587,13 @@ class ReactTooltip extends React.Component {
       }
     };
 
-    clearTimeout(this.delayShowLoop);
+    if (this.delayShowLoop) {
+      clearTimeout(this.delayShowLoop);
+    }
     if (delayTime) {
       this.delayShowLoop = setTimeout(updateState, delayTime);
     } else {
+      this.delayShowLoop = null;
       updateState();
     }
   }
@@ -737,10 +740,22 @@ class ReactTooltip extends React.Component {
    * CLear all kinds of timeout of interval
    */
   clearTimer() {
-    clearTimeout(this.delayShowLoop);
-    clearTimeout(this.delayHideLoop);
-    clearTimeout(this.delayReshow);
-    clearInterval(this.intervalUpdateContent);
+    if (this.delayShowLoop) {
+      clearTimeout(this.delayShowLoop);
+      this.delayShowLoop = null;
+    }
+    if (this.delayHideLoop) {
+      clearTimeout(this.delayHideLoop);
+      this.delayHideLoop = null;
+    }
+    if (this.delayReshow) {
+      clearTimeout(this.delayReshow);
+      this.delayReshow = null;
+    }
+    if (this.intervalUpdateContent) {
+      clearInterval(this.intervalUpdateContent);
+      this.intervalUpdateContent = null;
+    }
   }
 
   hasCustomColors() {