Remove the WebIconDatabase thread attachment since ThreadingPthreads does it.

Also, delete threadData if pthread_create fails. This is a rare occurance but we
don't want to leak memory just in case.

1 files changed, 2 insertions, 19 deletions

diff --git a/WebKit/android/jni/WebIconDatabase.cpp b/WebKit/android/jni/WebIconDatabase.cpp
index c982c78..daf0334 100644
--- a/WebKit/android/jni/WebIconDatabase.cpp
+++ b/WebKit/android/jni/WebIconDatabase.cpp
@@ -71,28 +71,11 @@ void WebIconDatabase::dispatchDidAddIconForPageURL(const WebCore::String& pageUR
     if (mClients.size() == 0)
         return;
 
-    // Attempt to attach to the current vm.
-    JavaVM* vm = JSC::Bindings::getJavaVM();
-    JavaVMAttachArgs args;
-
-    args.version = JNI_VERSION_1_4;
-    args.name = "IconDatabase";
-    args.group = NULL;
-
-    JNIEnv* env;
-    bool threadIsAttached = true;
-    if (vm->AttachCurrentThread(&env, (void*) &args) != JNI_OK) {
-        LOGE("Could not attach IconDatabase thread to the VM");
-        threadIsAttached = false;
-    }
-
     mNotificationsMutex.lock();
     mNotifications.append(pageURL);
     if (!mDeliveryRequested) {
-        if (threadIsAttached) {
-            mDeliveryRequested = true;
-            JavaSharedClient::EnqueueFunctionPtr(DeliverNotifications, this);
-        }
+        mDeliveryRequested = true;
+        JavaSharedClient::EnqueueFunctionPtr(DeliverNotifications, this);
     }
     mNotificationsMutex.unlock();
 }