Oilpan: Prepare to move AbstractSQLTransactionBackend, SQLTransactionBackend, and SQLTransactionCoo…

Source/heap/Handle.h

Index: Source/heap/Handle.h
diff --git a/Source/heap/Handle.h b/Source/heap/Handle.h
index 448f3f973144504d50f1099ba5b64889c7e57bed..20c629c0c16beea7e2a0ed4c51547584b94d5615 100644
--- a/Source/heap/Handle.h
+++ b/Source/heap/Handle.h
@@ -430,7 +430,6 @@ public:
     Member(T* raw) : m_raw(raw)
     {
         COMPILE_ASSERT_IS_GARBAGE_COLLECTED(T, NonGarbageCollectedObjectInMember);
-        ASSERT(!raw || ThreadStateFor<ThreadingTrait<T>::Affinity>::state()->contains(raw));

[tkent, 2014/03/19 04:46:50]

This assertion is too strict.
A SQLTransaction object is created in main/worker thread, and can be stored in a
Member<> in a database thread.

[Mads Ager (chromium), 2014/03/19 06:30:54]

Yes, that is true. I needed to relax the assert in the Persistent case for the
CrossThreadPersistents. We cannot do a similar relaxation here. What we would
have to check here is that the pointer is in one of the thread heaps. However,
we cannot do that unless we stop all threads to do it. :(

     }
 
     Member(WTF::HashTableDeletedValueType) : m_raw(reinterpret_cast<T*>(-1))