Don't create redundant ThreadData for Worker Threads

base/tracked_objects.cc

Index: base/tracked_objects.cc
diff --git a/base/tracked_objects.cc b/base/tracked_objects.cc
index d24cedf159241d58a92909650de374f187dd85ae..49f90e64e64041e0697d92d7077780c57ae2adda 100644
--- a/base/tracked_objects.cc
+++ b/base/tracked_objects.cc
@@ -354,11 +354,12 @@ ThreadData* ThreadData::first() {
 ThreadData* ThreadData::next() const { return next_; }
 
 // static
-void ThreadData::InitializeThreadContext(const std::string& suggested_name) {
+void ThreadData::InitializeThreadContext(const std::string& suggested_name,
+                                         bool is_worker_thread) {
   Initialize();
   ThreadData* current_thread_data =
       reinterpret_cast<ThreadData*>(tls_index_.Get());
-  if (current_thread_data)
+  if (current_thread_data or is_worker_thread)

[jar (doing other things), 2016/04/20 04:34:51]

The test for is_worker_thread should be done between line 358 and 359, and you
can return at that point.

The call on line 359 should probably be renamed for clarity:

Initialize() ---> EnsureTlsInitialization()

as it sets things up (see its definition later in this file) so that the call on
line 361 can proceed.

[Zhenyu Shan, 2016/05/04 23:33:03]

Done.

     return;  // Browser tests instigate this.
   current_thread_data = new ThreadData(suggested_name);
   tls_index_.Set(current_thread_data);