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

Side by Side Diff: webrtc/video/video_send_stream.cc

Issue 2493893003: Turn off error resilience for vp8 for no temporal layers if nack is enabled. (Closed)
Patch Set: Created 4 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2013 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2013 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 #include "webrtc/video/video_send_stream.h" 10 #include "webrtc/video/video_send_stream.h"
(...skipping 568 matching lines...) Expand 10 before | Expand all | Expand 10 after
579 rtc::VideoSourceInterface<webrtc::VideoFrame>* source, 579 rtc::VideoSourceInterface<webrtc::VideoFrame>* source,
580 const DegradationPreference& degradation_preference) { 580 const DegradationPreference& degradation_preference) {
581 RTC_DCHECK_RUN_ON(&thread_checker_); 581 RTC_DCHECK_RUN_ON(&thread_checker_);
582 vie_encoder_->SetSource(source, degradation_preference); 582 vie_encoder_->SetSource(source, degradation_preference);
583 } 583 }
584 584
585 void VideoSendStream::ReconfigureVideoEncoder(VideoEncoderConfig config) { 585 void VideoSendStream::ReconfigureVideoEncoder(VideoEncoderConfig config) {
586 // TODO(perkj): Some test cases in VideoSendStreamTest call 586 // TODO(perkj): Some test cases in VideoSendStreamTest call
587 // ReconfigureVideoEncoder from the network thread. 587 // ReconfigureVideoEncoder from the network thread.
588 // RTC_DCHECK_RUN_ON(&thread_checker_); 588 // RTC_DCHECK_RUN_ON(&thread_checker_);
589 vie_encoder_->ConfigureEncoder(std::move(config), 589 vie_encoder_->ConfigureEncoder(std::move(config), config_.rtp.max_packet_size,
590 config_.rtp.max_packet_size); 590 config_.rtp.nack.rtp_history_ms > 0);
591 } 591 }
592 592
593 VideoSendStream::Stats VideoSendStream::GetStats() { 593 VideoSendStream::Stats VideoSendStream::GetStats() {
594 // TODO(perkj, solenberg): Some test cases in EndToEndTest call GetStats from 594 // TODO(perkj, solenberg): Some test cases in EndToEndTest call GetStats from
595 // a network thread. See comment in Call::GetStats(). 595 // a network thread. See comment in Call::GetStats().
596 // RTC_DCHECK_RUN_ON(&thread_checker_); 596 // RTC_DCHECK_RUN_ON(&thread_checker_);
597 return stats_proxy_.GetStats(); 597 return stats_proxy_.GetStats();
598 } 598 }
599 599
600 void VideoSendStream::SignalNetworkState(NetworkState state) { 600 void VideoSendStream::SignalNetworkState(NetworkState state) {
(...skipping 518 matching lines...) Expand 10 before | Expand all | Expand 10 after
1119 &module_nack_rate); 1119 &module_nack_rate);
1120 *sent_video_rate_bps += module_video_rate; 1120 *sent_video_rate_bps += module_video_rate;
1121 *sent_nack_rate_bps += module_nack_rate; 1121 *sent_nack_rate_bps += module_nack_rate;
1122 *sent_fec_rate_bps += module_fec_rate; 1122 *sent_fec_rate_bps += module_fec_rate;
1123 } 1123 }
1124 return 0; 1124 return 0;
1125 } 1125 }
1126 1126
1127 } // namespace internal 1127 } // namespace internal
1128 } // namespace webrtc 1128 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698