zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Revert "Make min video target bitrate configurable."

Browse Source 
This reverts commit a471e797bc6bb5d26375e4c56ff4aacbab08b8a9.

Reason for revert: This CL adds a new symbol to .data instead of .rodata and the symbol should be a constant.

Original change's description:
> Make min video target bitrate configurable.
> 
> Change-Id: I5adf1e675be2114b648878078a8f2e6808c390c7
> Bug: webrtc:10915
> Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/150331
> Commit-Queue: Ying Wang <yinwa@webrtc.org>
> Reviewed-by: Niels Moller <nisse@webrtc.org>
> Reviewed-by: Erik Språng <sprang@webrtc.org>
> Reviewed-by: Christoffer Rodbro <crodbro@webrtc.org>
> Cr-Commit-Position: refs/heads/master@{#28959}

TBR=nisse@webrtc.org,sprang@webrtc.org,crodbro@webrtc.org,yinwa@webrtc.org

Change-Id: I90f23c2c849a6ec518710bbcbdd8e9eb249e9de8
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: webrtc:10915
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/150534
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#28967}
This commit is contained in:
Mirko Bonadei
2019-08-27 07:28:34 +00:00
committed by Commit Bot
parent 3c02842f2e
commit 7e896d0162
4 changed files with 1 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
media/engine/constants.cc
View File

@ -16,5 +16,4 @@ const int kMinVideoBitrateBps = 30000;
const int kVideoMtu = 1200; const int kVideoMtu = 1200;
const int kVideoRtpSendBufferSize = 65536; const int kVideoRtpSendBufferSize = 65536;
const int kVideoRtpRecvBufferSize = 262144; const int kVideoRtpRecvBufferSize = 262144;
const char* kMinVideoBitrateExperiment = "WebRTC-Video-MinVideoBitrate";
} // namespace cricket } // namespace cricket

1
media/engine/constants.h
View File

@ -20,7 +20,6 @@ extern const int kVideoRtpRecvBufferSize;
extern const char kH264CodecName[]; extern const char kH264CodecName[];
extern const int kMinVideoBitrateBps; extern const int kMinVideoBitrateBps;
extern const char* kMinVideoBitrateExperiment;
} // namespace cricket } // namespace cricket

4
media/engine/webrtc_video_engine.cc
View File

@ -27,6 +27,7 @@
#include "api/video_codecs/video_encoder.h" #include "api/video_codecs/video_encoder.h"
#include "api/video_codecs/video_encoder_factory.h" #include "api/video_codecs/video_encoder_factory.h"
#include "call/call.h" #include "call/call.h"
#include "media/engine/constants.h"
#include "media/engine/simulcast.h" #include "media/engine/simulcast.h"
#include "media/engine/webrtc_media_engine.h" #include "media/engine/webrtc_media_engine.h"
#include "media/engine/webrtc_voice_engine.h" #include "media/engine/webrtc_voice_engine.h"
@ -328,9 +329,6 @@ absl::optional<int> GetFallbackMinBpsFromFieldTrial(
} }
int GetMinVideoBitrateBps(webrtc::VideoCodecType type) { int GetMinVideoBitrateBps(webrtc::VideoCodecType type) {
if (webrtc::field_trial::IsEnabled(kMinVideoBitrateExperiment)) {
return MinVideoBitrateConfig().min_video_bitrate->bps();
}
return GetFallbackMinBpsFromFieldTrial(type).value_or(kMinVideoBitrateBps); return GetFallbackMinBpsFromFieldTrial(type).value_or(kMinVideoBitrateBps);
} }
} // namespace } // namespace

14
media/engine/webrtc_video_engine.h
View File

@ -29,15 +29,12 @@
#include "call/video_receive_stream.h" #include "call/video_receive_stream.h"
#include "call/video_send_stream.h" #include "call/video_send_stream.h"
#include "media/base/media_engine.h" #include "media/base/media_engine.h"
#include "media/engine/constants.h"
#include "media/engine/unhandled_packets_buffer.h" #include "media/engine/unhandled_packets_buffer.h"
#include "rtc_base/async_invoker.h" #include "rtc_base/async_invoker.h"
#include "rtc_base/critical_section.h" #include "rtc_base/critical_section.h"
#include "rtc_base/experiments/field_trial_parser.h"
#include "rtc_base/network_route.h" #include "rtc_base/network_route.h"
#include "rtc_base/thread_annotations.h" #include "rtc_base/thread_annotations.h"
#include "rtc_base/thread_checker.h" #include "rtc_base/thread_checker.h"
#include "system_wrappers/include/field_trial.h"
namespace webrtc { namespace webrtc {
class VideoDecoderFactory; class VideoDecoderFactory;
@ -51,17 +48,6 @@ class Thread;
namespace cricket { namespace cricket {
struct MinVideoBitrateConfig {
webrtc::FieldTrialParameter<webrtc::DataRate> min_video_bitrate;
MinVideoBitrateConfig()
: min_video_bitrate("br", webrtc::DataRate::KilobitsPerSec<30>()) {
webrtc::ParseFieldTrial(
{&min_video_bitrate},
webrtc::field_trial::FindFullName(kMinVideoBitrateExperiment));
}
};
class WebRtcVideoChannel; class WebRtcVideoChannel;
class UnsignalledSsrcHandler { class UnsignalledSsrcHandler {