Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(608)

Unified Diff: chrome/browser/resources/pdf/manifest.json

Issue 1217503012: Avoid cross-origin iframe issues when loading PDF in print preview (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 5 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/pdf/manifest.json
diff --git a/chrome/browser/resources/pdf/manifest.json b/chrome/browser/resources/pdf/manifest.json
index 8c44ec21b43756064f1c5a3bbdcf819e5ee551ad..93341c26c042775c73c2aa1669f0c92f5c545f12 100644
--- a/chrome/browser/resources/pdf/manifest.json
+++ b/chrome/browser/resources/pdf/manifest.json
@@ -14,18 +14,13 @@
"application/pdf"
],
"content_security_policy": "script-src 'self' blob: filesystem: chrome://resources; object-src *; plugin-types application/x-google-chrome-pdf",
- // This is to work-around an issue where this extension is not granted
- // permission to access chrome://resources when iframed for print preview.
- // See https://crbug.com/444752.
- "content_scripts": [
- {
- "matches": ["chrome://print/*"],
- "js": ["content_script.js"]
- }
- ],
"mime_types_handler": "<INDEX>",
"web_accessible_resources": [
"index.html",
- "<INDEX>"
+ "<INDEX>",
+ "*.js",
+ "*.html",
Charlie Reis 2015/07/16 19:25:03 Why do you need both this and index.html?
raymes 2015/07/17 01:58:29 Good point, we actually don't now. Done.
+ "*.css",
+ "*.png"
]
}

Powered by Google App Engine
This is Rietveld 408576698