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

Unified Diff: webrtc/video_engine/vie_channel.cc

Issue 1411573007: Removed vie_defines.h (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 5 years, 2 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
Index: webrtc/video_engine/vie_channel.cc
diff --git a/webrtc/video_engine/vie_channel.cc b/webrtc/video_engine/vie_channel.cc
index 147ecb1456764e0b97d78b4cdbb771e44874c4d1..0b4d504a043915196f87dd5d513927ae5cd9f65b 100644
--- a/webrtc/video_engine/vie_channel.cc
+++ b/webrtc/video_engine/vie_channel.cc
@@ -34,13 +34,15 @@
#include "webrtc/video_engine/call_stats.h"
#include "webrtc/video_engine/payload_router.h"
#include "webrtc/video_engine/report_block_stats.h"
-#include "webrtc/video_engine/vie_defines.h"
namespace webrtc {
const int kMaxDecodeWaitTimeMs = 50;
static const int kMaxTargetDelayMs = 10000;
static const float kMaxIncompleteTimeMultiplier = 3.5f;
+const int kMinSendSidePacketHistorySize = 600;
+const int kMaxPacketAgeToNack = 450;
+const int kMaxNackListSize = 250;
// Helper class receiving statistics callbacks.
class ChannelStatsObserver : public CallStatsObserver {
@@ -109,7 +111,7 @@ ViEChannel::ViEChannel(uint32_t number_of_cores,
packet_router_(packet_router),
bandwidth_observer_(bandwidth_observer),
transport_feedback_observer_(transport_feedback_observer),
- nack_history_size_sender_(kSendSidePacketHistorySize),
+ nack_history_size_sender_(kMinSendSidePacketHistorySize),
max_nack_reordering_threshold_(kMaxPacketAgeToNack),
pre_render_callback_(NULL),
report_block_stats_sender_(new ReportBlockStats()),
@@ -139,6 +141,7 @@ ViEChannel::ViEChannel(uint32_t number_of_cores,
}
int32_t ViEChannel::Init() {
+ const int kDefaultRenderDelayMs = 10;
pbos-webrtc 2015/10/29 15:39:40 static
mflodman 2015/11/06 11:38:36 Done.
module_process_thread_->RegisterModule(vie_receiver_.GetReceiveStatistics());
// RTP/RTCP initialization.
@@ -161,7 +164,7 @@ int32_t ViEChannel::Init() {
vcm_->RegisterFrameTypeCallback(this);
vcm_->RegisterReceiveStatisticsCallback(this);
vcm_->RegisterDecoderTimingCallback(this);
- vcm_->SetRenderDelay(kViEDefaultRenderDelayMs);
+ vcm_->SetRenderDelay(kDefaultRenderDelayMs);
module_process_thread_->RegisterModule(vcm_);
module_process_thread_->RegisterModule(&vie_sync_);
@@ -562,12 +565,12 @@ int ViEChannel::SetSenderBufferingMode(int target_delay_ms) {
}
if (target_delay_ms == 0) {
// Real-time mode.
- nack_history_size_sender_ = kSendSidePacketHistorySize;
+ nack_history_size_sender_ = kMinSendSidePacketHistorySize;
} else {
nack_history_size_sender_ = GetRequiredNackListSize(target_delay_ms);
// Don't allow a number lower than the default value.
- if (nack_history_size_sender_ < kSendSidePacketHistorySize) {
- nack_history_size_sender_ = kSendSidePacketHistorySize;
+ if (nack_history_size_sender_ < kMinSendSidePacketHistorySize) {
+ nack_history_size_sender_ = kMinSendSidePacketHistorySize;
}
}
for (RtpRtcp* rtp_rtcp : rtp_rtcp_modules_)

Powered by Google App Engine
This is Rietveld 408576698