Remove ThreadWrapper::GetThreadId. The method just calls rtc::CurrentThreadId(), which also has a m…

webrtc/system_wrappers/source/thread_posix.cc

 /*
  *  Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
 #include "webrtc/system_wrappers/source/thread_posix.h"
 
 #include <algorithm>
 
 #include <errno.h>
 #include <unistd.h>
 #ifdef WEBRTC_LINUX
 #include <linux/unistd.h>
 #include <sched.h>
 #include <sys/types.h>
 #endif
 
 #include "webrtc/base/checks.h"
-#include "webrtc/base/platform_thread.h"
 #include "webrtc/system_wrappers/include/critical_section_wrapper.h"
 #include "webrtc/system_wrappers/include/event_wrapper.h"
 #include "webrtc/system_wrappers/include/sleep.h"
 #include "webrtc/system_wrappers/include/trace.h"
 
 namespace webrtc {
 namespace {
 struct ThreadAttributes {
   ThreadAttributes() { pthread_attr_init(&attr); }
   ~ThreadAttributes() { pthread_attr_destroy(&attr); }
@@ skipping 35 matching lines @@
 ThreadPosix::ThreadPosix(ThreadRunFunction func, void* obj,
                          const char* thread_name)
     : run_function_(func),
       obj_(obj),
       stop_event_(false, false),
       name_(thread_name ? thread_name : "webrtc"),
       thread_(0) {
   RTC_DCHECK(name_.length() < 64);
 }
 
-uint32_t ThreadWrapper::GetThreadId() {
-  return rtc::CurrentThreadId();
-}
-
 ThreadPosix::~ThreadPosix() {
   RTC_DCHECK(thread_checker_.CalledOnValidThread());
 }
 
 // TODO(pbos): Make Start void, calling code really doesn't support failures
 // here.
 bool ThreadPosix::Start() {
   RTC_DCHECK(thread_checker_.CalledOnValidThread());
   RTC_DCHECK(!thread_) << "Thread already started?";
 
@@ skipping 63 matching lines @@
   // It's a requirement that for successful thread creation that the run
   // function be called at least once (see RunFunctionIsCalled unit test),
   // so to fullfill that requirement, we use a |do| loop and not |while|.
   do {
     if (!run_function_(obj_))
       break;
   } while (!stop_event_.Wait(0));
 }
 
 }  // namespace webrtc