Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(237)

Unified Diff: webrtc/system_wrappers/include/thread_wrapper.h

Issue 1467243003: Remove ThreadWrapper::GetThreadId. The method just calls rtc::CurrentThreadId(), which also has a m… (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 5 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/system_wrappers/include/thread_wrapper.h
diff --git a/webrtc/system_wrappers/include/thread_wrapper.h b/webrtc/system_wrappers/include/thread_wrapper.h
index d475302139f9faa023b6f5f9ed441843beb2e361..dbd548dfd9871f8c3578da5a4c683c321d39790e 100644
--- a/webrtc/system_wrappers/include/thread_wrapper.h
+++ b/webrtc/system_wrappers/include/thread_wrapper.h
@@ -67,12 +67,6 @@ class ThreadWrapper {
static rtc::scoped_ptr<ThreadWrapper> CreateThread(ThreadRunFunction func,
void* obj, const char* thread_name);
- // Get the current thread's thread ID.
- // NOTE: This is a static method. It returns the id of the calling thread,
- // *not* the id of the worker thread that a ThreadWrapper instance represents.
- // TODO(tommi): Move outside of the ThreadWrapper class to avoid confusion.
- static uint32_t GetThreadId();
-
// Tries to spawns a thread and returns true if that was successful.
// Additionally, it tries to set thread priority according to the priority
// from when CreateThread was called. However, failure to set priority will
« no previous file with comments | « webrtc/modules/remote_bitrate_estimator/test/bwe_test_logging.cc ('k') | webrtc/system_wrappers/source/thread_posix.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698