Refactor core/v8/* methods to take isolate as first parameter

Source/bindings/core/v8/NPV8Object.cpp

 /*
  * Copyright (C) 2004, 2006 Apple Computer, Inc.  All rights reserved.
  * Copyright (C) 2007, 2008, 2009 Google, Inc.  All rights reserved.
  * Copyright (C) 2014 Opera Software ASA. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
@@ skipping 79 matching lines @@
     ASSERT(npObject->_class == &V8NPObjectClass);
     V8NPObject* object = reinterpret_cast<V8NPObject*>(npObject);
     LocalDOMWindow* window = object->rootObject;
     if (!window || !window->isCurrentlyDisplayedInFrame())
         return 0;
     v8::HandleScope handleScope(isolate);
     v8::Local<v8::Context> context = toV8Context(object->rootObject->frame(), DOMWrapperWorld::mainWorld());
     return ScriptState::from(context);
 }
 
-static PassOwnPtr<v8::Local<v8::Value>[]> createValueListFromVariantArgs(const NPVariant* arguments, uint32_t argumentCount, NPObject* owner, v8::Isolate* isolate)
+static PassOwnPtr<v8::Local<v8::Value>[]> createValueListFromVariantArgs(v8::Isolate* isolate, const NPVariant* arguments, uint32_t argumentCount, NPObject* owner)
 {
     OwnPtr<v8::Local<v8::Value>[]> argv = adoptArrayPtr(new v8::Local<v8::Value>[argumentCount]);
     for (uint32_t index = 0; index < argumentCount; index++) {
         const NPVariant* arg = &arguments[index];
         argv[index] = convertNPVariantToV8Object(isolate, arg, owner);
     }
     return argv.release();
 }
 
 // Create an identifier (null terminated utf8 char*) from the NPIdentifier.
@@ skipping 154 matching lines @@
     if (functionObject->IsUndefined()) {
         VOID_TO_NPVARIANT(*result);
         return false;
     }
 
     LocalFrame* frame = v8NpObject->rootObject->frame();
     ASSERT(frame);
 
     // Call the function object.
     v8::Local<v8::Function> function = v8::Local<v8::Function>::Cast(functionObject);
-    OwnPtr<v8::Local<v8::Value>[]> argv = createValueListFromVariantArgs(arguments, argumentCount, npObject, isolate);
+    OwnPtr<v8::Local<v8::Value>[]> argv = createValueListFromVariantArgs(isolate, arguments, argumentCount, npObject);
     v8::Local<v8::Value> resultObject = frame->script().callFunction(function, v8Object, argumentCount, argv.get());
 
     // If we had an error, return false.  The spec is a little unclear here, but says "Returns true if the method was
     // successfully invoked".  If we get an error return value, was that successfully invoked?
     if (resultObject.IsEmpty())
         return false;
 
     convertV8ObjectToNPVariant(isolate, resultObject, npObject, result);
     return true;
 }
@@ skipping 28 matching lines @@
     v8::Local<v8::Object> functionObject = v8::Local<v8::Object>::New(isolate, v8NpObject->v8Object);
     if (!functionObject->IsFunction())
         return false;
 
     v8::Local<v8::Value> resultObject;
     v8::Local<v8::Function> function = v8::Local<v8::Function>::Cast(functionObject);
     if (!function->IsNull()) {
         LocalFrame* frame = v8NpObject->rootObject->frame();
         ASSERT(frame);
 
-        OwnPtr<v8::Local<v8::Value>[]> argv = createValueListFromVariantArgs(arguments, argumentCount, npObject, isolate);
+        OwnPtr<v8::Local<v8::Value>[]> argv = createValueListFromVariantArgs(isolate, arguments, argumentCount, npObject);
         resultObject = frame->script().callFunction(function, functionObject, argumentCount, argv.get());
     }
     // If we had an error, return false.  The spec is a little unclear here, but says "Returns true if the method was
     // successfully invoked".  If we get an error return value, was that successfully invoked?
     if (resultObject.IsEmpty())
         return false;
 
     convertV8ObjectToNPVariant(isolate, resultObject, npObject, result);
     return true;
 }
@@ skipping 267 matching lines @@
             return false;
 
         // Call the constructor.
         v8::Local<v8::Value> resultObject;
         v8::Local<v8::Function> ctor = v8::Local<v8::Function>::Cast(ctorObj);
         if (ctor->IsNull())
             return false;
 
         LocalFrame* frame = object->rootObject->frame();
         ASSERT(frame);
-        OwnPtr<v8::Local<v8::Value>[]> argv = createValueListFromVariantArgs(arguments, argumentCount, npObject, scriptState->isolate());
+        OwnPtr<v8::Local<v8::Value>[]> argv = createValueListFromVariantArgs(scriptState->isolate(), arguments, argumentCount, npObject);
         if (!V8ObjectConstructor::newInstanceInDocument(scriptState->isolate(), ctor, argumentCount, argv.get(), frame ? frame->document() : 0).ToLocal(&resultObject))
             return false;
 
         convertV8ObjectToNPVariant(scriptState->isolate(), resultObject, npObject, result);
         return true;
     }
 
     if (NP_CLASS_STRUCT_VERSION_HAS_CTOR(npObject->_class) && npObject->_class->construct)
         return npObject->_class->construct(npObject, arguments, argumentCount, result);
 
     return false;
 }