Commit 96013927 authored by steveblock@google.com's avatar steveblock@google.com

REGRESSION (r82194): jvalueToJavaValue() does not correctly set length of String property

https://bugs.webkit.org/show_bug.cgi?id=64730

Reviewed by Alexey Proskuryakov.

Covered by existing tests.

* bridge/jni/v8/JNIUtilityPrivate.cpp:
(JSC::Bindings::jvalueToJavaValue):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91260 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 34866095
2011-07-19 Steve Block <steveblock@google.com>
REGRESSION (r82194): jvalueToJavaValue() does not correctly set length of String property
https://bugs.webkit.org/show_bug.cgi?id=64730
Reviewed by Alexey Proskuryakov.
Covered by existing tests.
* bridge/jni/v8/JNIUtilityPrivate.cpp:
(JSC::Bindings::jvalueToJavaValue):
2011-07-19 Vitaly Repeshko <vitalyr@chromium.org>
[V8] Don't put CSSRuleLists in object groups.
......@@ -247,10 +247,10 @@ JavaValue jvalueToJavaValue(const jvalue& value, const JavaType& type)
case JavaTypeString:
{
jstring javaString = static_cast<jstring>(value.l);
const UChar* a = getUCharactersFromJStringInEnv(getJNIEnv(), javaString);
const UChar* characters = getUCharactersFromJStringInEnv(getJNIEnv(), javaString);
// We take a copy to allow the Java String to be released.
result.m_stringValue = String(a).threadsafeCopy();
releaseUCharactersForJStringInEnv(getJNIEnv(), javaString, a);
result.m_stringValue = String(characters, getJNIEnv()->GetStringLength(javaString));
releaseUCharactersForJStringInEnv(getJNIEnv(), javaString, characters);
}
break;
case JavaTypeBoolean:
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment