Issue 5701 - Inconsistent forum links and DNT link

new-options.js

Index: new-options.js
===================================================================
--- a/new-options.js
+++ b/new-options.js
@@ -866,6 +866,24 @@
       setLinks("enable-aa-description", link);
     });
 
+    // Get platform specific links
+    Promise.all([getPromisedInfo("platform"), getPromisedInfo("application")])

[Sebastian Noack, 2017/09/25 16:12:48]

Perhaps it would be simpler to move this logic into messageResponder.js, so that
you don't have to send two messages in the first place.

[saroyanm, 2017/09/25 16:34:23]

I agree that sending two messages is not a good idea.
Done.

+      .then(([platform, application]) =>
+      {

[saroyanm, 2017/09/25 15:59:50]

I wonder if we need to check for the "gecko" and assign Firefox to it ?
Same way we assign chrome to all chromium platforms that are not "opera" ?

Considering the fact that application for the Edge browser is "edge" we
shouldn't have problem here I guess.

[Sebastian Noack, 2017/09/25 16:05:55]

That is an excellent idea. Though in practice it wouldn't be a problem right
now, as we don't show this link on Firefox Mobile, and there are no other Gecko
applications that support WebExtensions. But in case this changes in the future,
I think it would be a good idea to get the logic here right from the beginning.
Yes, application=edge is hard-coded in the Microsoft Edge builds, so no need to
explicitly handle this here.

[saroyanm, 2017/09/25 16:34:23]

Done.

+        if (platform == "chromium" && application != "opera")
+          application = "chrome";
+
+        getDocLink(application + "_support", url =>
+        {
+          setLinks("visit-forum", url);
+        });
+
+        getDocLink("adblock_plus_" + application + "_dnt", url =>
+        {
+          setLinks("dnt", url);
+        });
+      });
+
     // Advanced tab
     let customize = document.querySelectorAll("#customize li[data-pref]");
     customize = Array.prototype.map.call(customize, (checkbox) =>
@@ -914,10 +932,6 @@
     {
       setLinks("report-bug", link);
     });
-    getDocLink("reporter_other_link", (link) =>
-    {
-      setLinks("report-forum", link);
-    });
     getDocLink("social_twitter", (link) =>
     {
       E("twitter").setAttribute("href", link);
@@ -1042,6 +1056,17 @@
       privacyList && privacyList.disabled == false;
   }
 
+  function getPromisedInfo(info)
+  {
+    return new Promise((resolve, reject) =>
+    {
+      ext.backgroundPage.sendMessage({
+        type: "app.get",
+        what: info
+      }, resolve);
+    });
+  }
+
   function populateLists()
   {
     subscriptionsMap = Object.create(null);