fix($rootScope): Set no context when calling listeners

When calling a listener, do not expose the inner workings with `this`.

Author: lgalfaso

src/ng/rootScope.js

@@ -744,7 +744,7 @@ function $RootScopeProvider() {
        *
        */
       $digest: function() {
-        var watch, value, last,
+        var watch, value, last, fn,
             watchers,
             length,
             dirty, ttl = TTL,
@@ -798,7 +798,8 @@ function $RootScopeProvider() {
                       dirty = true;
                       lastDirtyWatch = watch;
                       watch.last = watch.eq ? copy(value, null) : value;
-                      watch.fn(value, ((last === initWatchVal) ? value : last), current);
+                      fn = watch.fn;
+                      fn(value, ((last === initWatchVal) ? value : last), current);
                       if (ttl < 5) {
                         logIdx = 4 - ttl;
                         if (!watchLog[logIdx]) watchLog[logIdx] = [];

test/ng/rootScopeSpec.js

@@ -117,6 +117,16 @@ describe('Scope', function() {
     }));
 
 
+    it('should not expose the `inner working of watch', inject(function($rootScope) {
+      function Listener() {
+        expect(this).toBeUndefined();
+      }
+      if (msie < 10) return;
+      $rootScope.$watch('name', Listener);
+      $rootScope.$digest();
+    }));
+
+
     it('should watch and fire on expression change', inject(function($rootScope) {
       var spy = jasmine.createSpy();
       $rootScope.$watch('name.first', spy);