 Chromium Code Reviews
 Chromium Code Reviews Issue 2627153004:
  Use TLS storage for thread ids in wtf/ThreadingPThreads  (Closed)
    
  
    Issue 2627153004:
  Use TLS storage for thread ids in wtf/ThreadingPThreads  (Closed) 
  | Index: third_party/WebKit/Source/wtf/WTFThreadData.cpp | 
| diff --git a/third_party/WebKit/Source/wtf/WTFThreadData.cpp b/third_party/WebKit/Source/wtf/WTFThreadData.cpp | 
| index 33351aacb1c5166a40d7b050e03b730bc6d5b1d1..33abb350ad6961e7e70fada3cb27683297d177f4 100644 | 
| --- a/third_party/WebKit/Source/wtf/WTFThreadData.cpp | 
| +++ b/third_party/WebKit/Source/wtf/WTFThreadData.cpp | 
| @@ -35,7 +35,8 @@ ThreadSpecific<WTFThreadData>* WTFThreadData::staticData; | 
| WTFThreadData::WTFThreadData() | 
| : m_atomicStringTable(new AtomicStringTable), | 
| - m_cachedConverterICU(new ICUConverterWrapper) {} | 
| + m_cachedConverterICU(new ICUConverterWrapper), | 
| + m_threadId(internal::currentThreadSyscall()) {} | 
| 
esprehn
2017/01/12 19:59:11
How does this work on non-pthreads platforms? I th
 
Charlie Harrison
2017/01/12 20:07:46
Good catch yeah that would lead to link errors. I'
 | 
| WTFThreadData::~WTFThreadData() {} |