diff options
author | Steve Block <steveblock@google.com> | 2011-07-20 15:46:25 +0100 |
---|---|---|
committer | Steve Block <steveblock@google.com> | 2011-07-20 15:49:07 +0100 |
commit | 613e4f1d251f223daace8a7450e13eac83198ccd (patch) | |
tree | 82838a18e710fa3a2cb604ec2e61d58fdee9e6f7 /Source/WebCore/bridge/jni | |
parent | d2c2030045e0e26787ff8b9935d449e3be8a71cc (diff) | |
download | external_webkit-613e4f1d251f223daace8a7450e13eac83198ccd.zip external_webkit-613e4f1d251f223daace8a7450e13eac83198ccd.tar.gz external_webkit-613e4f1d251f223daace8a7450e13eac83198ccd.tar.bz2 |
Update JavaMethodJobject with upstreamed version of fix
See https://bugs.webkit.org/show_bug.cgi?id=62389
Bug: 4585284
Change-Id: Ib99f9ff5d3b552fb7a40b00324f3e96ffc7e3e23
Diffstat (limited to 'Source/WebCore/bridge/jni')
-rw-r--r-- | Source/WebCore/bridge/jni/JavaMethodJobject.cpp | 20 |
1 files changed, 8 insertions, 12 deletions
diff --git a/Source/WebCore/bridge/jni/JavaMethodJobject.cpp b/Source/WebCore/bridge/jni/JavaMethodJobject.cpp index 5bb873d..73cae54 100644 --- a/Source/WebCore/bridge/jni/JavaMethodJobject.cpp +++ b/Source/WebCore/bridge/jni/JavaMethodJobject.cpp @@ -130,13 +130,11 @@ const char* JavaMethodJobject::signature() const appendClassName(signatureBuilder, javaClassName.data()); else { signatureBuilder.append(signatureFromJavaType(type)); -#if PLATFORM(ANDROID) && USE(V8) - // FIXME: Upstream to WebKit - // https://bugs.webkit.org/show_bug.cgi?id=62389 - if (type == JavaTypeObject || type == JavaTypeString) { -#else - if (type == JavaTypeObject) { + if (type == JavaTypeObject +#if USE(V8) + || type == JavaTypeString #endif + ) { appendClassName(signatureBuilder, javaClassName.data()); signatureBuilder.append(';'); } @@ -149,13 +147,11 @@ const char* JavaMethodJobject::signature() const appendClassName(signatureBuilder, returnType); else { signatureBuilder.append(signatureFromJavaType(m_returnType)); -#if PLATFORM(ANDROID) && USE(V8) - // FIXME: Upstream to WebKit - // https://bugs.webkit.org/show_bug.cgi?id=62389 - if (m_returnType == JavaTypeObject || m_returnType == JavaTypeString) { -#else - if (m_returnType == JavaTypeObject) { + if (m_returnType == JavaTypeObject +#if USE(V8) + || m_returnType == JavaTypeString #endif + ) { appendClassName(signatureBuilder, returnType); signatureBuilder.append(';'); } |