fix(ui-ace): call $destroy when removed

Author: douglasduteil

test/ace.spec.js

@@ -156,6 +156,18 @@ describe('uiAce', function () {
         expect(compileWithObject).toThrow();
       });
     });
+
+    it('should call destroy when the element is removed', function () {
+      var element = $compile('<div ui-ace ng-model="foo">')(scope);
+      spyOn(_ace, 'destroy').andCallThrough();
+      spyOn(_ace.session, '$stopWorker').andCallThrough();
+
+      element.remove();
+      scope.$apply();
+
+      expect(_ace.session.$stopWorker).toHaveBeenCalled();
+      expect(_ace.destroy).toHaveBeenCalled();
+    });
   });
 
   describe('when the model is an object or an array', function () {
@@ -177,5 +189,4 @@ describe('uiAce', function () {
     });
   });
 
-
-});
+});

ui-ace.js

@@ -94,6 +94,10 @@ angular.module('ui.ace', [])
         // EVENTS
         session.on('change', onChange(opts.onChange));
 
+        elm.on('$destroy', function() {
+          acee.session.$stopWorker();
+          acee.destroy();
+        });
       }
     };
   }]);