Use NewScriptState in MIDIAccessResolver, CustomElementLifecycleCallbacks and V8MutationCallbacks

Source/bindings/v8/V8CustomElementLifecycleCallbacks.cpp

 /*
  * 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 74 matching lines @@
 
 template <typename T>
 static void weakCallback(const v8::WeakCallbackData<T, ScopedPersistent<T> >& data)
 {
     data.GetParameter()->clear();
 }
 
 V8CustomElementLifecycleCallbacks::V8CustomElementLifecycleCallbacks(ExecutionContext* executionContext, v8::Handle<v8::Object> prototype, v8::Handle<v8::Function> created, v8::Handle<v8::Function> attached, v8::Handle<v8::Function> detached, v8::Handle<v8::Function> attributeChanged)
     : CustomElementLifecycleCallbacks(flagSet(attached, detached, attributeChanged))
     , ContextLifecycleObserver(executionContext)
-    , m_isolate(toIsolate(executionContext))
     , m_owner(0)
-    , m_world(DOMWrapperWorld::current(m_isolate))
-    , m_prototype(m_isolate, prototype)
-    , m_created(m_isolate, created)
-    , m_attached(m_isolate, attached)
-    , m_detached(m_isolate, detached)
-    , m_attributeChanged(m_isolate, attributeChanged)
+    , m_scriptState(NewScriptState::current(toIsolate(executionContext)))
+    , m_prototype(m_scriptState->isolate(), prototype)
+    , m_created(m_scriptState->isolate(), created)
+    , m_attached(m_scriptState->isolate(), attached)
+    , m_detached(m_scriptState->isolate(), detached)
+    , m_attributeChanged(m_scriptState->isolate(), attributeChanged)
 {
     m_prototype.setWeak(&m_prototype, weakCallback<v8::Object>);
 
 #define MAKE_WEAK(Var, _) \
     if (!m_##Var.isEmpty()) \
         m_##Var.setWeak(&m_##Var, weakCallback<v8::Function>);
 
     CALLBACK_LIST(MAKE_WEAK)
 #undef MAKE_WEAK
 }
 
 V8PerContextData* V8CustomElementLifecycleCallbacks::creationContextData()
 {
     if (!executionContext())
         return 0;
 
-    v8::Handle<v8::Context> context = toV8Context(executionContext(), *m_world);
+    v8::Handle<v8::Context> context = m_scriptState->context();
     if (context.IsEmpty())
         return 0;
 
     return V8PerContextData::from(context);
 }
 
 V8CustomElementLifecycleCallbacks::~V8CustomElementLifecycleCallbacks()
 {
     if (!m_owner)
         return;
 
-    v8::HandleScope handleScope(m_isolate);
+    v8::HandleScope handleScope(m_scriptState->isolate());
     if (V8PerContextData* perContextData = creationContextData())
         perContextData->clearCustomElementBinding(m_owner);
 }
 
 bool V8CustomElementLifecycleCallbacks::setBinding(CustomElementDefinition* owner, PassOwnPtr<CustomElementBinding> binding)
 {
     ASSERT(!m_owner);
 
     V8PerContextData* perContextData = creationContextData();
     if (!perContextData)
@@ skipping 10 matching lines @@
 void V8CustomElementLifecycleCallbacks::created(Element* element)
 {
     // FIXME: callbacks while paused should be queued up for execution to
     // continue then be delivered in order rather than delivered immediately.
     // Bug 329665 tracks similar behavior for other synchronous events.
     if (!executionContext() || executionContext()->activeDOMObjectsAreStopped())
         return;
 
     element->setCustomElementState(Element::Upgraded);
 
-    v8::HandleScope handleScope(m_isolate);
-    v8::Handle<v8::Context> context = toV8Context(executionContext(), *m_world);
+    v8::Isolate* isolate = m_scriptState->isolate();
+    v8::HandleScope handleScope(isolate);
+    v8::Handle<v8::Context> context = m_scriptState->context();
     if (context.IsEmpty())
         return;
 
     v8::Context::Scope scope(context);
 
-    v8::Handle<v8::Object> receiver = DOMDataStore::current(m_isolate).get<V8Element>(element, m_isolate);
+    v8::Handle<v8::Object> receiver = DOMDataStore::current(isolate).get<V8Element>(element, isolate);
     if (!receiver.IsEmpty()) {
         // Swizzle the prototype of the existing wrapper. We don't need to
         // worry about non-existent wrappers; they will get the right
         // prototype when wrapped.
-        v8::Handle<v8::Object> prototype = m_prototype.newLocal(m_isolate);
+        v8::Handle<v8::Object> prototype = m_prototype.newLocal(isolate);
         if (prototype.IsEmpty())
             return;
         receiver->SetPrototype(prototype);
     }
 
-    v8::Handle<v8::Function> callback = m_created.newLocal(m_isolate);
+    v8::Handle<v8::Function> callback = m_created.newLocal(isolate);
     if (callback.IsEmpty())
         return;
 
     if (receiver.IsEmpty())
-        receiver = toV8(element, context->Global(), m_isolate).As<v8::Object>();
+        receiver = toV8(element, context->Global(), isolate).As<v8::Object>();
 
     ASSERT(!receiver.IsEmpty());
 
     InspectorInstrumentation::willExecuteCustomElementCallback(element);
 
     v8::TryCatch exceptionCatcher;
     exceptionCatcher.SetVerbose(true);
-    ScriptController::callFunction(executionContext(), callback, receiver, 0, 0, m_isolate);
+    ScriptController::callFunction(executionContext(), callback, receiver, 0, 0, isolate);
 }
 
 void V8CustomElementLifecycleCallbacks::attached(Element* element)
 {
     call(m_attached, element);
 }
 
 void V8CustomElementLifecycleCallbacks::detached(Element* element)
 {
     call(m_detached, element);
 }
 
 void V8CustomElementLifecycleCallbacks::attributeChanged(Element* element, const AtomicString& name, const AtomicString& oldValue, const AtomicString& newValue)
 {
     // FIXME: callbacks while paused should be queued up for execution to
     // continue then be delivered in order rather than delivered immediately.
     // Bug 329665 tracks similar behavior for other synchronous events.
     if (!executionContext() || executionContext()->activeDOMObjectsAreStopped())
         return;
 
-    v8::HandleScope handleScope(m_isolate);
-    v8::Handle<v8::Context> context = toV8Context(executionContext(), *m_world);
+    v8::Isolate* isolate = m_scriptState->isolate();
+    v8::HandleScope handleScope(isolate);
+    v8::Handle<v8::Context> context = m_scriptState->context();
     if (context.IsEmpty())
         return;
 
     v8::Context::Scope scope(context);
 
-    v8::Handle<v8::Object> receiver = toV8(element, context->Global(), m_isolate).As<v8::Object>();
+    v8::Handle<v8::Object> receiver = toV8(element, context->Global(), isolate).As<v8::Object>();
     ASSERT(!receiver.IsEmpty());
 
-    v8::Handle<v8::Function> callback = m_attributeChanged.newLocal(m_isolate);
+    v8::Handle<v8::Function> callback = m_attributeChanged.newLocal(isolate);
     if (callback.IsEmpty())
         return;
 
     v8::Handle<v8::Value> argv[] = {
-        v8String(m_isolate, name),
-        oldValue.isNull() ? v8::Handle<v8::Value>(v8::Null(m_isolate)) : v8::Handle<v8::Value>(v8String(m_isolate, oldValue)),
-        newValue.isNull() ? v8::Handle<v8::Value>(v8::Null(m_isolate)) : v8::Handle<v8::Value>(v8String(m_isolate, newValue))
+        v8String(isolate, name),
+        oldValue.isNull() ? v8::Handle<v8::Value>(v8::Null(isolate)) : v8::Handle<v8::Value>(v8String(isolate, oldValue)),
+        newValue.isNull() ? v8::Handle<v8::Value>(v8::Null(isolate)) : v8::Handle<v8::Value>(v8String(isolate, newValue))
     };
 
     InspectorInstrumentation::willExecuteCustomElementCallback(element);
 
     v8::TryCatch exceptionCatcher;
     exceptionCatcher.SetVerbose(true);
-    ScriptController::callFunction(executionContext(), callback, receiver, WTF_ARRAY_LENGTH(argv), argv, m_isolate);
+    ScriptController::callFunction(executionContext(), callback, receiver, WTF_ARRAY_LENGTH(argv), argv, isolate);
 }
 
 void V8CustomElementLifecycleCallbacks::call(const ScopedPersistent<v8::Function>& weakCallback, Element* element)
 {
     // FIXME: callbacks while paused should be queued up for execution to
     // continue then be delivered in order rather than delivered immediately.
     // Bug 329665 tracks similar behavior for other synchronous events.
     if (!executionContext() || executionContext()->activeDOMObjectsAreStopped())
         return;
 
-    v8::HandleScope handleScope(m_isolate);
-    v8::Handle<v8::Context> context = toV8Context(executionContext(), *m_world);
+    v8::Isolate* isolate = m_scriptState->isolate();
+    v8::HandleScope handleScope(isolate);
+    v8::Handle<v8::Context> context = m_scriptState->context();
     if (context.IsEmpty())
         return;
 
     v8::Context::Scope scope(context);
 
-    v8::Handle<v8::Function> callback = weakCallback.newLocal(m_isolate);
+    v8::Handle<v8::Function> callback = weakCallback.newLocal(isolate);
     if (callback.IsEmpty())
         return;
 
-    v8::Handle<v8::Object> receiver = toV8(element, context->Global(), m_isolate).As<v8::Object>();
+    v8::Handle<v8::Object> receiver = toV8(element, context->Global(), isolate).As<v8::Object>();
     ASSERT(!receiver.IsEmpty());
 
     InspectorInstrumentation::willExecuteCustomElementCallback(element);
 
     v8::TryCatch exceptionCatcher;
     exceptionCatcher.SetVerbose(true);
-    ScriptController::callFunction(executionContext(), callback, receiver, 0, 0, m_isolate);
+    ScriptController::callFunction(executionContext(), callback, receiver, 0, 0, isolate);
 }
 
 } // namespace WebCore