summaryrefslogtreecommitdiffstats
path: root/WebCore
diff options
context:
space:
mode:
authorSteve Block <steveblock@google.com>2010-01-14 10:52:53 +0000
committerSteve Block <steveblock@google.com>2010-01-14 11:25:00 +0000
commit76d21c39487c7753f1207935b120f3d4b06bae8d (patch)
treed0f5e01375dcfc6a72b9dc1117e0d6f9e19f5dbc /WebCore
parent47e3d63efb94462bdbb00e737e1a3b213fed62ae (diff)
downloadexternal_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.cpp15
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;