Oilpan: Move all supplements of Page, Document, and WorkerClients to the managed heap.

Source/web/DatabaseClientImpl.h

Index: Source/web/DatabaseClientImpl.h
diff --git a/Source/web/DatabaseClientImpl.h b/Source/web/DatabaseClientImpl.h
index 84279b1d5b475a54a3f2f5ff5d0bade763a21c02..a0960161a96664874b699c7dd6c7b9b1f6184777 100644
--- a/Source/web/DatabaseClientImpl.h
+++ b/Source/web/DatabaseClientImpl.h
@@ -36,9 +36,10 @@
 
 namespace blink {
 
-class DatabaseClientImpl FINAL : public WebCore::DatabaseClient {
+class DatabaseClientImpl FINAL : public NoBaseWillBeGarbageCollectedFinalized<DatabaseClientImpl>, public WebCore::DatabaseClient {

[haraken, 2014/04/30 02:41:22]

How about making DatabaseClient NoBaseWillBeGarbageCollectedFinalized, instead
of making DatabaseClientImpl NoBaseWillBeGarbageCollectedFinalized?

Your CL will work, but it would look nicer if DatabaseClient inherits from both
NoBaseWillBeGarbageCollectedFinalized and HeapSupplement.

[zerny-chromium, 2014/04/30 08:29:34]

We could do that, but I've placed the GC bases as the left-most leaf when
possible since that is the class that can actually be instantiated (eg, it has
the create method which connects with the base). Also, it is the most flexible
setup (eg, DatabaseClient can be used as a mixin to other GC'ed classes).

+    WILL_BE_USING_GARBAGE_COLLECTED_MIXIN(DatabaseClientImpl);
 public:
-    static PassOwnPtr<DatabaseClientImpl> create();
+    static PassOwnPtrWillBeRawPtr<DatabaseClientImpl> create();
 
     virtual ~DatabaseClientImpl();