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

chrome/test/base/module_system_test.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_TEST_BASE_MODULE_SYSTEM_TEST_H_
 #define CHROME_TEST_BASE_MODULE_SYSTEM_TEST_H_
 
+#include "chrome/renderer/extensions/chrome_v8_context.h"
 #include "chrome/renderer/extensions/module_system.h"
 #include "chrome/renderer/extensions/scoped_persistent.h"
 #include "v8/include/v8.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 class AssertNatives;
 class StringSourceMap;
 
 // Test fixture for testing JS that makes use of the module system.
 //
@@ skipping 29 matching lines @@
   // Make the test fail if any asserts are called. By default a test will fail
   // if no asserts are called.
   void ExpectNoAssertionsMade();
 
   // Create an empty object in the global scope with name |name|.
   v8::Handle<v8::Object> CreateGlobal(const std::string& name);
 
   v8::Isolate* isolate_;
   v8::HandleScope handle_scope_;
   extensions::ScopedPersistent<v8::Context> context_;
+  scoped_ptr<extensions::ChromeV8Context> chrome_v8_context_;

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

It would be nicer if this were just scoped_ptr<e::ChromeV8Context> context_
rather than having both a context_ and a chrome_v8_context_ field.

[marja, 2013/05/29 16:56:33]

Done.

   AssertNatives* assert_natives_;
   scoped_ptr<StringSourceMap> source_map_;
   scoped_ptr<extensions::ModuleSystem> module_system_;
   bool should_assertions_be_made_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(ModuleSystemTest);
 };
 
 #endif  // CHROME_TEST_BASE_MODULE_SYSTEM_TEST_H_