Switching to a new format for sending errors

chrome/browser/resources/google_now/utility.js

Index: chrome/browser/resources/google_now/utility.js
diff --git a/chrome/browser/resources/google_now/utility.js b/chrome/browser/resources/google_now/utility.js
index a6fed9478738f8b896bb97c941111d3c3b58f00b..c66b9fff0fec812a4f7746999a01a58b487c1baa 100644
--- a/chrome/browser/resources/google_now/utility.js
+++ b/chrome/browser/resources/google_now/utility.js
@@ -118,17 +118,18 @@ function sendErrorReport(error) {
   if (error.canSendMessageToServer)
     errorText = errorText + ': ' + error.message;
 
-  var requestParameters =
-      'error=' + encodeURIComponent(errorText) +
-      '&script=' + encodeURIComponent(file) +

[robliao, 2013/09/24 20:14:43]

Don't the unit tests need to update for this change?

[vadimt, 2013/09/24 21:10:50]

Ah, you are right!
Done.

-      '&line=' + encodeURIComponent(line) +
-      '&trace=' + encodeURIComponent(filteredStack);
-  var request = buildServerRequest('jserror',
-                                   'application/x-www-form-urlencoded');
+  var errorObject = {
+    message: errorText,
+    file: file,
+    line: line,
+    trace: filteredStack
+  };
+
+  var request = buildServerRequest('jserrors', 'application/json');
   request.onloadend = function(event) {
     console.log('sendErrorReport status: ' + request.status);
   };
-  request.send(requestParameters);
+  request.send(JSON.stringify(errorObject));
 }
 
 // Limiting 1 error report per background page load.