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

Unified Diff: webrtc/video_engine/vie_channel_group.cc

Issue 1208083002: Remove MimdRateControl and factories for RemoteBitrateEstimor. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Addressed comments Created 5 years, 5 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 | « webrtc/video/rampup_tests.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/video_engine/vie_channel_group.cc
diff --git a/webrtc/video_engine/vie_channel_group.cc b/webrtc/video_engine/vie_channel_group.cc
index 768365c37169616c41df18f5d5b1d4cc803a3c55..b0923b30dea3f0dc69b9ec2aba18de8ad508b74d 100644
--- a/webrtc/video_engine/vie_channel_group.cc
+++ b/webrtc/video_engine/vie_channel_group.cc
@@ -16,7 +16,8 @@
#include "webrtc/experiments.h"
#include "webrtc/modules/pacing/include/paced_sender.h"
#include "webrtc/modules/pacing/include/packet_router.h"
-#include "webrtc/modules/remote_bitrate_estimator/include/remote_bitrate_estimator.h"
+#include "webrtc/modules/remote_bitrate_estimator/remote_bitrate_estimator_abs_send_time.h"
+#include "webrtc/modules/remote_bitrate_estimator/remote_bitrate_estimator_single_stream.h"
#include "webrtc/modules/rtp_rtcp/interface/rtp_rtcp.h"
#include "webrtc/modules/utility/interface/process_thread.h"
#include "webrtc/system_wrappers/interface/critical_section_wrapper.h"
@@ -43,13 +44,11 @@ class WrappingBitrateEstimator : public RemoteBitrateEstimator {
clock_(clock),
crit_sect_(CriticalSectionWrapper::CreateCriticalSection()),
min_bitrate_bps_(config.Get<RemoteBitrateEstimatorMinRate>().min_rate),
- rbe_(RemoteBitrateEstimatorFactory().Create(observer_,
+ rbe_(new RemoteBitrateEstimatorSingleStream(observer_,
clock_,
- kAimdControl,
min_bitrate_bps_)),
using_absolute_send_time_(false),
- packets_since_absolute_send_time_(0) {
- }
+ packets_since_absolute_send_time_(0) {}
virtual ~WrappingBitrateEstimator() {}
@@ -122,11 +121,11 @@ class WrappingBitrateEstimator : public RemoteBitrateEstimator {
// Instantiate RBE for Time Offset or Absolute Send Time extensions.
void PickEstimator() EXCLUSIVE_LOCKS_REQUIRED(crit_sect_.get()) {
if (using_absolute_send_time_) {
- rbe_.reset(AbsoluteSendTimeRemoteBitrateEstimatorFactory().Create(
- observer_, clock_, kAimdControl, min_bitrate_bps_));
+ rbe_.reset(new RemoteBitrateEstimatorAbsSendTime(observer_, clock_,
+ min_bitrate_bps_));
} else {
- rbe_.reset(RemoteBitrateEstimatorFactory().Create(
- observer_, clock_, kAimdControl, min_bitrate_bps_));
+ rbe_.reset(new RemoteBitrateEstimatorSingleStream(observer_, clock_,
+ min_bitrate_bps_));
}
}
« no previous file with comments | « webrtc/video/rampup_tests.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698