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

Issue 29496582: Issue 5448 - Add content_security_policy to the manifest.json for Chrome-extensions (Closed)

Created:
July 24, 2017, 12:21 p.m. by tlucas
Modified:
July 25, 2017, 10:54 a.m.
Reviewers:
kzar
CC:
Sebastian Noack
Visibility:
Public.

Description

Issue 5448 - Add content_security_policy to the manifest.json for Chrome-extensions

Patch Set 1 #

Total comments: 2

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
M templates/manifest.json.tmpl View 1 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 4
tlucas
Patch Set 1 Added a section for content_security_policy into the jinja2 manifest template
July 24, 2017, 12:24 p.m. (2017-07-24 12:24:38 UTC) #1
kzar
(Moving Sebastian to Cc since he's away at the moment.) Just to check have you ...
July 24, 2017, 1:19 p.m. (2017-07-24 13:19:16 UTC) #2
tlucas
Patch Set 2 Adhering to camelCase for option-names On 2017/07/24 13:19:16, kzar wrote: > (Moving ...
July 24, 2017, 1:50 p.m. (2017-07-24 13:50:44 UTC) #3
kzar
July 24, 2017, 1:54 p.m. (2017-07-24 13:54:04 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld