Don't create redundant ThreadData for Worker Threads

base/threading/platform_thread_linux.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/threading/platform_thread.h"
 
 #include <errno.h>
 #include <sched.h>
 #include <stddef.h>
 
@@ skipping 48 matching lines @@
     *priority = ThreadPriority::REALTIME_AUDIO;
     return true;
   }
 #endif
   return false;
 }
 
 }  // namespace internal
 
 // static
-void PlatformThread::SetName(const std::string& name) {
+void PlatformThread::SetName(const std::string& name,
+                             bool is_worker_thread) {

[Lei Zhang, 2016/05/31 21:43:24]

nit: fix on the previous line, just like all the other platform impls.

   ThreadIdNameManager::GetInstance()->SetName(CurrentId(), name);
-  tracked_objects::ThreadData::InitializeThreadContext(name);
+  if (!is_worker_thread)
+    tracked_objects::ThreadData::InitializeThreadContext(name);
 
 #if !defined(OS_NACL)
   // On linux we can get the thread names to show up in the debugger by setting
   // the process name for the LWP.  We don't want to do this for the main
   // thread because that would rename the process, causing tools like killall
   // to stop working.
   if (PlatformThread::CurrentId() == getpid())
     return;
 
   // http://0pointer.de/blog/projects/name-your-threads.html
@@ skipping 16 matching lines @@
 #if !defined(THREAD_SANITIZER)
   return 0;
 #else
   // ThreadSanitizer bloats the stack heavily. Evidence has been that the
   // default stack size isn't enough for some browser tests.
   return 2 * (1 << 23);  // 2 times 8192K (the default stack size on Linux).
 #endif
 }
 
 }  // namespace base