WIP: Introduce CompressibleString

third_party/WebKit/Source/bindings/core/v8/V8ValueCache.cpp

Index: third_party/WebKit/Source/bindings/core/v8/V8ValueCache.cpp
diff --git a/third_party/WebKit/Source/bindings/core/v8/V8ValueCache.cpp b/third_party/WebKit/Source/bindings/core/v8/V8ValueCache.cpp
index afb0dde5a7098e3586512891f6b6fafd91578856..c66ea695549116be1439b98985f74f21c565f817 100644
--- a/third_party/WebKit/Source/bindings/core/v8/V8ValueCache.cpp
+++ b/third_party/WebKit/Source/bindings/core/v8/V8ValueCache.cpp
@@ -55,6 +55,29 @@ void StringCacheMapTraits::OnWeakCallback(const v8::WeakCallbackInfo<WeakCallbac
     V8PerIsolateData::from(data.GetIsolate())->stringCache()->InvalidateLastString();
 }
 
+
+CompressibleStringCacheMapTraits::MapType* CompressibleStringCacheMapTraits::MapFromWeakCallbackInfo(
+    const v8::WeakCallbackInfo<WeakCallbackDataType>& data)
+{
+    return &(V8PerIsolateData::from(data.GetIsolate())->stringCache()->m_compressibleStringCache);
+}
+
+void CompressibleStringCacheMapTraits::Dispose(
+    v8::Isolate* isolate, v8::Global<v8::String> value, CompressibleStringImpl* key)
+{
+    key->deref();
+}
+
+void CompressibleStringCacheMapTraits::DisposeWeak(const v8::WeakCallbackInfo<WeakCallbackDataType>& data)
+{
+    data.GetParameter()->deref();
+}
+
+void CompressibleStringCacheMapTraits::OnWeakCallback(const v8::WeakCallbackInfo<WeakCallbackDataType>& data)
+{
+}
+
+
 void StringCache::dispose()
 {
     // The MapType::Dispose callback calls StringCache::InvalidateLastString,
@@ -84,6 +107,27 @@ static v8::Local<v8::String> makeExternalString(v8::Isolate* isolate, const Stri
     return newString;
 }
 
+static v8::Local<v8::String> makeExternalString(v8::Isolate* isolate, const CompressibleString& string)
+{
+    if (string.is8Bit()) {
+        WebCoreCompressibleStringResource8* stringResource = new WebCoreCompressibleStringResource8(string);
+        v8::Local<v8::String> newString;
+        if (!v8::String::NewExternalOneByte(isolate, stringResource).ToLocal(&newString)) {
+            delete stringResource;
+            return v8::String::Empty(isolate);
+        }
+        return newString;
+    }
+
+    WebCoreCompressibleStringResource16* stringResource = new WebCoreCompressibleStringResource16(string);
+    v8::Local<v8::String> newString;
+    if (!v8::String::NewExternalTwoByte(isolate, stringResource).ToLocal(&newString)) {
+        delete stringResource;
+        return v8::String::Empty(isolate);
+    }
+    return newString;
+}
+
 v8::Local<v8::String> StringCache::v8ExternalStringSlow(v8::Isolate* isolate, StringImpl* stringImpl)
 {
     if (!stringImpl->length())
@@ -99,6 +143,14 @@ v8::Local<v8::String> StringCache::v8ExternalStringSlow(v8::Isolate* isolate, St
     return createStringAndInsertIntoCache(isolate, stringImpl);
 }
 
+v8::Local<v8::String> StringCache::v8ExternalStringSlow(v8::Isolate* isolate, const CompressibleString& string)
+{
+    if (!string.length())
+        return v8::String::Empty(isolate);
+
+    return createStringAndInsertIntoCache(isolate, string);
+}
+
 void StringCache::setReturnValueFromStringSlow(v8::ReturnValue<v8::Value> returnValue, StringImpl* stringImpl)
 {
     if (!stringImpl->length()) {
@@ -136,6 +188,27 @@ v8::Local<v8::String> StringCache::createStringAndInsertIntoCache(v8::Isolate* i
     return newString;
 }
 
+v8::Local<v8::String> StringCache::createStringAndInsertIntoCache(v8::Isolate* isolate, const CompressibleString& string)

[haraken, 2015/11/26 11:50:03]

The implementation of the StringCache looks correct.

[hajimehoshi, 2015/11/27 11:03:58]

Thanks.

+{
+    CompressibleStringImpl* stringImpl = string.impl().get();
+
+    ASSERT(!m_compressibleStringCache.Contains(stringImpl));
+    ASSERT(stringImpl->originalLength());
+
+    v8::Local<v8::String> newString = makeExternalString(isolate, string);
+    ASSERT(!newString.IsEmpty());
+    ASSERT(newString->Length());
+
+    v8::UniquePersistent<v8::String> wrapper(isolate, newString);
+
+    stringImpl->ref();
+    wrapper.MarkIndependent();
+    CompressibleStringCacheMapTraits::MapType::PersistentValueReference unused;
+    m_compressibleStringCache.Set(stringImpl, wrapper.Pass(), &unused);
+
+    return newString;
+}
+
 void StringCache::InvalidateLastString()
 {
     m_lastStringImpl = nullptr;