commit a3aa9bd75b094893160de8957592d7937a3b50da
author Jonas Oreland <jonaso@webrtc.org> Wed Apr 17 07:38:40 2019 +0200
committer Commit Bot <commit-bot@chromium.org> Wed Apr 17 06:17:34 2019 +0000
tree 272b40757d7a99070fae9158f3fe5a3c9d8a44a1
parent a1a93fba364d91ad4f662bba43d13f298f2d9aa3

Make VideoBitrateAllocatorFactory injectable.

This patch makes VideoBitrateAllocatorFactory injectable
by adding to PeerConnectionDependencies instead of allowing it to be
overridden using MediaEngine (on PeerConnectionFactory).

With this patch VideoBitrateAllocatorFactory is owned
by the PeerConnection.

WANT_LGTM (examples) : sakal@
WANT_LGTM (api/pc) : steveanton@

Bug: webrtc:10547
Change-Id: I768d400a621f2b7a98795eb7f410adb48651bfd6
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/132706
Commit-Queue: Jonas Oreland <jonaso@webrtc.org>
Reviewed-by: Sami Kalliomäki <sakal@webrtc.org>
Reviewed-by: Steve Anton <steveanton@webrtc.org>
Reviewed-by: Rasmus Brandt <brandtr@webrtc.org>
Reviewed-by: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#27654}

pc/peer_connection.h

diff --git a/pc/peer_connection.h b/pc/peer_connection.h
index 2cf3662..c21261c 100644
--- a/pc/peer_connection.h
+++ b/pc/peer_connection.h
@@ -1332,6 +1332,15 @@
   // channel manager and the session description factory.
   rtc::UniqueRandomIdGenerator ssrc_generator_
       RTC_GUARDED_BY(signaling_thread());
+
+  // A video bitrate allocator factory.
+  // This can injected using the PeerConnectionDependencies,
+  // or else the CreateBuiltinVideoBitrateAllocatorFactory() will be called.
+  // Note that one can still choose to override this in a MediaEngine
+  // if one wants too.
+  std::unique_ptr<webrtc::VideoBitrateAllocatorFactory>
+      video_bitrate_allocator_factory_;
+
   bool is_negotiation_needed_ RTC_GUARDED_BY(signaling_thread()) = false;
 };