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

Unified Diff: chrome/test/data/extensions/api_test/content_scripts/duplicate_script_injection/manifest.json

Issue 2116613002: Prevent duplicate content script injection defined in manifest.json (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Tests Added Created 4 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/test/data/extensions/api_test/content_scripts/duplicate_script_injection/manifest.json
diff --git a/chrome/test/data/extensions/api_test/content_scripts/duplicate_script_injection/manifest.json b/chrome/test/data/extensions/api_test/content_scripts/duplicate_script_injection/manifest.json
new file mode 100644
index 0000000000000000000000000000000000000000..70ad447d47ff44e709b93942fa7c9933d18b8b56
--- /dev/null
+++ b/chrome/test/data/extensions/api_test/content_scripts/duplicate_script_injection/manifest.json
@@ -0,0 +1,26 @@
+{
+ "name": "duplicate script injection",
+ "version": "0.0.1",
+ "manifest_version": 2,
+ "description": "Tests that content scripts assigned to two overlapping url matches are not injected twice. Note this can be extended to N matches",
+ "background": {
+ },
+ "content_scripts": [
+ {
+ "matches": ["*://*.google.com/*"],
+ "js": ["test.js"],
+ "css": ["test.css"],
+ "run_at": "document_end"
+ },
+ {
+ "matches": ["*://maps.google.com/*"],
+ "js": ["test.js"],
+ "css": ["test.css"],
+ "run_at": "document_end"
+ }
+ ],
+ "permissions": [
Devlin 2016/07/11 23:45:50 Do we need these permissions?
catmullings 2016/07/12 22:23:26 Done.
catmullings 2016/07/12 22:23:26 Nope. Removing them.
+ "http://*/",
+ "https://*/"
+ ]
+}

Powered by Google App Engine
This is Rietveld 408576698