Pass isolate to ScriptPromise and ScriptString constructors

Source/bindings/v8/ScriptPromise.h

 /*
  * Copyright (C) 2013 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 32 matching lines @@
 // So holding a ScriptPromise as a member variable in DOM object causes
 // memory leaks since it has a reference from C++ to V8.
 //
 class ScriptPromise {
 public:
     ScriptPromise()
         : m_promise()
     {
     }
 
-    explicit ScriptPromise(ScriptValue promise)
+    explicit ScriptPromise(const ScriptValue& promise)
         : m_promise(promise)
     {
         ASSERT(!m_promise.hasNoValue());
     }
 
-    // FIXME: This constructor should take an isolate.
-    explicit ScriptPromise(v8::Handle<v8::Value> promise)
-        : m_promise(promise, v8::Isolate::GetCurrent())
+    ScriptPromise(v8::Handle<v8::Value> promise, v8::Isolate* isolate)
+        : m_promise(promise, isolate)
     {
         ASSERT(!m_promise.hasNoValue());
     }
 
     bool isObject() const
     {
         return m_promise.isObject();
     }
 
     bool isNull() const
@@ skipping 22 matching lines @@
     }
 
 private:
     ScriptValue m_promise;
 };
 
 } // namespace WebCore
 
 
 #endif // ScriptPromise_h