IndexedDB: Fixed threading bugs with use of AtomicStrings.

Source/core/Init.h

Index: Source/core/Init.h
diff --git a/Source/core/Init.h b/Source/core/Init.h
index 3579688d5c74865fd0be1d202dbd2ffad5333b9d..d48cf83334974296131d1ba9724213573e748ef0 100644
--- a/Source/core/Init.h
+++ b/Source/core/Init.h
@@ -37,17 +37,17 @@ class CoreInitializer {
 public:
     CoreInitializer() : m_isInited(false) { }
     // Should be called by clients before trying to create Frames.
-    void init();
+    virtual void init();
 
+    // FIXME: Why is this function static?

[jsbell, 2014/08/12 18:46:12]

... because the initializer instance is held as a static local in
initializeWithoutV8, so there's no convenient reference to it.

I don't have a good suggestion for rewriting the FIXME other than "this is
asymmetrical and that's usually a bad code smell"

[cmumford, 2014/08/12 19:30:41]

Fixing that was unrelated to my change so I decided to leave it alone.

On 2014/08/12 18:46:12, jsbell wrote:

+    static void shutdown();
+
+protected:
     virtual void registerEventFactory();
     virtual void initEventNames();
     virtual void initEventTargetNames();
     virtual void initBindings() { }
 
-    // FIXME: Why is this function static?
-    static void shutdown();
-
-private:
     bool m_isInited;
 };