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

Unified Diff: packagerChrome.py

Issue 29743581: Issue 6552 - Support arbitrary manifest values (Closed) Base URL: https://hg.adblockplus.org/buildtools/file/a3db4a1a49e8
Patch Set: Issue 6552 - Support arbitrary manifest values Created April 11, 2018, 9:42 a.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
Index: packagerChrome.py
diff --git a/packagerChrome.py b/packagerChrome.py
index 1dadca95ccedcf55cd0ed0699fc59c11be7654f2..698211e4d86ee435bb34715a406fa16fda76325f 100644
--- a/packagerChrome.py
+++ b/packagerChrome.py
@@ -101,27 +101,20 @@ def createManifest(params, files):
templateData[opt] = {'icon': icon, 'popup': popup}
+ # Generate templatedataa which does not need special processing
+ arbitrary_manifest = metadata.as_json_object('manifest')
+ templateData.update(arbitrary_manifest)
+
if metadata.has_option('general', 'icons'):
templateData['icons'] = makeIcons(files,
metadata.get('general', 'icons').split())
- if metadata.has_option('general', 'permissions'):
- templateData['permissions'] = metadata.get('general', 'permissions').split()
-
- if metadata.has_option('general', 'optionalPermissions'):
- templateData['optionalPermissions'] = metadata.get(
- 'general', 'optionalPermissions').split()
-
if metadata.has_option('general', 'backgroundScripts'):
templateData['backgroundScripts'] = metadata.get(
'general', 'backgroundScripts').split()
if params['devenv']:
templateData['backgroundScripts'].append('devenvPoller__.js')
- if metadata.has_option('general', 'webAccessible') and metadata.get('general', 'webAccessible') != '':
- templateData['webAccessible'] = metadata.get('general',
- 'webAccessible').split()
-
if metadata.has_section('contentScripts'):
contentScripts = []
for run_at, scripts in metadata.items('contentScripts'):

Powered by Google App Engine
This is Rietveld