Make HTMLNames threadsafe

Source/WebCore/page/Frame.cpp

 /*
  * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
  *                     1999 Lars Knoll <knoll@kde.org>
  *                     1999 Antti Koivisto <koivisto@kde.org>
  *                     2000 Simon Hausmann <hausmann@kde.org>
  *                     2000 Stefan Schimanski <1Stein@gmx.de>
  *                     2001 George Staikos <staikos@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2005 Alexey Proskuryakov <ap@nypop.com>
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
@@ skipping 78 matching lines @@
 #include "VisibleUnits.h"
 #include "WebKitFontFamilyNames.h"
 #include "XMLNSNames.h"
 #include "XMLNames.h"
 #include "htmlediting.h"
 #include "markup.h"
 #include "npruntime_impl.h"
 #include <wtf/PassOwnPtr.h>
 #include <wtf/RefCountedLeakCounter.h>
 #include <wtf/StdLibExtras.h>
+#include <wtf/text/StringStatics.h>
 
 #include "MathMLNames.h"
 #include "SVGNames.h"
 #include "XLinkNames.h"
 
 #if ENABLE(SVG)
 #include "SVGDocument.h"
 #include "SVGDocumentExtensions.h"
 #endif
 
@@ skipping 21 matching lines @@
 }
 
 static inline float parentTextZoomFactor(Frame* frame)
 {
     Frame* parent = frame->tree()->parent();
     if (!parent)
         return 1;
     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))
     , m_loader(this, frameLoaderClient)
     , m_navigationScheduler(this)
     , m_ownerElement(ownerElement)
     , m_script(this)
     , m_editor(this)
     , m_selection(this)
     , m_eventHandler(this)
     , m_animationController(this)
     , m_pageZoomFactor(parentPageZoomFactor(this))
     , m_textZoomFactor(parentTextZoomFactor(this))
 #if ENABLE(ORIENTATION_EVENTS)
     , m_orientation(0)
 #endif
     , m_inViewSourceMode(false)
     , 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();
         ownerElement->setContentFrame(this);
     }
 
 #ifndef NDEBUG
     frameCounter.increment();
 #endif
 
@@ skipping 859 matching lines @@
     buffer->context()->translate(-paintingRect.x(), -paintingRect.y());
     buffer->context()->clip(FloatRect(0, 0, paintingRect.maxX(), paintingRect.maxY()));
 
     m_view->paintContents(buffer->context(), paintingRect);
 
     RefPtr<Image> image = buffer->copyImage();
     return createDragImageFromImage(image.get());
 }
 
 } // namespace WebCore