Switch blink to use a gin-managed isolate.

Source/bindings/core/v8/V8PerIsolateData.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 43 matching lines @@
 {
     switch (feature) {
     case v8::Isolate::kUseAsm:
         UseCounter::count(currentExecutionContext(isolate), UseCounter::UseAsm);
         break;
     default:
         ASSERT_NOT_REACHED();
     }
 }
 
-V8PerIsolateData::V8PerIsolateData(v8::Isolate* isolate)
-    : m_isolate(isolate)
-    , m_isolateHolder(adoptPtr(new gin::IsolateHolder(m_isolate, v8ArrayBufferAllocator())))
-    , m_stringCache(adoptPtr(new StringCache(m_isolate)))
+V8PerIsolateData::V8PerIsolateData()
+    : m_isolateHolder(adoptPtr(new gin::IsolateHolder()))
+    , m_stringCache(adoptPtr(new StringCache(m_isolateHolder->isolate())))

[haraken, 2014/09/11 09:07:06]

m_isolateHolder->isolate() => isolate() ?

     , m_hiddenValue(adoptPtr(new V8HiddenValue()))
     , m_constructorMode(ConstructorMode::CreateNewObject)
     , m_recursionLevel(0)
     , m_isHandlingRecursionLevelError(false)
 #if ENABLE(ASSERT)
     , m_internalScriptRecursionLevel(0)
 #endif
     , m_gcEventData(adoptPtr(new GCEventData()))
     , m_performingMicrotaskCheckpoint(false)
 {
+    // FIXME: Remove once all v8::Isolate::GetCurrent() calls are gone.
+    isolate()->Enter();
 #if ENABLE(ASSERT)
     // currentThread will always be non-null in production, but can be null in Chromium unit tests.
     if (blink::Platform::current()->currentThread())
-        isolate->AddCallCompletedCallback(&assertV8RecursionScope);
+        isolate()->AddCallCompletedCallback(&assertV8RecursionScope);
 #endif
     if (isMainThread()) {
         mainThreadPerIsolateData = this;
-        PageScriptDebugServer::setMainThreadIsolate(isolate);
+        PageScriptDebugServer::setMainThreadIsolate(isolate());
     }
-    isolate->SetUseCounterCallback(&useCounterCallback);
+    isolate()->SetUseCounterCallback(&useCounterCallback);
 }
 
 V8PerIsolateData::~V8PerIsolateData()
 {
     if (m_scriptRegexpScriptState)
         m_scriptRegexpScriptState->disposePerContextData();
     if (isMainThread())
         mainThreadPerIsolateData = 0;
 }
 
 v8::Isolate* V8PerIsolateData::mainThreadIsolate()
 {
     ASSERT(isMainThread());
     ASSERT(mainThreadPerIsolateData);
     return mainThreadPerIsolateData->isolate();
 }
 
-void V8PerIsolateData::ensureInitialized(v8::Isolate* isolate)
+v8::Isolate* V8PerIsolateData::initialize()
 {
-    ASSERT(isolate);
-    if (!isolate->GetData(gin::kEmbedderBlink)) {
-        V8PerIsolateData* data = new V8PerIsolateData(isolate);
-        isolate->SetData(gin::kEmbedderBlink, data);
-    }
+    V8PerIsolateData* data = new V8PerIsolateData();
+    v8::Isolate* isolate = data->isolate();
+    isolate->SetData(gin::kEmbedderBlink, data);
+    return isolate;
 }
 
 v8::Persistent<v8::Value>& V8PerIsolateData::ensureLiveRoot()
 {
     if (m_liveRoot.isEmpty())
-        m_liveRoot.set(m_isolate, v8::Null(m_isolate));
+        m_liveRoot.set(isolate(), v8::Null(isolate()));
     return m_liveRoot.getUnsafe();
 }
 
 void V8PerIsolateData::dispose(v8::Isolate* isolate)
 {
 #if ENABLE(ASSERT)
     if (blink::Platform::current()->currentThread())
         isolate->RemoveCallCompletedCallback(&assertV8RecursionScope);
 #endif
     void* data = isolate->GetData(gin::kEmbedderBlink);
+    // FIXME: Remove once all v8::Isolate::GetCurrent() calls are gone.
+    isolate->Exit();

[haraken, 2014/09/11 09:07:06]

Just help me understand: What's the relationship between Isolate::GetCurrent and
Isolate::Enter/Exit?

     delete static_cast<V8PerIsolateData*>(data);
-    isolate->SetData(gin::kEmbedderBlink, 0);

[haraken, 2014/09/11 09:07:06]

Why did you remove this? I guess it would be better to clear the data just in
case.

 }
 
 V8PerIsolateData::DOMTemplateMap& V8PerIsolateData::currentDOMTemplateMap()
 {
-    if (DOMWrapperWorld::current(m_isolate).isMainWorld())
+    if (DOMWrapperWorld::current(isolate()).isMainWorld())
         return m_domTemplateMapForMainWorld;
     return m_domTemplateMapForNonMainWorld;
 }
 
 v8::Handle<v8::FunctionTemplate> V8PerIsolateData::domTemplate(void* domTemplateKey, v8::FunctionCallback callback, v8::Handle<v8::Value> data, v8::Handle<v8::Signature> signature, int length)
 {
     DOMTemplateMap& domTemplateMap = currentDOMTemplateMap();
     DOMTemplateMap::iterator result = domTemplateMap.find(domTemplateKey);
     if (result != domTemplateMap.end())
-        return result->value.Get(m_isolate);
+        return result->value.Get(isolate());
 
-    v8::Local<v8::FunctionTemplate> templ = v8::FunctionTemplate::New(m_isolate, callback, data, signature, length);
-    domTemplateMap.add(domTemplateKey, v8::Eternal<v8::FunctionTemplate>(m_isolate, templ));
+    v8::Local<v8::FunctionTemplate> templ = v8::FunctionTemplate::New(isolate(), callback, data, signature, length);
+    domTemplateMap.add(domTemplateKey, v8::Eternal<v8::FunctionTemplate>(isolate(), templ));
     return templ;
 }
 
 v8::Handle<v8::FunctionTemplate> V8PerIsolateData::existingDOMTemplate(void* domTemplateKey)
 {
     DOMTemplateMap& domTemplateMap = currentDOMTemplateMap();
     DOMTemplateMap::iterator result = domTemplateMap.find(domTemplateKey);
     if (result != domTemplateMap.end())
-        return result->value.Get(m_isolate);
+        return result->value.Get(isolate());
     return v8::Local<v8::FunctionTemplate>();
 }
 
 void V8PerIsolateData::setDOMTemplate(void* domTemplateKey, v8::Handle<v8::FunctionTemplate> templ)
 {
-    currentDOMTemplateMap().add(domTemplateKey, v8::Eternal<v8::FunctionTemplate>(m_isolate, v8::Local<v8::FunctionTemplate>(templ)));
+    currentDOMTemplateMap().add(domTemplateKey, v8::Eternal<v8::FunctionTemplate>(isolate(), v8::Local<v8::FunctionTemplate>(templ)));
 }
 
 v8::Local<v8::Context> V8PerIsolateData::ensureScriptRegexpContext()
 {
     if (!m_scriptRegexpScriptState) {
-        v8::Local<v8::Context> context(v8::Context::New(m_isolate));
+        v8::Local<v8::Context> context(v8::Context::New(isolate()));
         m_scriptRegexpScriptState = ScriptState::create(context, DOMWrapperWorld::create());
     }
     return m_scriptRegexpScriptState->context();
 }
 
 bool V8PerIsolateData::hasInstance(const WrapperTypeInfo* info, v8::Handle<v8::Value> value)
 {
     return hasInstance(info, value, m_domTemplateMapForMainWorld)
         || hasInstance(info, value, m_domTemplateMapForNonMainWorld);
 }
 
 bool V8PerIsolateData::hasInstance(const WrapperTypeInfo* info, v8::Handle<v8::Value> value, DOMTemplateMap& domTemplateMap)
 {
     DOMTemplateMap::iterator result = domTemplateMap.find(info);
     if (result == domTemplateMap.end())
         return false;
-    v8::Handle<v8::FunctionTemplate> templ = result->value.Get(m_isolate);
+    v8::Handle<v8::FunctionTemplate> templ = result->value.Get(isolate());
     return templ->HasInstance(value);
 }
 
 v8::Handle<v8::Object> V8PerIsolateData::findInstanceInPrototypeChain(const WrapperTypeInfo* info, v8::Handle<v8::Value> value)
 {
     v8::Handle<v8::Object> wrapper = findInstanceInPrototypeChain(info, value, m_domTemplateMapForMainWorld);
     if (!wrapper.IsEmpty())
         return wrapper;
     return findInstanceInPrototypeChain(info, value, m_domTemplateMapForNonMainWorld);
 }
 
 v8::Handle<v8::Object> V8PerIsolateData::findInstanceInPrototypeChain(const WrapperTypeInfo* info, v8::Handle<v8::Value> value, DOMTemplateMap& domTemplateMap)
 {
     if (value.IsEmpty() || !value->IsObject())
         return v8::Handle<v8::Object>();
     DOMTemplateMap::iterator result = domTemplateMap.find(info);
     if (result == domTemplateMap.end())
         return v8::Handle<v8::Object>();
-    v8::Handle<v8::FunctionTemplate> templ = result->value.Get(m_isolate);
+    v8::Handle<v8::FunctionTemplate> templ = result->value.Get(isolate());
     return v8::Handle<v8::Object>::Cast(value)->FindInstanceInPrototypeChain(templ);
 }
 
 static void constructorOfToString(const v8::FunctionCallbackInfo<v8::Value>& info)
 {
     // The DOM constructors' toString functions grab the current toString
     // for Functions by taking the toString function of itself and then
     // calling it with the constructor as its receiver. This means that
     // changes to the Function prototype chain or toString function are
     // reflected when printing DOM constructors. The only wart is that
     // changes to a DOM constructor's toString's toString will cause the
     // toString of the DOM constructor itself to change. This is extremely
     // obscure and unlikely to be a problem.
     v8::Handle<v8::Value> value = info.Callee()->Get(v8AtomicString(info.GetIsolate(), "toString"));
     if (!value->IsFunction()) {
         v8SetReturnValue(info, v8::String::Empty(info.GetIsolate()));
         return;
     }
     v8SetReturnValue(info, V8ScriptRunner::callInternalFunction(v8::Handle<v8::Function>::Cast(value), info.This(), 0, 0, info.GetIsolate()));
 }
 
 v8::Handle<v8::FunctionTemplate> V8PerIsolateData::toStringTemplate()
 {
     if (m_toStringTemplate.isEmpty())
-        m_toStringTemplate.set(m_isolate, v8::FunctionTemplate::New(m_isolate, constructorOfToString));
-    return m_toStringTemplate.newLocal(m_isolate);
+        m_toStringTemplate.set(isolate(), v8::FunctionTemplate::New(isolate(), constructorOfToString));
+    return m_toStringTemplate.newLocal(isolate());
 }
 
 } // namespace blink