Extensions: pass ChromeV8Context around instead of v8::Handle.

chrome/test/base/module_system_test.cc

Index: chrome/test/base/module_system_test.cc
diff --git a/chrome/test/base/module_system_test.cc b/chrome/test/base/module_system_test.cc
index 56024269e7736c3e23ca20ea7ce0c9898aeeaf90..f91328b0ae13baa6ecf38d9de5b165f24404264d 100644
--- a/chrome/test/base/module_system_test.cc
+++ b/chrome/test/base/module_system_test.cc
@@ -20,7 +20,7 @@ using extensions::ObjectBackedNativeHandler;
 // Native JS functions for doing asserts.
 class AssertNatives : public ObjectBackedNativeHandler {
  public:
-  explicit AssertNatives(v8::Handle<v8::Context> context)
+  explicit AssertNatives(extensions::ChromeV8Context* context)
       : ObjectBackedNativeHandler(context),
         assertion_made_(false),
         failed_(false) {
@@ -87,12 +87,18 @@ class FailsOnException : public ModuleSystem::ExceptionHandler {
 ModuleSystemTest::ModuleSystemTest()
     : isolate_(v8::Isolate::GetCurrent()),
       handle_scope_(isolate_),
-      context_(v8::Context::New(isolate_)),
+      context_(
+          new extensions::ChromeV8Context(
+              v8::Context::New(isolate_),
+              reinterpret_cast<WebKit::WebFrame*>(1),
+              NULL,
+              extensions::Feature::UNSPECIFIED_CONTEXT)),
       source_map_(new StringSourceMap()),
       should_assertions_be_made_(true) {
-  context_->Enter();
+  context_->v8_context()->Enter();
   assert_natives_ = new AssertNatives(context_.get());
-  module_system_.reset(new ModuleSystem(context_.get(), source_map_.get()));
+  module_system_.reset(
+      new ModuleSystem(context_.get(), source_map_.get()));
   module_system_->RegisterNativeHandler("assert", scoped_ptr<NativeHandler>(
       assert_natives_));
   module_system_->SetExceptionHandlerForTest(
@@ -101,7 +107,7 @@ ModuleSystemTest::ModuleSystemTest()
 
 ModuleSystemTest::~ModuleSystemTest() {
   module_system_.reset();
-  context_->Exit();
+  context_->v8_context()->Exit();

[not at google - send to devlin, 2013/05/29 17:41:55]

idly wonder why this doesn't hold onto a context scope as a member. whatever,
nothing to do with this patch.

 }
 
 void ModuleSystemTest::RegisterModule(const std::string& name,