DOMDataStore should be cleared when shutting down worker threads

Source/bindings/v8/DOMWrapperWorld.h

Index: Source/bindings/v8/DOMWrapperWorld.h
diff --git a/Source/bindings/v8/DOMWrapperWorld.h b/Source/bindings/v8/DOMWrapperWorld.h
index da5e3efc5ba7c60a6013b79701e78c97bf50200c..4241b6cd424ecabe53e7144f174b7167e347f85e 100644
--- a/Source/bindings/v8/DOMWrapperWorld.h
+++ b/Source/bindings/v8/DOMWrapperWorld.h
@@ -63,9 +63,10 @@ public:
     static const int mainWorldExtensionGroup = 0;
     static PassRefPtr<DOMWrapperWorld> ensureIsolatedWorld(int worldId, int extensionGroup);
     ~DOMWrapperWorld();
+    void dispose();
 
     static bool isolatedWorldsExist() { return isolatedWorldCount; }
-    static void allWorldsInMainThread(Vector<RefPtr<DOMWrapperWorld> >& worlds);
+    static void allWorldsInMainThread(Vector<DOMWrapperWorld*>& worlds);

[dcarney, 2014/03/10 08:12:28]

i don't see what ensures the safety of this.  I think it's better just to leave
as RefPtr here, and if the user of this call needs a pointer, it can deref there

[haraken, 2014/03/10 08:23:57]

Fixed.

(Either is fine with me. Since current(), mainWorld() etc return raw pointers,
it won't matter whether allWorldsInMainThread returns raw pointers or RefPtrs.)

 
     static DOMWrapperWorld* world(v8::Handle<v8::Context> context)
     {