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

Side by Side Diff: templates/geckoInfo.js.tmpl

Issue 29542859: Issue 5660 - Check for runtime.getBrowserInfo API support (Closed) Base URL: https://hg.adblockplus.org/buildtools
Patch Set: Parse UA string for only version number Created Sept. 14, 2017, 6:04 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* This Source Code Form is subject to the terms of the Mozilla Public 1 /* This Source Code Form is subject to the terms of the Mozilla Public
2 * License, v. 2.0. If a copy of the MPL was not distributed with this 2 * License, v. 2.0. If a copy of the MPL was not distributed with this
3 * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ 3 * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
4 4
5 "use strict"; 5 "use strict";
6 6
7 exports.addonName = {{ basename|json }}; 7 exports.addonName = {{ basename|json }};
8 exports.addonVersion = {{ version|json }}; 8 exports.addonVersion = {{ version|json }};
9 9
10 exports.application = "unknown"; 10 exports.application = "unknown";
11 exports.applicationVersion = "0"; 11 exports.applicationVersion = "0";
12 12
13 exports.platform = "gecko"; 13 exports.platform = "gecko";
14 exports.platformVersion = "0"; 14 exports.platformVersion = "0";
15 15
16 let match = /\brv:(\d+(?:\.\d+)?)\b/.exec(navigator.userAgent); 16 let match = /\brv:(\d+(?:\.\d+)?)\b/.exec(navigator.userAgent);
17 if (match) 17 if (match)
18 exports.platformVersion = match[1]; 18 exports.platformVersion = match[1];
19 19
20 browser.runtime.getBrowserInfo().then(browserInfo => 20 // Firefox 50 does not support runtime.getBrowserInfo
21 if ("getBrowserInfo" in browser.runtime)
21 { 22 {
22 exports.application = browserInfo.name.toLowerCase(); 23 browser.runtime.getBrowserInfo().then(browserInfo =>
23 exports.applicationVersion = browserInfo.version; 24 {
24 }); 25 exports.application = browserInfo.name.toLowerCase();
26 exports.applicationVersion = browserInfo.version;
27 });
28 }
29 else
30 {
31 let browserInfo = /\bFirefox\/(\S+)/.exec(navigator.userAgent);
32 exports.application = "firefox";
33 if (browserInfo)
34 exports.applicationVersion = browserInfo[1];
35 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld