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

Unified Diff: webrtc/modules/video_coding/protection_bitrate_calculator.cc

Issue 2121983002: Remove frame rate and target bitrate from ProtectionBitrateCalculator::SetEncodingData (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Fix size_t problem. Created 4 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
Index: webrtc/modules/video_coding/protection_bitrate_calculator.cc
diff --git a/webrtc/modules/video_coding/protection_bitrate_calculator.cc b/webrtc/modules/video_coding/protection_bitrate_calculator.cc
index ba93442f391764680ae6f5e1b00c649ea6a5e94a..411ed1a95ca57838792f0b73711e31ea3d5cd557 100644
--- a/webrtc/modules/video_coding/protection_bitrate_calculator.cc
+++ b/webrtc/modules/video_coding/protection_bitrate_calculator.cc
@@ -39,18 +39,11 @@ ProtectionBitrateCalculator::~ProtectionBitrateCalculator(void) {
loss_prot_logic_->Release();
}
-void ProtectionBitrateCalculator::SetEncodingData(uint32_t target_bitrate,
- uint16_t width,
- uint16_t height,
- uint32_t frame_rate,
+void ProtectionBitrateCalculator::SetEncodingData(size_t width,
+ size_t height,
size_t num_temporal_layers,
size_t max_payload_size) {
CritScope lock(&crit_sect_);
- // Everything codec specific should be reset here since this means the codec
- // has changed.
- float target_bitrate_kbps = static_cast<float>(target_bitrate) / 1000.0f;
- loss_prot_logic_->UpdateBitRate(target_bitrate_kbps);
- loss_prot_logic_->UpdateFrameRate(static_cast<float>(frame_rate));
loss_prot_logic_->UpdateFrameSize(width, height);
loss_prot_logic_->UpdateNumLayers(num_temporal_layers);
max_payload_size_ = max_payload_size;

Powered by Google App Engine
This is Rietveld 408576698