fix: make drawGutterDecoration a free function

atom-minimap · Jan 2, 2021 · 30e9017 · 30e9017
1 parent 404393d
commit 30e9017
Showing 1 changed file with 15 additions and 14 deletions.
diff --git a/lib/mixins/canvas-drawer.js b/lib/mixins/canvas-drawer.js
@@ -391,7 +391,7 @@ export default class CanvasDrawer extends Mixin {
       renderData.screenRow = screenRow
 
       this.drawDecorations(screenRow, decorations, renderData, {
-        gutter: this.drawGutterDecoration,
+        gutter: drawGutterDecoration,
         'highlight-over': drawHighlightDecoration,
         'highlight-outline': drawHighlightOutlineDecoration,
         'foreground-custom': this.drawCustomDecoration
@@ -511,19 +511,6 @@ export default class CanvasDrawer extends Mixin {
     data.context.fillRect(0, data.yRow, data.canvasWidth, data.lineHeight)
   }
 
-  /**
-   * Draws a gutter decoration.
-   *
-   * @param  {Decoration} decoration the decoration to render
-   * @param  {Object} data the data need to perform the render
-   * @param {string} decorationColor decoration color
-   * @access private
-   */
-  drawGutterDecoration (decoration, data, decorationColor) {
-    data.context.fillStyle = decorationColor
-    data.context.fillRect(0, data.yRow, 1, data.lineHeight)
-  }
-
   /**
    * Draws a custom decoration.
    *
@@ -548,6 +535,7 @@ export default class CanvasDrawer extends Mixin {
 const SPEC_MODE = atom.inSpecMode()
 if (SPEC_MODE) {
   // class methods only used for spying the calls
+  CanvasDrawer.drawGutterDecoration = drawGutterDecoration
   CanvasDrawer.drawHighlightDecoration = drawHighlightDecoration
   CanvasDrawer.drawHighlightOutlineDecoration = drawHighlightOutlineDecoration
 }
@@ -655,6 +643,19 @@ function getInvisibleRegExp (editor) {
   }
 }
 
+/**
+ * Draws a gutter decoration.
+ *
+ * @param  {Decoration} decoration the decoration to render
+ * @param  {Object} data the data need to perform the render
+ * @param {string} decorationColor decoration color
+ * @access private
+ */
+function drawGutterDecoration (decoration, data, decorationColor) {
+  data.context.fillStyle = decorationColor
+  data.context.fillRect(0, data.yRow, 1, data.lineHeight)
+}
+
 /**
  * Draws a highlight decoration.
  *