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

Delta Between Two Patch Sets: templates/manifest.json.tmpl

Issue 29365583: Issue 4670 - Add a new build type for Gecko-based WebExtensions (Closed) Base URL: https://hg.adblockplus.org/buildtools
Left Patch Set: Updated the way options page is specified in the manifest Created Nov. 30, 2016, 1:23 p.m.
Right Patch Set: Split out info module into a Chrome and Gecko specific one Created Dec. 3, 2016, 7:02 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « templates/geckoInfo.js.tmpl ('k') | no next file » | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
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 { 5 {
6 "manifest_version": 2, 6 "manifest_version": 2,
7 {%- if releaseBuild %} 7 {%- if releaseBuild %}
8 "name": "__MSG_name__", 8 "name": "__MSG_name__",
9 {%- else %} 9 {%- else %}
10 "name": "__MSG_name_devbuild__", 10 "name": "__MSG_name_devbuild__",
(...skipping 12 matching lines...) Expand all
23 {%- if metadata.has_section('compat') and metadata.has_option('compat', 'edge' ) %} 23 {%- if metadata.has_section('compat') and metadata.has_option('compat', 'edge' ) %}
24 "minimum_edge_version": {{metadata.get('compat', 'edge')|json}}, 24 "minimum_edge_version": {{metadata.get('compat', 'edge')|json}},
25 {%- endif %} 25 {%- endif %}
26 26
27 {%- if type == 'gecko-webext' %} 27 {%- if type == 'gecko-webext' %}
28 "applications": 28 "applications":
29 { 29 {
30 "gecko": 30 "gecko":
31 { 31 {
32 "id": {{metadata.get('general', 'id')|json}} 32 "id": {{metadata.get('general', 'id')|json}}
33 {%- if metadata.has_section('compat') and metadata.has_option('compat', 'g ecko') %} 33 {%- if metadata.has_section('compat') and metadata.has_option('compat', 'g ecko') %}
Sebastian Noack 2016/12/01 16:24:54 If I understand correctly this will result into a
Wladimir Palant 2016/12/01 21:39:32 It doesn't matter because this JSON file is being
34 , 34 ,
35 "strict_min_version": {{metadata.get('compat', 'gecko')|json}} 35 "strict_min_version": {{metadata.get('compat', 'gecko')|json}}
36 {%- endif %} 36 {%- endif %}
37 } 37 }
38 }, 38 },
39 {%- endif %} 39 {%- endif %}
40 40
41 {%- if pageAction %} 41 {%- if pageAction %}
42 "page_action": { 42 "page_action": {
43 "default_icon": {{pageAction.icon|json}}, 43 "default_icon": {{pageAction.icon|json}},
(...skipping 27 matching lines...) Expand all
71 {%- endif %} 71 {%- endif %}
72 72
73 {%- if backgroundScripts %} 73 {%- if backgroundScripts %}
74 "background": { 74 "background": {
75 "scripts": {{backgroundScripts|json}}, 75 "scripts": {{backgroundScripts|json}},
76 "persistent": true 76 "persistent": true
77 }, 77 },
78 {%- endif %} 78 {%- endif %}
79 79
80 {%- if metadata.has_option('general', 'options') %} 80 {%- if metadata.has_option('general', 'options') %}
81 {%- if type != 'edge' %}
81 "options_ui": { 82 "options_ui": {
Sebastian Noack 2016/12/01 16:24:54 Did you test whether this is supported on Microsof
Wladimir Palant 2016/12/01 21:39:31 You are right, it doesn't. So fallback it is.
82 "page": {{metadata.get('general', 'options')|json}}, 83 "page": {{metadata.get('general', 'options')|json}},
83 "open_in_tab": true 84 "open_in_tab": true
84 }, 85 },
86 {%- else %}
87 "options_page": {{metadata.get('general', 'options')|json}},
88 {%- endif %}
85 {%- endif %} 89 {%- endif %}
86 90
87 {%- if metadata.has_option('general', 'devtools') %} 91 {%- if metadata.has_option('general', 'devtools') %}
88 "devtools_page": {{metadata.get('general', 'devtools')|json}}, 92 "devtools_page": {{metadata.get('general', 'devtools')|json}},
89 {%- endif %} 93 {%- endif %}
90 94
91 {%- if contentScripts %} 95 {%- if contentScripts %}
92 "content_scripts": {{contentScripts|json}}, 96 "content_scripts": {{contentScripts|json}},
93 {%- endif %} 97 {%- endif %}
94 98
95 {%- if webAccessible %} 99 {%- if webAccessible %}
96 "web_accessible_resources": {{webAccessible|json}}, 100 "web_accessible_resources": {{webAccessible|json}},
97 {%- endif %} 101 {%- endif %}
98 102
99 {%- if metadata.has_option('general', 'managedStorageSchema') %} 103 {%- if metadata.has_option('general', 'managedStorageSchema') %}
100 "storage": { 104 "storage": {
101 "managed_schema": {{metadata.get('general', 'managedStorageSchema')|json}} 105 "managed_schema": {{metadata.get('general', 'managedStorageSchema')|json}}
102 }, 106 },
103 {%- endif %} 107 {%- endif %}
104 108
105 "_dummy": false 109 "_dummy": false
106 } 110 }
LEFTRIGHT

Powered by Google App Engine
This is Rietveld