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

Unified Diff: webrtc/video/video_receive_stream.cc

Issue 2745813003: Don't set the priority of the decoder to 'high' on Android. (Closed)
Patch Set: Created 3 years, 9 months 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/video/video_receive_stream.cc
diff --git a/webrtc/video/video_receive_stream.cc b/webrtc/video/video_receive_stream.cc
index eb2404c174ed749baef44638c831249a0499a415..be623cef56ade6bd4c3f60357bdc7eb47db151ad 100644
--- a/webrtc/video/video_receive_stream.cc
+++ b/webrtc/video/video_receive_stream.cc
@@ -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();
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698