DatabaseBackend should not post a task to an already terminated database thread

Source/modules/webdatabase/DatabaseThread.cpp

Index: Source/modules/webdatabase/DatabaseThread.cpp
diff --git a/Source/modules/webdatabase/DatabaseThread.cpp b/Source/modules/webdatabase/DatabaseThread.cpp
index 92bd6faaafd082a2364a2d812cd30e194366c29b..9d36272f48254d600ca167d67e76b970e335e039 100644
--- a/Source/modules/webdatabase/DatabaseThread.cpp
+++ b/Source/modules/webdatabase/DatabaseThread.cpp
@@ -91,13 +91,8 @@ void DatabaseThread::requestTermination(TaskSynchronizer *cleanupSync)
     m_thread->postTask(new Task(WTF::bind(&DatabaseThread::cleanupDatabaseThread, this)));
 }
 
-bool DatabaseThread::terminationRequested(TaskSynchronizer* taskSynchronizer) const
+bool DatabaseThread::terminationRequested() const
 {
-#if ENABLE(ASSERT)
-    if (taskSynchronizer)
-        taskSynchronizer->setHasCheckedForTermination();
-#endif
-
     MutexLocker lock(m_terminationRequestedMutex);
     return m_terminationRequested;
 }
@@ -157,7 +152,7 @@ bool DatabaseThread::isDatabaseOpen(DatabaseBackend* database)
 void DatabaseThread::scheduleTask(PassOwnPtr<DatabaseTask> task)
 {
     ASSERT(m_thread);
-    ASSERT(!task->hasSynchronizer() || task->hasCheckedForTermination());
+    ASSERT(!terminationRequested());

[haraken, 2014/09/24 01:06:27]

This is a stronger (more straightforward) way to verify that the database thread
is not yet terminated at this point.

     // WebThread takes ownership of the task.
     m_thread->postTask(task.leakPtr());
 }