diff options
author | Steve Block <steveblock@google.com> | 2010-01-14 10:52:53 +0000 |
---|---|---|
committer | Steve Block <steveblock@google.com> | 2010-01-14 11:25:00 +0000 |
commit | 76d21c39487c7753f1207935b120f3d4b06bae8d (patch) | |
tree | d0f5e01375dcfc6a72b9dc1117e0d6f9e19f5dbc /WebCore | |
parent | 47e3d63efb94462bdbb00e737e1a3b213fed62ae (diff) | |
download | external_webkit-76d21c39487c7753f1207935b120f3d4b06bae8d.zip external_webkit-76d21c39487c7753f1207935b120f3d4b06bae8d.tar.gz external_webkit-76d21c39487c7753f1207935b120f3d4b06bae8d.tar.bz2 |
Update jni_runtime.cpp with the latest changes to appendClassName.
WebKit change http://trac.webkit.org/changeset/53193 updated appendClassName to use
JSC::StringBuilder, rather than JSC-specific types. This allows this method to be
used with V8 as well as JSC.
A clean cherry-pick of this change would require changes 52026, 52028, 52075 and
52329 to be cherry-picked also. To avoid this churn, we simply update jni_runtime.cpp
to reflect the version at change 53193.
Change-Id: I6602469aeea59d8146ab1e72f312888b77522406
Diffstat (limited to 'WebCore')
-rw-r--r-- | WebCore/bridge/jni/jni_runtime.cpp | 15 |
1 files changed, 10 insertions, 5 deletions
diff --git a/WebCore/bridge/jni/jni_runtime.cpp b/WebCore/bridge/jni/jni_runtime.cpp index be11981..7f9fb72 100644 --- a/WebCore/bridge/jni/jni_runtime.cpp +++ b/WebCore/bridge/jni/jni_runtime.cpp @@ -28,6 +28,8 @@ #if ENABLE(MAC_JAVA_BRIDGE) +#include "CString.h" +#include "StringBuilder.h" #include "jni_utility.h" #include "jni_utility_private.h" #include "runtime_array.h" @@ -47,6 +49,7 @@ using namespace JSC; using namespace JSC::Bindings; +using namespace WebCore; JavaParameter::JavaParameter (JNIEnv *env, jstring type) @@ -296,7 +299,7 @@ JavaMethod::~JavaMethod() // JNI method signatures use '/' between components of a class name, but // we get '.' between components from the reflection API. -static void appendClassName(UString& aString, const char* className) +static void appendClassName(StringBuilder& builder, const char* className) { ASSERT(JSLock::lockCount() > 0); @@ -309,9 +312,9 @@ static void appendClassName(UString& aString, const char* className) cp++; } - aString.append(result); + builder.append(result); - free (result); + free(result); } const char *JavaMethod::signature() const @@ -319,7 +322,8 @@ const char *JavaMethod::signature() const if (!_signature) { JSLock lock(SilenceAssertionsOnly); - UString signatureBuilder("("); + StringBuilder signatureBuilder; + signatureBuilder.append("("); for (int i = 0; i < _numParameters; i++) { JavaParameter* aParameter = parameterAt(i); JNIType _JNIType = aParameter->getJNIType(); @@ -346,7 +350,8 @@ const char *JavaMethod::signature() const } } - _signature = strdup(signatureBuilder.ascii()); + String signatureString = signatureBuilder.toString(); + _signature = strdup(signatureString.utf8().data()); } return _signature; |