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

Unified Diff: packagerChrome.py

Issue 30061555: Noissue - Added info.locale (Closed)
Patch Set: Created Oct. 4, 2019, 11:54 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 | « no previous file | templates/chromeInfo.js.tmpl » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: packagerChrome.py
===================================================================
--- a/packagerChrome.py
+++ b/packagerChrome.py
@@ -151,11 +151,6 @@
def create_bundles(params, files, bundle_tests):
base_extension_path = params['baseDir']
- info_templates = {
- 'chrome': 'chromeInfo.js.tmpl',
- 'edge': 'edgeInfo.js.tmpl',
- 'gecko': 'geckoInfo.js.tmpl',
- }
aliases = {
# To use our custom loader for the info module we must first set up an
# alias to a file that exists.
@@ -179,10 +174,11 @@
resolve_paths = [os.path.join(base_extension_path, dir, 'lib')
for dir in ['', 'adblockpluscore', 'adblockplusui']]
- info_template = getTemplate(info_templates[params['type']])
+ info_template = getTemplate('info.js.tmpl')
info_module = info_template.render(
basename=params['metadata'].get('general', 'basename'),
version=params['version'],
+ type=params['type'],
).encode('utf-8')
configuration = {
« no previous file with comments | « no previous file | templates/chromeInfo.js.tmpl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld