Fix leak bindings

Author: aki77

lib/minimap-git-diff-binding.coffee

@@ -13,6 +13,7 @@ class MinimapGitDiffBinding
     @subscriptions = new CompositeDisposable
 
     @subscriptions.add @editor.getBuffer().onDidStopChanging @updateDiffs
+    @subscriptions.add @minimap.onDidDestroy @destroy
 
     if repository = @getRepo()
       @subscriptions.add repository.onDidChangeStatuses =>
@@ -59,10 +60,11 @@ class MinimapGitDiffBinding
     @markers ?= []
     @markers.push(marker)
 
-  destroy: ->
+  destroy: =>
     @removeDecorations()
     @subscriptions.dispose()
     @diffs = null
+    @minimap = null
 
   getPath: -> @editor.getBuffer()?.getPath()
 

lib/minimap-git-diff.coffee

@@ -4,21 +4,20 @@ MinimapGitDiffBinding = null
 
 class MinimapGitDiff
 
-  bindings: {}
   pluginActive: false
   constructor: ->
     @subscriptions = new CompositeDisposable
 
   isActive: -> @pluginActive
 
   activate: ->
+    @bindings = new WeakMap
 
   consumeMinimapServiceV1: (@minimap) ->
     @minimap.registerPlugin 'git-diff', this
 
   deactivate: ->
-    binding.destroy() for id,binding of @bindings
-    @bindings = {}
+    @destroyBindings()
     @gitDiff = null
     @minimap = null
 
@@ -60,14 +59,15 @@ class MinimapGitDiff
 
       return unless editor?
 
-      id = editor.id
       binding = new MinimapGitDiffBinding @gitDiff, minimap
-      @bindings[id] = binding
+      @bindings.set(minimap, binding)
 
   getRepositories: -> atom.project.getRepositories().filter (repo) -> repo?
 
   destroyBindings: =>
-    binding.destroy() for id,binding of @bindings
-    @bindings = {}
+    return unless @minimap?
+    for minimap in @minimap.editorsMinimaps
+      @bindings.get(minimap)?.destroy()
+      @bindings.delete(minimap)
 
 module.exports = new MinimapGitDiff