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

Delta Between Two Patch Sets: tests/metadata.chrome

Issue 29501558: Issue 5383 - Add tests for the Chrome and Firefox packagers (Closed)
Left Patch Set: Adding pytest-cov, adhering to comments Created Aug. 14, 2017, 2:19 p.m.
Right Patch Set: Addressing Vasily's comments Created Oct. 22, 2017, 11:11 a.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 | « tests/expecteddata/manifest_gecko_release_build.json ('k') | tests/metadata.edge » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 [general] 1 [general]
2 version = 1.2.3 2 version = 1.2.3
3 author = Eyeo GmbH 3 author = eyeo GmbH
Sebastian Noack 2017/08/14 15:20:11 "eyeo" is spelled lower-case. Alternativley, you m
tlucas 2017/09/06 09:04:52 Done.
4 basename = adblockpluschrome 4 basename = adblockpluschrome
5 permissions = tabs 5 permissions = tabs
6 <all_urls> 6 <all_urls>
7 contextMenus 7 optionalPermissions = webRequest
8 webRequest
9 webRequestBlocking
10 webNavigation
11 storage
12 unlimitedStorage
13 notifications
14 backgroundScripts = ext/common.js 8 backgroundScripts = ext/common.js
15 ext/background.js 9 ext/background.js
16 lib/compat.js
17 lib/publicSuffixList.js
18 lib/adblockplus.js
19 messageResponder.js
20 background.js
21 testScripts = ../ext/common.js 10 testScripts = ../ext/common.js
22 ../ext/background.js 11 ../ext/background.js
23 ../lib/compat.js
Sebastian Noack 2017/08/14 15:20:12 Do we need all the files listed in "backgroundScri
tlucas 2017/09/06 09:04:51 Done.
24 devtools = devtools.html 12 devtools = devtools.html
25 options = options.html 13 options = options.html
26 managedStorageSchema = managed-storage-schema.json 14 managedStorageSchema = storage.json
15 browserAction = icons/abp-16.png icons/abp-19.png icons/abp-53.png popup.html
16 icons = icons/abp-16.png
17 webAccessible = images/*.png
27 18
Sebastian Noack 2017/08/14 15:20:11 We probably should also add a [contentScripts] sec
tlucas 2017/09/06 09:04:51 Done, but: packagerEdge does not handle a section
Sebastian Noack 2017/09/06 18:00:00 Not sure what you mean but there are definitely co
28 [convert_js] 19 [contentScripts]
29 lib/foo.js = 20 planets = content.js
30 lib/a.js
31 lib/b.js
32 lib/foo.js[module] = true
33 lib/foo.js[injectinfomodule] = true
34 lib/foo.js[autoload] = icons
35 21
36 [mapping] 22 [mapping]
37 logo_44.png = chrome/icons/abp-44.png 23 logo_44.png = chrome/icons/abp-44.png
38 foo/logo_50.png = chrome/icons/abp-50.png 24 foo/logo_50.png = chrome/icons/abp-50.png
39 icons/logo_150.png = chrome/icons/abp-150.png 25 icons/logo_150.png = chrome/icons/abp-150.png
40 26
41 [compat] 27 [compat]
42 chrome = 49.0 28 chrome = 49.0
43 opera = 36.0 29 opera = 36.0
44 30
45 [import_locales] 31 [import_locales]
46 _trans/*/test.properties = name 32 _modules/*/module.json = =*
47 name_devbuild 33 _modules/*/unit.properties = =*
34 _imp/*/gecko.dtd = =*
35
36 [bundles]
37 lib/foo.js = ext/a.js
38 lib/b.js
LEFTRIGHT

Powered by Google App Engine
This is Rietveld