Issue 5967 - Don't rely on URLSearchParams() constructor to accept a sequence

issue-reporter.js

Index: issue-reporter.js
===================================================================
--- a/issue-reporter.js
+++ b/issue-reporter.js
@@ -398,22 +398,27 @@ function initSendPage()
     let component = uuid[i].toString(16);
     while (component.length < 4)
       component = "0" + component;
     uuidString += component;
     if (i >= 1 && i <= 4)
       uuidString += "-";
   }
 
-  let params = new URLSearchParams({
-    version: 1,
-    guid: uuidString,
-    lang: reportData.getElementsByTagName("adblock-plus")[0]
-                    .getAttribute("locale")
-  });
+  let params = new URLSearchParams();
+  for (let [param, value] of [

[kzar, 2017/11/03 10:12:02]

I can't remember, does array parameter destructuring work with Chrome 49?

[kzar, 2017/11/03 10:12:03]

Maybe add a comment explaining this is a workaround for older versions of Chrome
and Firefox?

[Wladimir Palant, 2017/11/03 10:45:41]

It does. That's part of the reason why we require Chrome 49.
Done.

+    ["version", 1],
+    ["guid", uuidString],
+    ["lang", reportData.getElementsByTagName("adblock-plus")[0]
+                       .getAttribute("locale")]
+  ])
+  {
+    params.append(param, value);
+  }
+
   let url = "https://reports.adblockplus.org/submitReport?" + params;
 
   let reportSent = event =>
   {
     let success = false;
     let errorMessage = browser.i18n.getMessage(
       "filters_subscription_lastDownload_connectionError"
     );