Don't create redundant ThreadData for Worker Threads

base/threading/worker_pool_posix.cc

Index: base/threading/worker_pool_posix.cc
diff --git a/base/threading/worker_pool_posix.cc b/base/threading/worker_pool_posix.cc
index 53ae4e69627807b209618834104fecf70ed65fc9..043e5d0be6a8a7b954e9ead826f6cc5277b417fd 100644
--- a/base/threading/worker_pool_posix.cc
+++ b/base/threading/worker_pool_posix.cc
@@ -80,7 +80,7 @@ void WorkerThread::ThreadMain() {
   const std::string name = base::StringPrintf("%s/%d", name_prefix_.c_str(),
                                               PlatformThread::CurrentId());
   // Note |name.c_str()| must remain valid for for the whole life of the thread.
-  PlatformThread::SetName(name);
+  PlatformThread::SetName(name, true);

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

It is surprising to me that this is the *only* place where we SetName() for a
worker thread (in your CL).  What about Windows? Android? etc.?

Why don't the other platforms need this code?

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

Android shares posix WorkerThread::ThreadMain code.
Windows uses system managed worker pool implementation, see
worker_pool_win.cc:50 calling QueueUserWorkItem().

 
   for (;;) {
     PendingTask pending_task = pool_->WaitForTask();