admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Don't set the priority of the decoder to 'high' on Android.

Browse Source 
Doing so competes with the actual decoding that happens on a different thread.

BUG=695438

Review-Url: https://codereview.webrtc.org/2745813003
Cr-Commit-Position: refs/heads/master@{#17184}
This commit is contained in:
tommi 2017-03-11 04:54:06 -08:00 committed by Commit bot
parent d118bacc9b
commit ca37cf6691
1 changed files with 7 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
webrtc/video/video_receive_stream.cc
View File

@ -310,7 +310,14 @@ void VideoReceiveStream::Start() {
call_stats_->RegisterStatsObserver(video_stream_decoder_.get());
// Start the decode thread
decode_thread_.Start();
#if !defined(WEBRTC_ANDROID)
// On android, the decoding happens on a different thread and frames
// are delivered on that thread (that in itself needs to be fixed).
// In any event, the actual decoding work is higher priority than the
// |decode_thread_| on Android, so we only raise the deocode priority on
// platforms other than Android.
decode_thread_.SetPriority(rtc::kHighestPriority);
#endif
rtp_stream_receiver_.StartReceive();
}