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

Unified Diff: extensions/renderer/content_watcher.cc

Issue 344433003: Prepare declarativeContent API for new script injection feature. Added Javascript types and functio… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add note about what it means to revert RequestContentScript Created 6 years, 6 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: extensions/renderer/content_watcher.cc
diff --git a/extensions/renderer/content_watcher.cc b/extensions/renderer/content_watcher.cc
index 684de5f4b51a4e3ac0aaae463bf8a0feab8f67bf..5c903263fdff53e3f0fa5e804de81090a1fa2f53 100644
--- a/extensions/renderer/content_watcher.cc
+++ b/extensions/renderer/content_watcher.cc
@@ -19,7 +19,10 @@ using blink::WebString;
using blink::WebVector;
using blink::WebView;
-ContentWatcher::ContentWatcher() {}
+ContentWatcher::ContentWatcher(const std::string& event_name,
+ WatchedPagesRecipient watched_pages_recipient) :
+ event_name_(event_name),
+ watched_pages_recipient_(watched_pages_recipient) {}
ContentWatcher::~ContentWatcher() {}
void ContentWatcher::OnWatchPages(
@@ -113,8 +116,19 @@ void ContentWatcher::NotifyBrowserOfChange(
selector_strings.push_back(it->as_string());
content::RenderView* view =
content::RenderView::FromWebView(top_frame->view());
- view->Send(new ExtensionHostMsg_OnWatchedPageChange(view->GetRoutingID(),
- selector_strings));
+ switch (watched_pages_recipient_) {
+ case CONTENT_RULES_REGISTRY: {
+ view->Send(new ExtensionHostMsg_OnWatchedPageChange(view->GetRoutingID(),
+ event_name_,
+ selector_strings));
+ break;
+ }
+ case USER_SCRIPT_MANAGER: {
+ // TODO(markdittmer): Notify renderer-side script manager of
+ // selector_strings.
+ break;
+ }
+ }
}
} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698