Issue 5085 - Add edgeInfo.js template for edge specific builds

templates/edgeInfo.js.tmpl

Index: templates/edgeInfo.js.tmpl
===================================================================
new file mode 100644
--- /dev/null
+++ b/templates/edgeInfo.js.tmpl
@@ -0,0 +1,36 @@
+/* This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
+
+"use strict";
+
+let platform = "edgehtml";
+let platformVersion = null;
+let application = "edge";
+let applicationVersion = "0";
+
+let regexp = /(\S+)\/(\S+)(?:\s*\(.*?\))?/g;
+let match;
+
+while (match = regexp.exec(navigator.userAgent))
+{
+  let app = match[1];
+  let ver = match[2];
+  platformVersion = ver;
+}
+
+// not a Chromium-based UA, probably modifed by the user

[Jon Sonesen, 2017/06/07 11:08:43]

Not sure if I should change the comment, seems it should have chromium replaed
with edge?

+if (!platformVersion)
+{
+  application = "unknown";
+  applicationVersion = platformVersion = "0";
+}
+
+exports.addonName = {{ basename|json }};
+exports.addonVersion = {{ version|json }};
+
+exports.application = application;

[Sebastian Noack, 2017/06/07 11:14:13]

Since this code is only bundled with the build for Microsoft Edge, there is no
way that the application (this code is running on) is anything. If the user
agent string indicates some other browser, it is wrong. So just hard-code "edge"
here, and "edgehtml" for the platform below.

Then the logic above can be simplified to merely extract the version out of the
user agent string, using an inlined regexp, no need for a loop.

[Jon Sonesen, 2017/06/07 11:41:38]

Acknowledged. What about the logic if platform version is null (where my comment
above is)? would this just be wrong and we would want it to fail? Should it be
removed?

[Sebastian Noack, 2017/06/07 11:51:46]

If the regular expression fails we should still set "platformVersion" to "0" to
indicate that it is unknown, "platform" should still be "edgehtml".

[Jon Sonesen, 2017/06/07 13:09:28]

Acknowledged.

+exports.applicationVersion = applicationVersion;
+
+exports.platform = platform;
+exports.platformVersion = platformVersion;