diff --git a/webrtc/media/BUILD.gn b/webrtc/media/BUILD.gn index 5114ba65d5..be75f6f57d 100644 --- a/webrtc/media/BUILD.gn +++ b/webrtc/media/BUILD.gn @@ -112,6 +112,7 @@ rtc_static_library("rtc_media_base") { } deps += [ + ":rtc_h264_profile_id", "..:webrtc_common", "../api:libjingle_peerconnection_api", "../p2p", @@ -119,8 +120,6 @@ rtc_static_library("rtc_media_base") { "../rtc_base:rtc_base_approved", ] - public_deps += [ ":rtc_h264_profile_id" ] - if (is_nacl) { deps += [ "//native_client_sdk/src/libraries/nacl_io" ] } diff --git a/webrtc/media/engine/videoencodersoftwarefallbackwrapper.cc b/webrtc/media/engine/videoencodersoftwarefallbackwrapper.cc index 540597838a..2bc74a7e72 100644 --- a/webrtc/media/engine/videoencodersoftwarefallbackwrapper.cc +++ b/webrtc/media/engine/videoencodersoftwarefallbackwrapper.cc @@ -10,7 +10,6 @@ #include "webrtc/media/engine/videoencodersoftwarefallbackwrapper.h" -#include "webrtc/media/base/h264_profile_level_id.h" #include "webrtc/media/engine/internalencoderfactory.h" #include "webrtc/modules/video_coding/include/video_error_codes.h" #include "webrtc/rtc_base/checks.h" @@ -90,7 +89,6 @@ VideoEncoderSoftwareFallbackWrapper::VideoEncoderSoftwareFallbackWrapper( } bool VideoEncoderSoftwareFallbackWrapper::InitFallbackEncoder() { - MaybeModifyCodecForFallback(); cricket::InternalEncoderFactory internal_factory; if (!FindMatchingCodec(internal_factory.supported_codecs(), codec_)) { LOG(LS_WARNING) @@ -357,13 +355,4 @@ bool VideoEncoderSoftwareFallbackWrapper::ForcedFallbackParams::ShouldStop( (codec.width * codec.height >= kMinPixelsStop); } -void VideoEncoderSoftwareFallbackWrapper::MaybeModifyCodecForFallback() { - // We have a specific case for H264 ConstrainedBaseline because that is the - // only supported profile in Sw fallback. - if (!cricket::CodecNamesEq(codec_.name.c_str(), cricket::kH264CodecName)) - return; - codec_.SetParam(cricket::kH264FmtpProfileLevelId, - cricket::kH264ProfileLevelConstrainedBaseline); -} - } // namespace webrtc diff --git a/webrtc/media/engine/videoencodersoftwarefallbackwrapper.h b/webrtc/media/engine/videoencodersoftwarefallbackwrapper.h index 8061c008c6..d11a0f678e 100644 --- a/webrtc/media/engine/videoencodersoftwarefallbackwrapper.h +++ b/webrtc/media/engine/videoencodersoftwarefallbackwrapper.h @@ -77,7 +77,6 @@ class VideoEncoderSoftwareFallbackWrapper : public VideoEncoder { bool TryReInitForcedFallbackEncoder(); void ValidateSettingsForForcedFallback(); bool IsForcedFallbackActive() const; - void MaybeModifyCodecForFallback(); // Settings used in the last InitEncode call and used if a dynamic fallback to // software is required. @@ -95,7 +94,7 @@ class VideoEncoderSoftwareFallbackWrapper : public VideoEncoder { uint32_t packet_loss_; int64_t rtt_; - cricket::VideoCodec codec_; + const cricket::VideoCodec codec_; webrtc::VideoEncoder* const encoder_; std::unique_ptr fallback_encoder_;