From 4676df5833941529c8558f607993c4fee50aa9b3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Mi=C5=A1ko=20Hevery?= <misko@hevery.com>
Date: Thu, 9 Feb 2017 13:31:30 -0600
Subject: [PATCH] fix(http): REVERT: remove dots from jsonp callback name
 (#13219)

This reverts commit 9e5617e41e222d29c3952ba71a401804030cd153.
---
 modules/@angular/http/src/backends/browser_jsonp.ts    | 10 +++++-----
 .../@angular/http/test/backends/jsonp_backend_spec.ts  |  5 -----
 2 files changed, 5 insertions(+), 10 deletions(-)

diff --git a/modules/@angular/http/src/backends/browser_jsonp.ts b/modules/@angular/http/src/backends/browser_jsonp.ts
index 022431c136..2734836afc 100644
--- a/modules/@angular/http/src/backends/browser_jsonp.ts
+++ b/modules/@angular/http/src/backends/browser_jsonp.ts
@@ -32,23 +32,23 @@ export class BrowserJsonp {
 
   nextRequestID(): string { return `__req${_nextRequestId++}`; }
 
-  requestCallback(id: string): string { return `${JSONP_HOME}${id}_finished`; }
+  requestCallback(id: string): string { return `${JSONP_HOME}.${id}.finished`; }
 
-  exposeConnection(id: string, connection: any): void {
+  exposeConnection(id: string, connection: any) {
     const connections = _getJsonpConnections();
     connections[id] = connection;
   }
 
-  removeConnection(id: string): void {
+  removeConnection(id: string) {
     const connections = _getJsonpConnections();
     connections[id] = null;
   }
 
   // Attach the <script> element to the DOM
-  send(node: any): void { document.body.appendChild(<Node>(node)); }
+  send(node: any) { document.body.appendChild(<Node>(node)); }
 
   // Remove <script> element from the DOM
-  cleanup(node: any): void {
+  cleanup(node: any) {
     if (node.parentNode) {
       node.parentNode.removeChild(<Node>(node));
     }
diff --git a/modules/@angular/http/test/backends/jsonp_backend_spec.ts b/modules/@angular/http/test/backends/jsonp_backend_spec.ts
index 8a5c2b422e..81b5ce6373 100644
--- a/modules/@angular/http/test/backends/jsonp_backend_spec.ts
+++ b/modules/@angular/http/test/backends/jsonp_backend_spec.ts
@@ -70,11 +70,6 @@ export function main() {
       expect(instance).toBeAnInstanceOf(JSONPConnection);
     });
 
-    it('callback name should not contain dots', () => {
-      const domJsonp = new MockBrowserJsonp();
-      const callback: string = domJsonp.requestCallback(domJsonp.nextRequestID());
-      expect(callback.indexOf('.') === -1).toBeTruthy();
-    });
 
     describe('JSONPConnection', () => {
       it('should use the injected BaseResponseOptions to create the response',