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

Unified Diff: chrome/renderer/extensions/user_script_slave.h

Issue 226663003: Allow content script insertion on about:-URLs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: use url of parent(s) instead of origin, more tests Created 6 years, 8 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/renderer/extensions/user_script_slave.h
diff --git a/chrome/renderer/extensions/user_script_slave.h b/chrome/renderer/extensions/user_script_slave.h
index a538014c06fc2075d3f874966e2fe18b205f9f93..533f3bfc34a360b16d0f3e50e90c3fbaf02010f6 100644
--- a/chrome/renderer/extensions/user_script_slave.h
+++ b/chrome/renderer/extensions/user_script_slave.h
@@ -36,6 +36,14 @@ class UserScriptSlave {
// committed, this is the commited URL. Otherwise it is the provisional URL.
static GURL GetDataSourceURLForFrame(const blink::WebFrame* frame);
+ // Returns the first non-about:-URL in the document hierarchy above and
+ // including |frame|. The document hierarchy is only traversed if
+ // |document_url| is an about:-URL and if |match_about_blank| is true.
+ static GURL GetEffectiveDocumentURL(const blink::WebFrame* frame,
+ const GURL& document_url,
+ bool match_about_blank);
+
+
not at google - send to devlin 2014/05/01 20:32:19 delete 1 of these blank lines
robwu 2014/05/01 21:30:38 Done.
explicit UserScriptSlave(const ExtensionSet* extensions);
~UserScriptSlave();

Powered by Google App Engine
This is Rietveld 408576698