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

Unified Diff: chrome/renderer/extensions/content_watcher.cc

Issue 16032015: Extensions: pass ChromeV8Context around instead of v8::Handle. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebased Created 7 years, 7 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/content_watcher.cc
diff --git a/chrome/renderer/extensions/content_watcher.cc b/chrome/renderer/extensions/content_watcher.cc
index 72c3334b86d009c70ffbd8d609ba035aed0bd502..d19960e1e8c5a58dfa08b644b1cda251bde9fb49 100644
--- a/chrome/renderer/extensions/content_watcher.cc
+++ b/chrome/renderer/extensions/content_watcher.cc
@@ -20,9 +20,9 @@ namespace {
class MutationHandler : public ChromeV8Extension {
public:
explicit MutationHandler(Dispatcher* dispatcher,
- v8::Handle<v8::Context> v8_context,
+ ChromeV8Context* context,
base::WeakPtr<ContentWatcher> content_watcher)
- : ChromeV8Extension(dispatcher, v8_context),
+ : ChromeV8Extension(dispatcher, context),
content_watcher_(content_watcher) {
RouteFunction("FrameMutated",
base::Bind(&MutationHandler::FrameMutated,
@@ -33,7 +33,7 @@ class MutationHandler : public ChromeV8Extension {
v8::Handle<v8::Value> FrameMutated(const v8::Arguments& args) {
if (content_watcher_.get()) {
content_watcher_->ScanAndNotify(
- WebKit::WebFrame::frameForContext(v8_context()));
+ WebKit::WebFrame::frameForContext(context()->v8_context()));
}
return v8::Undefined();
}
@@ -49,9 +49,9 @@ ContentWatcher::ContentWatcher(Dispatcher* dispatcher)
ContentWatcher::~ContentWatcher() {}
scoped_ptr<NativeHandler> ContentWatcher::MakeNatives(
- v8::Handle<v8::Context> v8_context) {
+ ChromeV8Context* context) {
return scoped_ptr<NativeHandler>(new MutationHandler(
- dispatcher_, v8_context, weak_ptr_factory_.GetWeakPtr()));
+ dispatcher_, context, weak_ptr_factory_.GetWeakPtr()));
}
void ContentWatcher::OnWatchPages(
« no previous file with comments | « chrome/renderer/extensions/content_watcher.h ('k') | chrome/renderer/extensions/context_menus_custom_bindings.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698