diff options
| author | Markus Handell <handellm@webrtc.org> | 2021-06-22 10:46:48 +0200 |
|---|---|---|
| committer | WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2021-06-22 14:51:04 +0000 |
| commit | eb61b7f620c1f84cb9d34c7752cea65fe5d1d721 (patch) | |
| tree | 4240c7418e1c2a9fde352664bd6927b20fac07f9 /modules/rtp_rtcp/source/rtcp_sender.cc | |
| parent | 6e65f6a4288674ff33614f557c33c9dc871fa2e2 (diff) | |
| download | webrtc-upstream-master.tar.gz | |
ModuleRtcRtcpImpl2: remove Module inheritance.upstream-master
This change achieves an Idle Wakeup savings of 200 Hz.
ModuleRtcRtcpImpl2 had Process() logic only active if TMMBR() is
enabled in RtcpSender, which it never is. Hence the Module
inheritance could be removed. The change removes all known
dependencies of the module inheritance, and any related mentions
of ProcessThread.
Fixed: webrtc:11581
Change-Id: I440942f07187fdb9ac18186dab088633969b340e
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/222604
Reviewed-by: Tommi <tommi@webrtc.org>
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Commit-Queue: Markus Handell <handellm@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#34358}
Diffstat (limited to 'modules/rtp_rtcp/source/rtcp_sender.cc')
| -rw-r--r-- | modules/rtp_rtcp/source/rtcp_sender.cc | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/modules/rtp_rtcp/source/rtcp_sender.cc b/modules/rtp_rtcp/source/rtcp_sender.cc index 30a9d01ebb..8f5e3b104c 100644 --- a/modules/rtp_rtcp/source/rtcp_sender.cc +++ b/modules/rtp_rtcp/source/rtcp_sender.cc @@ -955,6 +955,8 @@ void RTCPSender::SendCombinedRtcpPacket( void RTCPSender::SetNextRtcpSendEvaluationDuration(TimeDelta duration) { next_time_to_send_rtcp_ = clock_->CurrentTime() + duration; + // TODO(bugs.webrtc.org/11581): make unconditional once downstream consumers + // are using the callback method. if (schedule_next_rtcp_send_evaluation_function_) schedule_next_rtcp_send_evaluation_function_(duration); } |