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

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: . 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 3176fdd0cf7dfac7a65aee9825988953358de025..858692da483f2143a27c90bdafb673f1cebd34bf 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,
@@ -49,7 +49,7 @@ ContentWatcher::ContentWatcher(Dispatcher* dispatcher)
ContentWatcher::~ContentWatcher() {}
scoped_ptr<NativeHandler> ContentWatcher::MakeNatives(
- v8::Handle<v8::Context> v8_context) {
+ ChromeV8Context* v8_context) {
not at google - send to devlin 2013/05/29 15:55:13 v8_context -> context
marja 2013/05/29 16:56:33 Done.
return scoped_ptr<NativeHandler>(new MutationHandler(
dispatcher_, v8_context, weak_ptr_factory_.GetWeakPtr()));
}

Powered by Google App Engine
This is Rietveld 408576698