Make HTMLNames threadsafe

Source/WebKit/chromium/src/WebKit.cpp

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 15 matching lines @@
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
 #include "WebKit.h"
 
 #include "CustomElementRegistry.h"
 #include "EventTracer.h"
+#include "Frame.h"
 #include "IDBFactoryBackendProxy.h"
 #include "ImageDecodingStore.h"
 #include "LayoutTestSupport.h"
 #include "Logging.h"
 #include "MutationObserver.h"
 #include "Page.h"
 #include "RuntimeEnabledFeatures.h"
 #include "Settings.h"
 #include "TextEncoding.h"
 #include "V8Binding.h"
@@ skipping 88 matching lines @@
     s_webKitInitialized = true;
 
 #if OS(DARWIN)
     InitWebCoreSystemInterface();
 #endif
 
     ASSERT(webKitPlatformSupport);
     ASSERT(!s_webKitPlatformSupport);
     s_webKitPlatformSupport = webKitPlatformSupport;
     Platform::initialize(s_webKitPlatformSupport);
-    WebCore::ImageDecodingStore::initializeOnce();
 
     WTF::initializeThreading();
     WTF::initializeMainThread();
-    WTF::AtomicString::init();
+    WebCore::init();

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

This is also really just about initalializing static strings and QualifiedNames.
 Maybe that should be exposed in the naming?

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

I'd like to move all the initialization code into it.  For example,
WebCore::ImageDecodingStore::initializeOnce() should probably go in there too.

+    WebCore::ImageDecodingStore::initializeOnce();
 
     // There are some code paths (for example, running WebKit in the browser
     // process and calling into LocalStorage before anything else) where the
     // UTF8 string encoding tables are used on a background thread before
     // they're set up.  This is a problem because their set up routines assert
     // they're running on the main WebKitThread.  It might be possible to make
     // the initialization thread-safe, but given that so many code paths use
     // this, initializing this lazily probably doesn't buy us much.
     WebCore::UTF8Encoding();
 
@@ skipping 53 matching lines @@
     UNUSED_PARAM(name);
 #endif // !LOG_DISABLED
 }
 
 void resetPluginCache(bool reloadPages)
 {
     WebCore::Page::refreshPlugins(reloadPages);
 }
 
 } // namespace WebKit