Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: templates/chromeInfo.js.tmpl

Issue 29365583: Issue 4670 - Add a new build type for Gecko-based WebExtensions (Closed) Base URL: https://hg.adblockplus.org/buildtools
Patch Set: Fixed application info Created Nov. 30, 2016, 12:21 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « packagerChrome.py ('k') | templates/manifest.json.tmpl » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: templates/chromeInfo.js.tmpl
===================================================================
--- a/templates/chromeInfo.js.tmpl
+++ b/templates/chromeInfo.js.tmpl
@@ -24,16 +24,28 @@
}
else if (app == "Edge")
{
platform = "edgehtml";
platformVersion = ver;
application = "edge";
applicationVersion = "0";
}
+ else if (app == "Gecko")
+ {
+ platform = "gecko";
+ match = /\brv:(\d+(?:\.\d+)?)\b/.exec(navigator.userAgent);
+ if (match)
+ platformVersion = match[1];
+ browser.runtime.getBrowserInfo().then(function(info)
+ {
+ require.scopes.info.application = info.name.toLowerCase();
+ require.scopes.info.applicationVersion = info.version;
+ });
+ }
else if (app != "Mozilla" && app != "AppleWebKit" && app != "Safari")
{
// For compatibility with legacy websites, Chrome's UA
// also includes a Mozilla, AppleWebKit and Safari token.
// Any further name/version pair indicates a fork.
application = app == "OPR" ? "opera" : app.toLowerCase();
applicationVersion = ver;
}
« no previous file with comments | « packagerChrome.py ('k') | templates/manifest.json.tmpl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld