fix: make updateFrontDecorationsLayer a free function

Author: aminya

lib/mixins/canvas-drawer.js

@@ -160,7 +160,7 @@ export default class CanvasDrawer extends Mixin {
 
     renderData.context = this.frontLayer.context
 
-    this.updateFrontDecorationsLayer(firstRow, lastRow, renderData, lineHeight, editorElement, decorations)
+    updateFrontDecorationsLayer(this.frontLayer, firstRow, lastRow, this.offscreenFirstRow, this.offscreenLastRow, this.pendingFrontDecorationChanges, renderData, lineHeight, editorElement, decorations)
 
     this.pendingChanges = []
     this.pendingBackDecorationChanges = []
@@ -180,58 +180,6 @@ export default class CanvasDrawer extends Mixin {
     this.offscreenLastRow = lastRow
   }
 
-  /**
-   * Performs an update of the front decorations layer using the pending front
-   * decorations changes arrays.
-   *
-   * @param  {number} firstRow firstRow the first row of the range to update
-   * @param  {number} lastRow lastRow the last row of the range to update
-   *
-   * @param {Object} renderData
-   * @param {number} lineHeight  this.minimap.getLineHeight() * devicePixelRatio
-   * @param {TextEditorElement} editorElement  this.minimap.getTextEditorElement()
-   * @param {Array<Decoration>} decorations
-   * @access private
-   */
-  updateFrontDecorationsLayer (firstRow, lastRow, renderData, lineHeight, editorElement, decorations) {
-    const intactRanges = computeIntactRanges(firstRow, lastRow, this.pendingFrontDecorationChanges, this.offscreenFirstRow, this.offscreenLastRow)
-
-    // NOTE: this method is the hot function of Minimap. Do not refactor. The code is inlined delibarately.
-
-    // redrawRangesOnLayer
-
-    this.frontLayer.clearCanvas()
-
-    if (intactRanges.length === 0) {
-      drawFrontDecorationsForLines(firstRow, lastRow, 0, renderData, lineHeight, editorElement, decorations)
-    } else {
-      for (let j = 0, len = intactRanges.length; j < len; j++) {
-        const intact = intactRanges[j]
-
-        this.frontLayer.copyPartFromOffscreen(
-          intact.offscreenRow * lineHeight,
-          (intact.start - firstRow) * lineHeight,
-          (intact.end - intact.start) * lineHeight
-        )
-      }
-      // drawLinesForRanges
-      let currentRow = firstRow
-      for (let i = 0, len = intactRanges.length; i < len; i++) {
-        const range = intactRanges[i]
-
-        drawFrontDecorationsForLines(currentRow, range.start, currentRow - firstRow, renderData, lineHeight, editorElement, decorations)
-
-        currentRow = range.end
-      }
-      if (currentRow <= lastRow) {
-        drawFrontDecorationsForLines(currentRow, lastRow, currentRow - firstRow, renderData, lineHeight, editorElement, decorations)
-      }
-    }
-
-    this.frontLayer.resetOffscreenSize()
-    this.frontLayer.copyToOffscreen()
-  }
-
   //    ########  ########     ###    ##      ##
   //    ##     ## ##     ##   ## ##   ##  ##  ##
   //    ##     ## ##     ##  ##   ##  ##  ##  ##
@@ -424,6 +372,62 @@ function updateBackDecorationsLayer (backLayer, firstRow, lastRow, offscreenFirs
   backLayer.copyToOffscreen()
 }
 
+/**
+ * Performs an update of the front decorations layer using the pending front
+ * decorations changes arrays.
+ *
+ * @param {CanvasLayer} frontLayer
+ * @param  {number} firstRow firstRow the first row of the range to update
+ * @param  {number} lastRow lastRow the last row of the range to update
+ *
+ * @param {number} offscreenFirstRow
+ * @param {number} offscreenLastRow
+ * @param {Array<>} pendingFrontDecorationChanges
+ * @param {Object} renderData
+ * @param {number} lineHeight  this.minimap.getLineHeight() * devicePixelRatio
+ * @param {TextEditorElement} editorElement  this.minimap.getTextEditorElement()
+ * @param {Array<Decoration>} decorations
+ * @access private
+ */
+function updateFrontDecorationsLayer (frontLayer, firstRow, lastRow, offscreenFirstRow, offscreenLastRow, pendingFrontDecorationChanges, renderData, lineHeight, editorElement, decorations) {
+  const intactRanges = computeIntactRanges(firstRow, lastRow, pendingFrontDecorationChanges, offscreenFirstRow, offscreenLastRow)
+
+  // NOTE: this method is the hot function of Minimap. Do not refactor. The code is inlined delibarately.
+
+  // redrawRangesOnLayer
+
+  frontLayer.clearCanvas()
+
+  if (intactRanges.length === 0) {
+    drawFrontDecorationsForLines(firstRow, lastRow, 0, renderData, lineHeight, editorElement, decorations)
+  } else {
+    for (let j = 0, len = intactRanges.length; j < len; j++) {
+      const intact = intactRanges[j]
+
+      frontLayer.copyPartFromOffscreen(
+        intact.offscreenRow * lineHeight,
+        (intact.start - firstRow) * lineHeight,
+        (intact.end - intact.start) * lineHeight
+      )
+    }
+    // drawLinesForRanges
+    let currentRow = firstRow
+    for (let i = 0, len = intactRanges.length; i < len; i++) {
+      const range = intactRanges[i]
+
+      drawFrontDecorationsForLines(currentRow, range.start, currentRow - firstRow, renderData, lineHeight, editorElement, decorations)
+
+      currentRow = range.end
+    }
+    if (currentRow <= lastRow) {
+      drawFrontDecorationsForLines(currentRow, lastRow, currentRow - firstRow, renderData, lineHeight, editorElement, decorations)
+    }
+  }
+
+  frontLayer.resetOffscreenSize()
+  frontLayer.copyToOffscreen()
+}
+
 const whitespaceTokenRegex = /^\s+$/
 const oneOrMoreWhiteSpaceRegexp = /\s+/