fix($jsonpCallbacks): do not overwrite callbacks added by other apps

Author: gkalpak

src/ng/jsonpCallbacks.js

@@ -11,9 +11,9 @@
  */
 var $jsonpCallbacksProvider = function() {
   this.$get = ['$window', function($window) {
-    var counter = 0;
-    $window.angular.callbacks = {};
     var callbackMap = {};
+    var callbacks = $window.angular.callbacks = $window.angular.callbacks || {};
+    callbacks.$$counter = callbacks.$$counter || 0;
 
     function createCallback(callbackId) {
       var callback = function(data) {
@@ -35,10 +35,10 @@ var $jsonpCallbacksProvider = function() {
        * to pass to the server, which will be used to call the callback with its payload in the JSONP response.
        */
       createCallback: function(url) {
-        var callbackId = '_' + (counter++).toString(36);
+        var callbackId = '_' + (callbacks.$$counter++).toString(36);
         var callbackPath = 'angular.callbacks.' + callbackId;
         var callback = createCallback(callbackId);
-        callbackMap[callbackPath] = $window.angular.callbacks[callbackId] = callback;
+        callbackMap[callbackPath] = callbacks[callbackId] = callback;
         return callbackPath;
       },
       /**
@@ -75,7 +75,7 @@ var $jsonpCallbacksProvider = function() {
        */
       removeCallback: function(callbackPath) {
         var callback = callbackMap[callbackPath];
-        delete $window.angular.callbacks[callback.id];
+        delete callbacks[callback.id];
         delete callbackMap[callbackPath];
       }
     };

src/ngMock/angular-mocks.js

@@ -2954,7 +2954,7 @@ angular.mock.$RootScopeDecorator = ['$delegate', function($delegate) {
     angular.forEach(angular.callbacks, function(val, key) {
       delete angular.callbacks[key];
     });
-    angular.callbacks.counter = 0;
+    angular.callbacks.$$counter = 0;
   };
 
   (window.beforeEach || window.setup)(module.$$beforeEach);

test/ng/jsonpCallbacksSpec.js

@@ -36,6 +36,19 @@ describe('$jsonpCallbacks', function() {
       $jsonpCallbacks.createCallback('http://some.dummy.com/jsonp/request');
       expect($window.angular.callbacks._3).toEqual(jasmine.any(Function));
     }));
+
+    it('should produce unique callback paths across multiple instances', function() {
+      var $jsonpCallbacks1 = angular.injector(['ng', 'ngMock']).get('$jsonpCallbacks');
+      var $jsonpCallbacks2 = angular.injector(['ng', 'ngMock']).get('$jsonpCallbacks');
+
+      var path1 = $jsonpCallbacks1.createCallback('http://some.dummy.com/jsonp/request');
+      var path2 = $jsonpCallbacks2.createCallback('http://some.dummy.com/jsonp/request');
+
+      expect(path1).toBe('angular.callbacks._0');
+      expect(path2).toBe('angular.callbacks._1');
+      expect(angular.callbacks._0).toBeDefined();
+      expect(angular.callbacks._1).toBeDefined();
+    });
   });