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

Side by Side Diff: safariInfo.js.tmpl

Issue 29332683: Noissue - Removed unused info.addonID and info.addonRoot for Chrome and Safari (Closed)
Patch Set: Also remove unused addonRoot Created Dec. 15, 2015, 4:52 p.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 | « chromeInfo.js.tmpl ('k') | 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 require.scopes.info = { 5 require.scopes.info = {
6 addonID: '',
7 addonName: {{ metadata.get('general', 'basename')|json }}, 6 addonName: {{ metadata.get('general', 'basename')|json }},
8 addonVersion: {{ version|json }}, 7 addonVersion: {{ version|json }},
9 addonRoot: '',
10 8
11 application: 'safari', 9 application: 'safari',
12 get applicationVersion() { 10 get applicationVersion() {
13 return navigator.userAgent.match(/Version\/([\d.]+)/)[1]; 11 return navigator.userAgent.match(/Version\/([\d.]+)/)[1];
14 }, 12 },
13
15 platform: 'safari', 14 platform: 'safari',
16 get platformVersion() { 15 get platformVersion() {
17 return this.applicationVersion; 16 return this.applicationVersion;
18 } 17 }
19 }; 18 };
OLDNEW
« no previous file with comments | « chromeInfo.js.tmpl ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld