fix($jsonpCallbacks): allow $window to be mocked in unit tests

Fixes #15685

Closes #15686

Author: Matt Lewis

src/ng/jsonpCallbacks.js

@@ -10,8 +10,8 @@
  * how they vary compared to the requested url.
  */
 var $jsonpCallbacksProvider = /** @this */ function() {
-  this.$get = ['$window', function($window) {
-    var callbacks = $window.angular.callbacks;
+  this.$get = function() {
+    var callbacks = angular.callbacks;
     var callbackMap = {};
 
     function createCallback(callbackId) {
@@ -78,5 +78,5 @@ var $jsonpCallbacksProvider = /** @this */ function() {
         delete callbackMap[callbackPath];
       }
     };
-  }];
+  };
 };

test/ng/jsonpCallbacksSpec.js

@@ -79,4 +79,17 @@ describe('$jsonpCallbacks', function() {
       expect($window.angular.callbacks._0).toBeUndefined();
     }));
   });
+
+  describe('mocked $window', function() {
+
+    beforeEach(module(function($provide) {
+      $provide.value('$window', {});
+    }));
+
+    it('should not throw when $window.angular does not exist', inject(function($injector) {
+      expect(function() {
+        $injector.get('$jsonpCallbacks');
+      }).not.toThrow();
+    }));
+  });
 });