History log of /external/webrtc/webrtc/base/platform_thread.cc
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
8c38e8b9b96d72317d6ce94c1442113b4e385dcb 26-Nov-2015 Peter Boström <pbos@webrtc.org> Clean up PlatformThread.

* Move PlatformThread to rtc::.
* Remove ::CreateThread factory method.
* Make non-scoped_ptr from a lot of invocations.
* Make Start/Stop void.
* Remove rtc::Thread priorities, which were unused and would collide.
* Add ::IsRunning() to PlatformThread.

BUG=
R=tommi@webrtc.org

Review URL: https://codereview.webrtc.org/1476453002 .

Cr-Commit-Position: refs/heads/master@{#10812}
/external/webrtc/webrtc/base/platform_thread.cc
7d842d660ef5e053da0305bc1d9405fb070b05dc 24-Nov-2015 Peter Boström <pbos@webrtc.org> Move thread_ conditional back under defines.

Unbreaks Windows builds.

BUG=
TBR=tommi@webrtc.org

Review URL: https://codereview.webrtc.org/1476543002 .

Cr-Commit-Position: refs/heads/master@{#10778}
/external/webrtc/webrtc/base/platform_thread.cc
c661213a63afaaf9736aa3e1940e96c9fdf87117 24-Nov-2015 Peter Boström <pbos@webrtc.org> Skip setting thread priorities in NaCl.

Fixes Chromium build since PlatformThread is now built under NaCl.

BUG=
R=tommi@webrtc.org

Review URL: https://codereview.webrtc.org/1472083002 .

Cr-Commit-Position: refs/heads/master@{#10777}
/external/webrtc/webrtc/base/platform_thread.cc
97c821dc56a3cfa3674936b012c10b91fd6ea00e 24-Nov-2015 Peter Boström <pbos@webrtc.org> Inline ConvertToSystemPriority.

Unused function when building Chromium, triggered build errors when
importing webrtc.

BUG=
R=tommi@webrtc.org

Review URL: https://codereview.webrtc.org/1471073005 .

Cr-Commit-Position: refs/heads/master@{#10768}
/external/webrtc/webrtc/base/platform_thread.cc
12411ef40e08c5e28ccde54ab3418c96676ffcbc 23-Nov-2015 pbos <pbos@webrtc.org> Move ThreadWrapper to ProcessThread in base.

Also removes all virtual methods. Permits using a thread from
rtc_base_approved (namely event tracing).

BUG=webrtc:5158
R=tommi@webrtc.org

Review URL: https://codereview.webrtc.org/1469013002

Cr-Commit-Position: refs/heads/master@{#10760}
/external/webrtc/webrtc/base/platform_thread.cc
91d6edef35e7275879c30ce16ecb8b6dc73c6e4a 17-Sep-2015 henrikg <henrikg@webrtc.org> Add RTC_ prefix to (D)CHECKs and related macros.

We must remove dependency on Chromium, i.e. we can't use Chromium's base/logging.h. That means we need to define these macros in WebRTC also when doing Chromium builds. And this causes redefinition.

Alternative solutions:
* Check if we already have defined e.g. CHECK, and don't define them in that case. This makes us depend on include order in Chromium, which is not acceptable.
* Don't allow using the macros in WebRTC headers. Error prone since if someone adds it there by mistake it may compile fine, but later break if a header in added or order is changed in Chromium. That will be confusing and hard to enforce.
* Ensure that headers that are included by an embedder don't include our macros. This would require some heavy refactoring to be maintainable and enforcable.
* Changes in Chromium for this is obviously not an option.

BUG=chromium:468375
NOTRY=true

Review URL: https://codereview.webrtc.org/1335923002

Cr-Commit-Position: refs/heads/master@{#9964}
/external/webrtc/webrtc/base/platform_thread.cc
ea14f0ac1160ee581e43108063794fc6da576c17 18-May-2015 Tommi <tommi@webrtc.org> Move SetCurrentThreadName to platform_thread.* in rtc_base_approved,
update all webrtc and libjingle code to use the same function and remove
extra implementations.

BUG=
R=andresp@webrtc.org

Review URL: https://webrtc-codereview.appspot.com/55439004

Cr-Commit-Position: refs/heads/master@{#9205}
/external/webrtc/webrtc/base/platform_thread.cc
bebc69010d09bab32f59ae110a2c181a09191f02 18-May-2015 Tommi <tommi@webrtc.org> Add platform_thread source files and move types from thread_checker_impl to there.

BUG=
R=henrika@webrtc.org

Review URL: https://webrtc-codereview.appspot.com/52529004

Cr-Commit-Position: refs/heads/master@{#9201}
/external/webrtc/webrtc/base/platform_thread.cc