Changeset 43392 in webkit for trunk/WebCore/storage/DatabaseThread.cpp

Timestamp:

May 7, 2009, 11:47:19 PM (16 years ago)

Author:

[email protected]

Message:

Fix <​https://bugs.webkit.org/show_bug.cgi?id=25640>.
Bug 25640: Crash on quit in r43384 nightly build on Leopard w/ Safari 4 beta installed

Rubber-stamped by Oliver Hunt.

Roll out r43366 as it removed symbols that Safari 4 Beta uses.

File:

• trunk/WebCore/storage/DatabaseThread.cpp (modified) (3 diffs)

trunk/WebCore/storage/DatabaseThread.cpp

@@ -40 +40 @@
 
 DatabaseThread::DatabaseThread()
+    : m_threadID(0)
 {
     m_selfRef = this;
@@ -53 +54 @@
     MutexLocker lock(m_threadCreationMutex);
 
-    if (m_threadID.isValid())
+    if (m_threadID)
         return true;
 
     m_threadID = createThread(DatabaseThread::databaseThreadStart, this, "WebCore: Database");
 
-    return m_threadID.isValid();
+    return m_threadID;
 }
 
@@ -97 +98 @@
     }
 
-    LOG(StorageAPI, "About to detach thread and clear the ref to DatabaseThread %p, which currently has %i ref(s)", this, refCount());
+    LOG(StorageAPI, "About to detach thread %i and clear the ref to DatabaseThread %p, which currently has %i ref(s)", m_threadID, this, refCount());
 
     // Detach the thread so its resources are no longer of any concern to anyone else