Oilpan: move WorkerGlobalScope to oilpan's heap.

Source/modules/crypto/WorkerGlobalScopeCrypto.h

Index: Source/modules/crypto/WorkerGlobalScopeCrypto.h
diff --git a/Source/modules/crypto/WorkerGlobalScopeCrypto.h b/Source/modules/crypto/WorkerGlobalScopeCrypto.h
index 9937bb6f4cea387cec74c642ef2813a3d39e5ca9..03ca68a1b94fbcb9f5932dc8e8e11ce4a819d910 100644
--- a/Source/modules/crypto/WorkerGlobalScopeCrypto.h
+++ b/Source/modules/crypto/WorkerGlobalScopeCrypto.h
@@ -31,6 +31,7 @@
 #ifndef WorkerGlobalScopeCrypto_h
 #define WorkerGlobalScopeCrypto_h
 
+#include "core/workers/WorkerGlobalScope.h"
 #include "core/workers/WorkerSupplementable.h"
 #include "heap/Handle.h"
 
@@ -39,18 +40,20 @@ namespace WebCore {
 class WorkerCrypto;
 class ExecutionContext;
 
-class WorkerGlobalScopeCrypto FINAL : public WorkerSupplement {
+class WorkerGlobalScopeCrypto FINAL : public NoBaseWillBeGarbageCollectedFinalized<WorkerGlobalScopeCrypto>, public WorkerSupplement {

[Mads Ager (chromium), 2014/02/26 13:09:08]

As far as I can tell we can get away with not using Finalized here. This is
Final, so there are no subclasses, Supplement doesn't have state, and this class
only has a member which doesn't require the destructor to be called.

[sof, 2014/02/26 14:36:25]

Yes, I've verified this (and for other supplements.) Nice.

 public:
     virtual ~WorkerGlobalScopeCrypto();
     static WorkerGlobalScopeCrypto& from(WorkerSupplementable&);
     static WorkerCrypto* crypto(WorkerSupplementable&);
     WorkerCrypto* crypto() const;
 
+    virtual void trace(Visitor*);
+
 private:
     WorkerGlobalScopeCrypto();
     static const char* supplementName();
 
-    mutable RefPtrWillBePersistent<WorkerCrypto> m_crypto;
+    mutable RefPtrWillBeMember<WorkerCrypto> m_crypto;
 };
 
 } // namespace WebCore