Issue 5701 - Inconsistent forum links and DNT link

messageResponder.js

Index: messageResponder.js
===================================================================
--- a/messageResponder.js
+++ b/messageResponder.js
@@ -179,6 +179,14 @@
       };
     }
 
+    if (message.what == "browserInfo")

[Sebastian Noack, 2017/09/25 17:09:22]

Just an idea; how about merging the logic into the app.what[doclink] message:

  if (message.what == "doclink")
  {
    let application = info.application;
    if (info.platform == "chromium" && application != "opera")
      application = "chrome";
    else if (info.platform == "gecko")
      application = "firefox;
    return Utils.getDocLink(message.link.replace("{application}", application);
  }

That way we wouldn't have to introduce a new message sub-type, and the logic
would be reusable.

[saroyanm, 2017/09/25 18:07:53]

Don't have strong opinion, on one hand it makes this specific code reusable, but
on other hand less flexible.
I think having platform + application information in future will help us make
more browser specific tweaks, like show browser specific messages. So we
probably will introduce "browserInfo" if the issue is sending 1 request instead
of 2.

[Sebastian Noack, 2017/09/25 19:07:47]

Also note that the reason you introduced "browserInfo" was to avoid an
unnecessary round-trip to the background page. If you handle this logic in
"doclink", you would avoid yet another round-trip. As for flexibility, I don't
see why it matters much whether to implement it in the background page or on the
content side, you are in control of both, and can just adapt the logic (or move
it back to the content) whenever this becomes necessary. However, handling it in
the "doclink" message seems to yield the simplest solution for now.

[saroyanm, 2017/09/25 20:50:12]

I agree with you, looks much more better now.
Done.

+    {
+      return {
+        platform: info.platform,
+        application: info.application
+      };
+    }
+
     return info[message.what];
   });