Make HTMLNames threadsafe

Source/WebCore/page/Frame.cpp

Index: Source/WebCore/page/Frame.cpp
diff --git a/Source/WebCore/page/Frame.cpp b/Source/WebCore/page/Frame.cpp
index 71998722d1a6e8676c087c2807390e4a99abfb7e..6b07fd3221237b61da332bfcc34e4b26e4460578 100644
--- a/Source/WebCore/page/Frame.cpp
+++ b/Source/WebCore/page/Frame.cpp
@@ -96,6 +96,7 @@
 #include <wtf/PassOwnPtr.h>
 #include <wtf/RefCountedLeakCounter.h>
 #include <wtf/StdLibExtras.h>
+#include <wtf/text/StringStatics.h>
 
 #include "MathMLNames.h"
 #include "SVGNames.h"
@@ -137,6 +138,21 @@ static inline float parentTextZoomFactor(Frame* frame)
     return parent->textZoomFactor();
 }
 
+void init()

[eseidel, 2013/04/10 20:53:50]

We should just move this to its own file.  WebCoreInit.cpp/WebCoreInit.h, no?

[abarth-chromium, 2013/04/10 21:00:51]

Yes.  Would you mind if I do that in the next CL?  I want to make all the
initialization explicit, which will dovetail nicely with that change.

+{
+    AtomicString::init();
+    HTMLNames::init();
+    SVGNames::init();
+    XLinkNames::init();
+    MathMLNames::init();
+    XMLNSNames::init();
+    XMLNames::init();
+    WebKitFontFamilyNames::init();
+    MediaFeatureNames::init();
+    WTF::StringStatics::init();
+    QualifiedName::init();
+}
+
 inline Frame::Frame(Page* page, HTMLFrameOwnerElement* ownerElement, FrameLoaderClient* frameLoaderClient)
     : m_page(page)
     , m_treeNode(this, parentFromOwnerElement(ownerElement))
@@ -157,16 +173,7 @@ inline Frame::Frame(Page* page, HTMLFrameOwnerElement* ownerElement, FrameLoader
     , m_activeDOMObjectsAndAnimationsSuspendedCount(0)
 {
     ASSERT(page);
-    AtomicString::init();
-    HTMLNames::init();
-    QualifiedName::init();
-    MediaFeatureNames::init();
-    SVGNames::init();
-    XLinkNames::init();
-    MathMLNames::init();
-    XMLNSNames::init();
-    XMLNames::init();
-    WebKitFontFamilyNames::init();
+    WebCore::init();
 
     if (ownerElement) {
         page->incrementSubframeCount();