chrome/common/extensions/docs/examples/api/tabs/screenshot/manifest.json - Issue 8318001: Adding `content_security_policy` to a few sample extensions

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: chrome/common/extensions/docs/examples/api/tabs/screenshot/manifest.json

Issue 8318001: Adding `content_security_policy` to a few sample extensions (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: processes/show_tabs Created 9 years, 2 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch | Annotate | Revision Log

« chrome/common/extensions/docs/examples/api/processes/show_tabs/manifest.json ('K') | « chrome/common/extensions/docs/examples/api/tabs/screenshot/background.js ('k') | chrome/common/extensions/docs/examples/api/tabs/screenshot/screenshot.html » ('j') | chrome/common/extensions/docs/examples/api/windows/merge_windows/background.html » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 {	1 {

2 "name": "Test Screenshot Extension",	2 "name": "Test Screenshot Extension",

3 "version": "1.0",	3 "version": "1.0",

4 "description": "Demonstrate screenshot functionality in the chrome.tabs api. N ote: only works for code.google.com",	4 "description": "Demonstrate screenshot functionality in the chrome.tabs api. N ote: only works for code.google.com",

5 "background_page": "background.html",	5 "background_page": "background.html",

6 "browser_action": {	6 "browser_action": {

7 "default_icon": "camera.png",	7 "default_icon": "camera.png",

8 "default_title": "Take a screen shot!"	8 "default_title": "Take a screen shot!"

9 },	9 },

10 "permissions": ["tabs", "http://code.google.com/"]	10 "permissions": ["tabs", "http://code.google.com/"],

	11 "content_security_policy": "default-src 'none'; script-src 'self'; img-src dat a:"
	Boris Smus 2011/10/18 21:42:06 Cool, just read about this! One concern is that th Cool, just read about this! One concern is that the spec will change to remove some of these features. Thoughts? Mike West 2011/10/19 08:39:12 I don't think that's likely (maybe I'm misundersta Show quoted text On 2011/10/18 21:42:06, Boris Smus wrote: > Cool, just read about this! One concern is that the spec will change to remove > some of these features. Thoughts? I don't think that's likely (maybe I'm misunderstanding your question?). Script and images are relatively critical aspects of websites; I don't think the spec would remove the ability to whitelist them.
11 }	12 }

OLD	NEW