Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1538)

Unified Diff: Source/web/WebKit.cpp

Issue 794223003: Cheaper thread-safe atomic initialization of static references. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Add type check for initial value Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/web/WebKit.cpp
diff --git a/Source/web/WebKit.cpp b/Source/web/WebKit.cpp
index 99add6deef0707be5cdeccfaa3e60584f6e7a1a8..4d572cce7720533727aad1270b98e95d18c2cd19 100644
--- a/Source/web/WebKit.cpp
+++ b/Source/web/WebKit.cpp
@@ -165,15 +165,6 @@ void initializeWithoutV8(Platform* platform)
DEFINE_STATIC_LOCAL(ModulesInitializer, initializer, ());
initializer.init();
- // 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.
- WTF::UTF8Encoding();
-
setIndexedDBClientCreateFunction(IndexedDBClientImpl::create);
MediaPlayer::setMediaEngineCreateFunction(WebMediaPlayerClientImpl::create);
« no previous file with comments | « Source/web/WebImageCache.cpp ('k') | Source/wtf/ArrayBuffer.h » ('j') | Source/wtf/Threading.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698