fix($http): JSON parse failure

docs/content/error/$http/baddata.ngdoc

@@ -0,0 +1,16 @@
+@ngdoc error
+@name $http:baddata
+@fullName Bad JSON Data
+@description
+
+The default @{link ng$http#default-transformations `transformResponse`} will try to parse the
+response as JSON if the `Content-Type` header is `application/json` or the response looks like a
+valid JSON-stringified object or array.
+This error occurs when that data is not a valid JSON object.
+
+The error message should provide additional context such as the actual response.
+
+To resolve this error, make sure you pass a valid JSON data object to `transformResponse`.
+
+For more information, see the {@link ng$http#default-transformations `transformResponse`} service
+API documentation.

src/ng/http.js

@@ -138,7 +138,12 @@ function defaultHttpResponseTransform(data, headers) {
     if (tempData) {
       var contentType = headers('Content-Type');
       if ((contentType && (contentType.indexOf(APPLICATION_JSON) === 0)) || isJsonLike(tempData)) {
-        data = fromJson(tempData);
+        try {
+          data = fromJson(tempData);
+        } catch (e) {
+          throw $httpMinErr('baddata', 'Data must be a valid JSON object. Received: "{0}". ' +
+          'Error occurred: "{1}"', data, e);
+        }
       }
     }
   }

test/ng/httpSpec.js

@@ -1442,6 +1442,17 @@ describe('$http', function() {
           expect(callback.calls.argsFor(1)[0]).toBe(null);
           expect(callback.calls.argsFor(2)[0]).toBe('');
         });
+
+        it('should return JSON data with error message if JSON is invalid', function() {
+          var errCallback = jasmine.createSpy('error');
+          $httpBackend.expect('GET', '/url').respond('{abcd}', {'Content-Type': 'application/json'});
+            $http({method: 'GET', url: '/url'}).then(callback).catch(errCallback);
+            $httpBackend.flush();
+
+            expect(callback).not.toHaveBeenCalled();
+            expect(errCallback).toHaveBeenCalledOnce();
+            expect(errCallback.calls.mostRecent().args[0]).toEqualMinErr('$http', 'baddata');
+        });
       });
     });