fix(modal): clean up event listeners when hiding modal

Author: perrygovier

Diff for: js/angular/service/modal.js

@@ -218,7 +218,7 @@ function($rootScope, $ionicBody, $compile, $timeout, $ionicPlatform, $ionicTempl
 
       // clean up event listeners
       if (self.positionView) {
-        $window.addEventListener('resize', self._onWindowResize);
+        $window.removeEventListener('resize', self._onWindowResize);
       }
 
       return $timeout(function() {

Diff for: test/unit/angular/service/popover.unit.js

@@ -94,6 +94,16 @@ describe('Ionic Popover', function() {
     expect(angular.element(document.body).hasClass('popover-open')).toBe(false);
   }));
 
+  it('expect hide to remove event listeners', inject(function($window) {
+    var instance = popover.fromTemplate('<div class="popover">hi</div>');
+    spyOn($window,'removeEventListener');
+    instance.show();
+    timeout.flush();
+    instance.hide();
+    timeout.flush();
+    expect($window.removeEventListener).toHaveBeenCalled();
+  }));
+
   it('should animate leave and destroy scope on remove', inject(function($animate) {
     var instance = popover.fromTemplate('<div class="popover"></div>');
     spyOn($animate, 'leave').andCallFake(function(el, cb) { cb(); });