Refactor out VideoStreamEncoder's overuse logic to separate module.
This CL puts the VideoStreamEncoder's current adaptation logic inside
the new class OveruseFrameDetectorResourceAdaptationModule. The
intention is not to change any behavior, only to move code.
Future CLs should step by step decrease the coupling between
OveruseFrameDetectorResourceAdaptationModule, VideoStreamEncoder and
the VideoStreamEncoder's QualityScaler by introducing more abstract
interfaces. This is not done in this CL because it is large enough as
it is, but the long term goal is to make it possible to replace the
existing overuse module with a different implementation.
This CL relies on existing tests exercising the VideoStreamEncoder, but
part of making overuse logic modular should include testing each module
separately as well as continued integration testing of the
VideoStreamEncoder.
Bug: webrtc:11222
Change-Id: I316a174adfd00d60cdd224a23a5f616efd235d13
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/161953
Commit-Queue: Henrik Boström <hbos@webrtc.org>
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Reviewed-by: Evan Shrubsole <eshr@google.com>
Cr-Commit-Position: refs/heads/master@{#30163}
diff --git a/video/BUILD.gn b/video/BUILD.gn
index 0f9829e..51cd40b 100644
--- a/video/BUILD.gn
+++ b/video/BUILD.gn
@@ -184,11 +184,14 @@
"frame_encode_metadata_writer.h",
"overuse_frame_detector.cc",
"overuse_frame_detector.h",
+ "overuse_frame_detector_resource_adaptation_module.cc",
+ "overuse_frame_detector_resource_adaptation_module.h",
"video_stream_encoder.cc",
"video_stream_encoder.h",
]
deps = [
+ "../api:rtp_parameters",
"../api/units:data_rate",
"../api/video:encoded_image",
"../api/video:video_bitrate_allocation",