refactor(fromJson): Remove error() and just throw

It's more likely you are using angular.fromJson() inside Angular world, which means you get proper
exception handling by $exceptionHandler.

There is no point to explicitly push it to console and it causes memory leaks on most browsers 
(tried Chrome stable/canary, Safari, FF).

1 files changed, 5 insertions, 10 deletions

diff --git a/src/JSON.js b/src/JSON.js
index db145879..983a53e8 100644
--- a/src/JSON.js
+++ b/src/JSON.js
@@ -37,17 +37,12 @@ function fromJson(json, useNative) {
 
   var obj;
 
-  try {
-    if (useNative && window.JSON && window.JSON.parse) {
-      obj = JSON.parse(json);
-    } else {
-      obj = parseJson(json, true)();
-    }
-    return transformDates(obj);
-  } catch (e) {
-    error("fromJson error: ", json, e);
-    throw e;
+  if (useNative && window.JSON && window.JSON.parse) {
+    obj = JSON.parse(json);
+  } else {
+    obj = parseJson(json, true)();
   }
+  return transformDates(obj);
 
   // TODO make forEach optionally recursive and remove this function
   // TODO(misko): remove this once the $http service is checked in.