Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 { | 1 { |
| 2 "name": "__MSG_extensionName__", | 2 "name": "__MSG_extensionName__", |
| 3 "description": "__MSG_extensionDescription__", | 3 "description": "__MSG_extensionDescription__", |
| 4 "default_locale": "en", | 4 "default_locale": "en", |
| 5 "version": "2.0", | 5 "version": "2.0", |
| 6 "manifest_version": 2, | 6 "manifest_version": 2, |
| 7 "minimum_chrome_version": "52", | 7 "minimum_chrome_version": "52", |
| 8 "icons": { | 8 "icons": { |
| 9 "16": "icons/icon16.png", | 9 "16": "icons/icon16.png", |
| 10 "32": "icons/icon32.png", | 10 "32": "icons/icon32.png", |
| (...skipping 17 matching lines...) Expand all Loading... | |
| 28 "js": ["is_editable.js", "content_script.js"], | 28 "js": ["is_editable.js", "content_script.js"], |
| 29 "all_frames": true, | 29 "all_frames": true, |
| 30 "run_at": "document_start" | 30 "run_at": "document_start" |
| 31 } | 31 } |
| 32 ], | 32 ], |
| 33 "options_ui": { | 33 "options_ui": { |
| 34 "page": "pages/options.html", | 34 "page": "pages/options.html", |
| 35 "chrome_style": true | 35 "chrome_style": true |
| 36 }, | 36 }, |
| 37 "permissions": [ | 37 "permissions": [ |
| 38 "activeTab", | |
|
Pam (message me for reviews)
2016/10/07 23:12:43
It turns out this is needed after all, to get the
Devlin
2016/10/29 00:30:31
Sorry, don't follow - why is this needed? Which p
Pam (message me for reviews)
2016/10/29 05:10:36
The popup attached to the browser action, pages/po
Devlin
2016/11/01 15:12:14
That sounds like a bug to me. If you have all url
| |
| 38 "management", | 39 "management", |
| 39 "storage", | 40 "storage", |
| 40 "<all_urls>" | 41 "<all_urls>" |
| 41 ] | 42 ] |
| 42 } | 43 } |
| OLD | NEW |