Changeset 165719 in webkit for trunk/Source/JavaScriptCore/API/OpaqueJSString.cpp

Timestamp:

Mar 16, 2014, 11:15:51 PM (11 years ago)

Author:

[email protected]

Message:

REGRESSION(r165703): JSC tests crashing in StringImpl::destroy().
<​https://webkit.org/b/130304>

Reviewed by Anders Carlsson.

Unreviewed, restoring the old behavior of OpaqueJSString::identifier()
that doesn't put a potentially unwanted string into the Identifier table.

• API/OpaqueJSString.cpp:

(OpaqueJSString::identifier):

File:

• trunk/Source/JavaScriptCore/API/OpaqueJSString.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/API/OpaqueJSString.cpp

@@ -72 +72 @@
         return Identifier(Identifier::EmptyIdentifier);
 
-    return Identifier(vm, m_string);
+    if (m_string.is8Bit())
+        return Identifier(vm, m_string.characters8(), m_string.length());
+
+    return Identifier(vm, m_string.characters16(), m_string.length());
 }