Bug 1274708 Use Context.jsonStringify() in connectNative r=kmag

MozReview-Commit-ID: J7q49nLg4F3

--HG--
extra : rebase_source : 9bdddffa43811fdff10fa75ea77bc0641012c9a3

toolkit/components/extensions/NativeMessaging.jsm

@@ -267,7 +267,7 @@ this.NativeApp = class extends EventEmitter {
 
     let json;
     try {
-      json = JSON.stringify(msg);
+      json = this.context.jsonStringify(msg);
     } catch (err) {
       throw new this.context.cloneScope.Error(err.message);
     }

toolkit/components/extensions/test/mochitest/test_chrome_ext_native_messaging.html

@@ -248,15 +248,14 @@ add_task(function* test_happy_path() {
       what: "object with several properties",
     },
 
-    // enable with bug 1274708
-    //{
-    //  data: {
-    //    ignoreme: true,
-    //    _json: {data: "i have a tojson method"},
-    //  },
-    //  expected: {data: "i have a tojson method"},
-    //  what: "object with toJSON() method",
-    //},
+    {
+      data: {
+        ignoreme: true,
+        _json: {data: "i have a tojson method"},
+      },
+      expected: {data: "i have a tojson method"},
+      what: "object with toJSON() method",
+    },
   ];
   for (let test of tests) {
     extension.sendMessage("send", test.data);

toolkit/components/extensions/test/xpcshell/test_native_messaging.js

@@ -70,9 +70,13 @@ add_task(function* setup() {
   notEqual(PYTHON, null, "Found a suitable python interpreter");
 });
 
+let global = this;
+
 // Test of HostManifestManager.lookupApplication() begin here...
 let context = {
   url: null,
+  jsonStringify(...args) { return JSON.stringify(...args); },
+  cloneScope: global,
   logError() {},
   preprocessors: {},
   callOnClose: () => {},