diff options
| author | Steve Block <steveblock@google.com> | 2010-12-03 10:04:19 +0000 |
|---|---|---|
| committer | Steve Block <steveblock@google.com> | 2010-12-03 10:11:00 +0000 |
| commit | 50114931eb31b3ee57480bf3d0068891aa62571f (patch) | |
| tree | 8e56a7156865974345ccffd616bc5915d73d95eb | |
| parent | 8fe068b72b160f7bc81892e9c40bfa917aa132d1 (diff) | |
| download | external_webkit-50114931eb31b3ee57480bf3d0068891aa62571f.zip external_webkit-50114931eb31b3ee57480bf3d0068891aa62571f.tar.gz external_webkit-50114931eb31b3ee57480bf3d0068891aa62571f.tar.bz2 | |
Remove some temporary logging in the JNI bridge
This was added to try to track down a crashing bug but is no longer
required.
Bug: 2435360
Change-Id: I5e88a842e42cd89e96c7f2eea54e96a440f890ca
| -rw-r--r-- | WebCore/bridge/jni/jsc/JavaInstanceJSC.cpp | 13 | ||||
| -rw-r--r-- | WebCore/bridge/jni/v8/JavaInstanceV8.cpp | 13 |
2 files changed, 1 insertions, 25 deletions
diff --git a/WebCore/bridge/jni/jsc/JavaInstanceJSC.cpp b/WebCore/bridge/jni/jsc/JavaInstanceJSC.cpp index db57d2c..6332545 100644 --- a/WebCore/bridge/jni/jsc/JavaInstanceJSC.cpp +++ b/WebCore/bridge/jni/jsc/JavaInstanceJSC.cpp @@ -42,11 +42,6 @@ #include <runtime/Error.h> #include <runtime/JSLock.h> -#if PLATFORM(ANDROID) -#define LOG_TAG JavaInstanceJSC.cpp -#include <utils/Log.h> -#endif - using namespace JSC::Bindings; using namespace JSC; using namespace WebCore; @@ -367,10 +362,6 @@ JObjectWrapper::JObjectWrapper(jobject instance) : m_refCount(0) { ASSERT(instance); -#if PLATFORM(ANDROID) - if (!instance) - LOGE("Attempted to create JObjectWrapper for null object"); -#endif // Cache the JNIEnv used to get the global ref for this java instance. // It'll be used to delete the reference. @@ -381,11 +372,7 @@ JObjectWrapper::JObjectWrapper(jobject instance) LOG(LiveConnect, "JObjectWrapper ctor new global ref %p for %p", m_instance, instance); if (!m_instance) -#if PLATFORM(ANDROID) - LOGE("%s: could not get GlobalRef for %p\n", __PRETTY_FUNCTION__, instance); -#else LOG_ERROR("Could not get GlobalRef for %p", instance); -#endif } JObjectWrapper::~JObjectWrapper() diff --git a/WebCore/bridge/jni/v8/JavaInstanceV8.cpp b/WebCore/bridge/jni/v8/JavaInstanceV8.cpp index dd4750f..27adca3 100644 --- a/WebCore/bridge/jni/v8/JavaInstanceV8.cpp +++ b/WebCore/bridge/jni/v8/JavaInstanceV8.cpp @@ -33,11 +33,6 @@ #include <assert.h> -// ANDROID -#define LOG_TAG "v8binding" -#include <utils/Log.h> -// END ANDROID - using namespace JSC::Bindings; JavaInstance::JavaInstance(jobject instance) @@ -161,10 +156,6 @@ JObjectWrapper::JObjectWrapper(jobject instance) : m_refCount(0) { assert(instance); -// ANDROID - if (!instance) - LOGE("Attempted to create JObjectWrapper for null object"); -// END ANDROID // Cache the JNIEnv used to get the global ref for this java instanace. // It'll be used to delete the reference. @@ -173,9 +164,7 @@ JObjectWrapper::JObjectWrapper(jobject instance) m_instance = m_env->NewGlobalRef(instance); if (!m_instance) -// ANDROID - LOGE("%s: could not get GlobalRef for %p\n", __PRETTY_FUNCTION__, instance); -// END ANDROID + fprintf(stderr, "%s: could not get GlobalRef for %p\n", __PRETTY_FUNCTION__, instance); } JObjectWrapper::~JObjectWrapper() |