fix: requests flush changes if not already requested

This fixes the previous commit which did not cancel animation frame

Author: aminya

lib/minimap.js

@@ -312,18 +312,14 @@ class Minimap {
     this.pendingChangeEvents = this.pendingChangeEvents.concat(changes)
 
     // Optimisation: If the redraw delay is set to 0, do not even schedule a timer
-    if (!this.redrawDelay && !this.requestedFlushChanges) {
-      this.requestedFlushChanges = true
-      requestAnimationFrame(() => { this.flushChanges(); this.requestedFlushChanges = false })
+    if (!this.redrawDelay) {
+      this.requestFlushChanges()
     }
 
-    if (!this.flushChangesTimer && !this.requestedFlushChanges) {
+    if (!this.flushChangesTimer) {
       // If any changes happened within the timeout's delay, a timeout will already have been
       // scheduled -> no need to schedule again
-      this.requestedFlushChanges = true
-      this.flushChangesTimer = setTimeout(
-        () => { requestAnimationFrame(() => { this.flushChanges(); this.requestedFlushChanges = false }) }
-      , this.redrawDelay)
+      this.flushChangesTimer = setTimeout(() => { this.requestFlushChanges() }, this.redrawDelay)
     }
   }
 
@@ -340,6 +336,24 @@ class Minimap {
     this.pendingChangeEvents = []
   }
 
+  /**
+   * Requests flush changes if not already requested
+   *
+   * @return void
+   * @access private
+   */
+  requestFlushChanges() {
+    if (!this.requestedFlushChanges) {
+      this.requestedFlushChanges = requestAnimationFrame(() => {
+        this.flushChanges();
+        if (this.requestedFlushChanges) {
+          cancelAnimationFrame(this.requestedFlushChanges);
+          this.requestedFlushChanges = null
+        }
+      })
+    }
+  }
+
   /**
    * Registers an event listener to the `did-change` event.
    *