Revert two instances of num_active_spatial_layers.
The variable, num_active_spatial_layers, is used to construct ssData.
This CL reverts two instances of num_active_spatial_layers not
related to ssData construction.
Bug: None
Change-Id: I4d90d4578684dfdf8bd5a39c7a2fe778fce4414c
Reviewed-on: https://webrtc-review.googlesource.com/85643
Commit-Queue: Michael Horowitz <mhoro@webrtc.org>
Reviewed-by: Sergey Silkin <ssilkin@webrtc.org>
Reviewed-by: Rasmus Brandt <brandtr@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#23756}diff --git a/modules/video_coding/codecs/vp9/vp9_impl.cc b/modules/video_coding/codecs/vp9/vp9_impl.cc
index f0747e5..5b7f5b9 100644
--- a/modules/video_coding/codecs/vp9/vp9_impl.cc
+++ b/modules/video_coding/codecs/vp9/vp9_impl.cc
@@ -163,7 +163,7 @@
} else {
float rate_ratio[VPX_MAX_LAYERS] = {0};
float total = 0;
- for (i = 0; i < num_active_spatial_layers_; ++i) {
+ for (i = 0; i < num_spatial_layers_; ++i) {
if (svc_params_.scaling_factor_num[i] <= 0 ||
svc_params_.scaling_factor_den[i] <= 0) {
RTC_LOG(LS_ERROR) << "Scaling factors not specified!";
@@ -174,7 +174,7 @@
total += rate_ratio[i];
}
- for (i = 0; i < num_active_spatial_layers_; ++i) {
+ for (i = 0; i < num_spatial_layers_; ++i) {
RTC_CHECK_GT(total, 0);
config_->ss_target_bitrate[i] = static_cast<unsigned int>(
config_->rc_target_bitrate * rate_ratio[i] / total);
@@ -200,14 +200,6 @@
}
}
}
-
- // For now, temporal layers only supported when having one spatial layer.
- if (num_spatial_layers_ == 1) {
- for (i = 0; i < num_temporal_layers_; ++i) {
- config_->ts_target_bitrate[i] = config_->layer_target_bitrate[i];
- }
- }
-
return true;
}