fix($rootScope): deallocate $$listeners array when empty

Author: jbedard

src/ng/rootScope.js

@@ -955,7 +955,6 @@ function $RootScopeProvider() {
         // Disable listeners, watchers and apply/digest methods
         this.$destroy = this.$digest = this.$apply = this.$evalAsync = this.$applyAsync = noop;
         this.$on = this.$watch = this.$watchGroup = function() { return noop; };
-        this.$$listeners = {};
 
         // Disconnect the next sibling to prevent `cleanUpScope` destroying those too
         this.$$nextSibling = null;
@@ -1364,6 +1363,7 @@ function $RootScopeProvider() {
 
         if (current.$$listenerCount[name] === 0) {
           delete current.$$listenerCount[name];
+          delete current.$$listeners[name];
         }
       } while ((current = current.$parent));
     }

test/ng/rootScopeSpec.js

@@ -2562,6 +2562,26 @@ describe('Scope', function() {
             expect(secondListener).not.toHaveBeenCalled();
           });
         });
+
+
+        it('should deallocate the $$listenerCount entry when reaching 0', inject(function($rootScope) {
+          var child = $rootScope.$new();
+
+          child.$on('abc', noop)();
+
+          expect('abc' in $rootScope.$$listenerCount).toBe(false);
+          expect('abc' in child.$$listenerCount).toBe(false);
+        }));
+
+
+        it('should deallocate the $$listeners entry when empty', inject(function($rootScope) {
+          var child = $rootScope.$new();
+
+          child.$on('abc', noop)();
+
+          expect('abc' in $rootScope.$$listeners).toBe(false);
+          expect('abc' in child.$$listeners).toBe(false);
+        }));
       });
     });