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

chrome/renderer/extensions/file_system_natives.cc

 // 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.
 
 #include "chrome/renderer/extensions/file_system_natives.h"
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/logging.h"
 #include "chrome/common/url_constants.h"
+#include "chrome/renderer/extensions/chrome_v8_context.h"
 #include "chrome/renderer/extensions/user_script_slave.h"
 #include "extensions/common/constants.h"
 #include "grit/renderer_resources.h"
 #include "third_party/WebKit/public/platform/WebFileSystem.h"
 #include "third_party/WebKit/public/platform/WebFileSystemType.h"
 #include "third_party/WebKit/public/platform/WebString.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebFrame.h"
 #include "webkit/common/fileapi/file_system_types.h"
 #include "webkit/common/fileapi/file_system_util.h"
 
 namespace extensions {
 
-FileSystemNatives::FileSystemNatives(v8::Handle<v8::Context> context)
+FileSystemNatives::FileSystemNatives(ChromeV8Context* context)
     : ObjectBackedNativeHandler(context) {
   RouteFunction("GetFileEntry",
       base::Bind(&FileSystemNatives::GetFileEntry, base::Unretained(this)));
   RouteFunction("GetIsolatedFileSystem",
       base::Bind(&FileSystemNatives::GetIsolatedFileSystem,
                  base::Unretained(this)));
   RouteFunction("CrackIsolatedFileSystemName",
       base::Bind(&FileSystemNatives::CrackIsolatedFileSystemName,
                  base::Unretained(this)));
 }
 
 v8::Handle<v8::Value> FileSystemNatives::GetIsolatedFileSystem(
     const v8::Arguments& args) {
   DCHECK(args.Length() == 1 || args.Length() == 2);
   DCHECK(args[0]->IsString());
   std::string file_system_id(*v8::String::Utf8Value(args[0]));
-  WebKit::WebFrame* webframe = WebKit::WebFrame::frameForContext(v8_context());
+  WebKit::WebFrame* webframe =
+      WebKit::WebFrame::frameForContext(context()->v8_context());
   DCHECK(webframe);
 
   GURL context_url =
       extensions::UserScriptSlave::GetDataSourceURLForFrame(webframe);
   CHECK(context_url.SchemeIs(extensions::kExtensionScheme));
 
   std::string name(fileapi::GetIsolatedFileSystemName(context_url.GetOrigin(),
                                                       file_system_id));
 
   // The optional second argument is the subfolder within the isolated file
@@ skipping 32 matching lines @@
   DCHECK(args[3]->IsString());
   std::string file_system_name(*v8::String::Utf8Value(args[1]->ToString()));
   std::string file_system_root_url(*v8::String::Utf8Value(args[2]->ToString()));
   std::string file_path_string(*v8::String::Utf8Value(args[3]->ToString()));
   base::FilePath file_path = base::FilePath::FromUTF8Unsafe(file_path_string);
   DCHECK(fileapi::VirtualPath::IsAbsolute(file_path.value()));
 
   DCHECK(args[4]->IsBoolean());
   bool is_directory = args[4]->BooleanValue();
 
-  WebKit::WebFrame* webframe = WebKit::WebFrame::frameForContext(v8_context());
+  WebKit::WebFrame* webframe =
+      WebKit::WebFrame::frameForContext(context()->v8_context());
   DCHECK(webframe);
   return webframe->createFileEntry(
       type,
       WebKit::WebString::fromUTF8(file_system_name),
       WebKit::WebString::fromUTF8(file_system_root_url),
       WebKit::WebString::fromUTF8(file_path_string),
       is_directory);
 }
 
 v8::Handle<v8::Value> FileSystemNatives::CrackIsolatedFileSystemName(
     const v8::Arguments& args) {
   DCHECK_EQ(args.Length(), 1);
   DCHECK(args[0]->IsString());
   std::string filesystem_name = *v8::String::Utf8Value(args[0]->ToString());
   std::string filesystem_id;
   if (!fileapi::CrackIsolatedFileSystemName(filesystem_name, &filesystem_id))
     return v8::Undefined();
 
   return v8::String::New(filesystem_id.c_str(), filesystem_id.size());
 }
 
 }  // namespace extensions