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

Source/bindings/core/v8/V8ValueCache.cpp

 /*
  * Copyright (C) 2009 Google Inc. 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.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 36 matching lines @@
 
 
 StringCache::~StringCache()
 {
     // The MapType::Dispose callback calls StringCache::InvalidateLastString,
     // which will only work while the destructor has not yet finished. Thus,
     // we need to clear the map before the destructor has completed.
     m_stringCache.Clear();
 }
 
-static v8::Local<v8::String> makeExternalString(const String& string, v8::Isolate* isolate)
+static v8::Local<v8::String> makeExternalString(v8::Isolate* isolate, const String& string)
 {
     if (string.is8Bit()) {
         WebCoreStringResource8* stringResource = new WebCoreStringResource8(string);
         v8::Local<v8::String> newString;
         if (!v8::String::NewExternalOneByte(isolate, stringResource).ToLocal(&newString)) {
             delete stringResource;
             return v8::String::Empty(isolate);
         }
         return newString;
     }
@@ skipping 38 matching lines @@
     }
 
     returnValue.Set(createStringAndInsertIntoCache(returnValue.GetIsolate(), stringImpl));
 }
 
 v8::Local<v8::String> StringCache::createStringAndInsertIntoCache(v8::Isolate* isolate, StringImpl* stringImpl)
 {
     ASSERT(!m_stringCache.Contains(stringImpl));
     ASSERT(stringImpl->length());
 
-    v8::Local<v8::String> newString = makeExternalString(String(stringImpl), isolate);
+    v8::Local<v8::String> newString = makeExternalString(isolate, String(stringImpl));
     ASSERT(!newString.IsEmpty());
     ASSERT(newString->Length());
 
     v8::UniquePersistent<v8::String> wrapper(isolate, newString);
 
     stringImpl->ref();
     wrapper.MarkIndependent();
     m_stringCache.Set(stringImpl, wrapper.Pass(), &m_lastV8String);
     m_lastStringImpl = stringImpl;
 
     return newString;
 }
 
 void StringCache::InvalidateLastString()
 {
     m_lastStringImpl = nullptr;
     m_lastV8String.Reset();
 }
 
 } // namespace blink