Upstream version 7.36.149.0

[platform/framework/web/crosswalk.git] / src / third_party / WebKit / Source / bindings / v8 / V8ValueCache.cpp

diff --git a/src/third_party/WebKit/Source/bindings/v8/V8ValueCache.cpp b/src/third_party/WebKit/Source/bindings/v8/V8ValueCache.cpp
index 77b0cbc..6d64a56 100644 (file)
--- a/src/third_party/WebKit/Source/bindings/v8/V8ValueCache.cpp
+++ b/src/third_party/WebKit/Source/bindings/v8/V8ValueCache.cpp
@@ -27,23 +27,45 @@
 #include "bindings/v8/V8ValueCache.h"
 
 #include "bindings/v8/V8Binding.h"
-#include "bindings/v8/V8Utilities.h"
 #include "wtf/text/StringHash.h"
 
 namespace WebCore {
 
-static v8::Local<v8::String> makeExternalString(const String& string)
+StringCacheMapTraits::MapType* StringCacheMapTraits::MapFromWeakCallbackData(
+    const v8::WeakCallbackData<v8::String, WeakCallbackDataType>& data)
+{
+    return &(V8PerIsolateData::from(data.GetIsolate())->stringCache()->m_stringCache);
+}
+
+
+void StringCacheMapTraits::Dispose(
+    v8::Isolate* isolate, v8::UniquePersistent<v8::String> value, StringImpl* key)
+{
+    V8PerIsolateData::from(isolate)->stringCache()->InvalidateLastString();
+    key->deref();
+}
+
+
+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)
 {
     if (string.is8Bit()) {
         WebCoreStringResource8* stringResource = new WebCoreStringResource8(string);
-        v8::Local<v8::String> newString = v8::String::NewExternal(stringResource);
+        v8::Local<v8::String> newString = v8::String::NewExternal(isolate, stringResource);
         if (newString.IsEmpty())
             delete stringResource;
         return newString;
     }
 
     WebCoreStringResource16* stringResource = new WebCoreStringResource16(string);
-    v8::Local<v8::String> newString = v8::String::NewExternal(stringResource);
+    v8::Local<v8::String> newString = v8::String::NewExternal(isolate, stringResource);
     if (newString.IsEmpty())
         delete stringResource;
     return newString;
@@ -54,11 +76,11 @@ v8::Handle<v8::String> StringCache::v8ExternalStringSlow(StringImpl* stringImpl,
     if (!stringImpl->length())
         return v8::String::Empty(isolate);
 
-    UnsafePersistent<v8::String> cachedV8String = m_stringCache.get(stringImpl);
-    if (!cachedV8String.isEmpty()) {
+    StringCacheMapTraits::MapType::PersistentValueReference cachedV8String = m_stringCache.GetReference(stringImpl);
+    if (!cachedV8String.IsEmpty()) {
         m_lastStringImpl = stringImpl;
         m_lastV8String = cachedV8String;
-        return cachedV8String.newLocal(isolate);
+        return m_lastV8String.NewLocal(isolate);
     }
 
     return createStringAndInsertIntoCache(stringImpl, isolate);
@@ -71,11 +93,11 @@ void StringCache::setReturnValueFromStringSlow(v8::ReturnValue<v8::Value> return
         return;
     }
 
-    UnsafePersistent<v8::String> cachedV8String = m_stringCache.get(stringImpl);
-    if (!cachedV8String.isEmpty()) {
+    StringCacheMapTraits::MapType::PersistentValueReference cachedV8String = m_stringCache.GetReference(stringImpl);
+    if (!cachedV8String.IsEmpty()) {
         m_lastStringImpl = stringImpl;
         m_lastV8String = cachedV8String;
-        returnValue.Set(*cachedV8String.persistent());
+        m_lastV8String.SetReturnValue(returnValue);
         return;
     }
 
@@ -84,34 +106,27 @@ void StringCache::setReturnValueFromStringSlow(v8::ReturnValue<v8::Value> return
 
 v8::Local<v8::String> StringCache::createStringAndInsertIntoCache(StringImpl* stringImpl, v8::Isolate* isolate)
 {
-    ASSERT(!m_stringCache.contains(stringImpl));
+    ASSERT(!m_stringCache.Contains(stringImpl));
     ASSERT(stringImpl->length());
 
-    v8::Local<v8::String> newString = makeExternalString(String(stringImpl));
+    v8::Local<v8::String> newString = makeExternalString(String(stringImpl), isolate);
     if (newString.IsEmpty())
         return newString;
 
-    v8::Persistent<v8::String> wrapper(isolate, newString);
+    v8::UniquePersistent<v8::String> wrapper(isolate, newString);
 
     stringImpl->ref();
     wrapper.MarkIndependent();
-    wrapper.MakeWeak(stringImpl, &makeWeakCallback);
-    m_lastV8String = UnsafePersistent<v8::String>(wrapper);
+    m_stringCache.Set(stringImpl, wrapper.Pass(), &m_lastV8String);
     m_lastStringImpl = stringImpl;
-    m_stringCache.set(stringImpl, m_lastV8String);
 
     return newString;
 }
 
-void StringCache::makeWeakCallback(v8::Isolate* isolate, v8::Persistent<v8::String>* wrapper, StringImpl* stringImpl)
+void StringCache::InvalidateLastString()
 {
-    StringCache* stringCache = V8PerIsolateData::from(isolate)->stringCache();
-    stringCache->m_lastStringImpl = 0;
-    stringCache->m_lastV8String.clear();
-    ASSERT(stringCache->m_stringCache.contains(stringImpl));
-    stringCache->m_stringCache.remove(stringImpl);
-    stringImpl->deref();
-    wrapper->Dispose();
+    m_lastStringImpl = nullptr;
+    m_lastV8String.Reset();
 }
 
 } // namespace WebCore