Applied changes from emailed code review

lib/typedItCollector.js

Index: lib/typedItCollector.js
===================================================================
--- a/lib/typedItCollector.js
+++ b/lib/typedItCollector.js
@@ -148,46 +148,28 @@
 {
   if (!Prefs.domainOptIn || (domains.length == 0 && falsePositives.length == 0 && userCorrections.length == 0) || privateBrowsingEnabled())
     return;
-
-  let args = [];
+  
+  let postData = {};
   let savedDomains = domains;
   let savedFalsePositives = falsePositives;
   let savedUserCorrections = userCorrections;
 
   if(domains.length > 0)
   {
-    args.push(domains.map(function(d) "domains[]=" + encodeURIComponent(d)).join("&"));
+    postData.domains = domains;

[Wladimir Palant, 2012/09/21 14:40:25]

Now we are using JSON but not its advantages - the server is still receiving
three lists of domains that it has to compare. As mentioned in previous review,
it would be better to store and send structured data: each entry should
correspond to one string the user typed into the URL bar. That entry should
contain what the user type in, if we autocorrected it - what we corrected it
into and whether the user marked it as a false positive.

     domains = [];
   }
   if(falsePositives.length > 0)
   {
-    args.push(
-      falsePositives.map(
-        function(fp)
-        {
-          return "falsePositives[]=" + encodeURIComponent(
-            encodeURIComponent(fp[0]) + ((fp[1]) ? "&" + encodeURIComponent(fp[1]) : "")
-          );
-        }
-      ).join("&")
-    );
+    postData.falsePositives = falsePositives;
     falsePositives = [];
   }
   if(userCorrections.length > 0)
   {
-    args.push(
-      userCorrections.map(
-        function(uc)
-        {
-          return "userCorrections[]=" + encodeURIComponent(
-            encodeURIComponent(uc[0]) + ((uc[1]) ? "&" + encodeURIComponent(uc[1]) : "")
-          );
-        }
-      ).join("&")
-    );
+    postData.userCorrections = userCorrections;
     userCorrections = [];
   }
-
+  
   let request = Cc["@mozilla.org/xmlextras/xmlhttprequest;1"].createInstance(Ci.nsIXMLHttpRequest);
   request.open("POST", "http://typed.it/submitData");
   request.setRequestHeader("Content-Type", "application/x-www-form-urlencoded");
@@ -200,7 +182,7 @@
       userCorrections = userCorrections.concat(savedUserCorrections);
     }
   }, false);
-  request.send(args.join("&"));
+  request.send("data=" + JSON.stringify(postData));

[Wladimir Palant, 2012/09/21 14:40:25]

There is little point to send JSON data in pseudo-urlencoded format.
Content-Type header should be application/json and request body just the plain
JSON data.

Side-note: Please verify that Unicode data is correctly encoded as UTF-8 here (I
think it is).

 }
 
 function initWebUI(event)