commit 4bac79ece2c3ddfa06c69e7d190044a90645ae68
author Ruslan Burakov <kuddai@google.com> Wed Apr 03 19:55:33 2019 +0200
committer Commit Bot <commit-bot@chromium.org> Thu Apr 04 09:00:16 2019 +0000
tree f79fae072eebb868f4645f3fac74b909ec9f2487
parent ada9b89b99be4387ff40529bbec1887698a0c03a

Add SetJitterBufferMinimumDelay method to RtpReceiverInterface.

This change is required to allow modification of Jitter Buffer delay
in javascript via Origin Trial Experiment.
Link to experiment description:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/Tgm4qiNepJc

Bug: webrtc:10287
Change-Id: I4f21380aad5982a4a60c55683b5173ce72ce0392
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/131144
Commit-Queue: Ruslan Burakov <kuddai@google.com>
Reviewed-by: Steve Anton <steveanton@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#27444}

api/rtp_receiver_interface.cc

diff --git a/api/rtp_receiver_interface.cc b/api/rtp_receiver_interface.cc
index 52f72df..615a31d 100644
--- a/api/rtp_receiver_interface.cc
+++ b/api/rtp_receiver_interface.cc
@@ -58,4 +58,7 @@
   return nullptr;
 }
 
+void RtpReceiverInterface::SetJitterBufferMinimumDelay(
+    absl::optional<double> delay_seconds) {}
+
 }  // namespace webrtc

api/rtp_receiver_interface.h

diff --git a/api/rtp_receiver_interface.h b/api/rtp_receiver_interface.h
index 7d9870b..d1ef137 100644
--- a/api/rtp_receiver_interface.h
+++ b/api/rtp_receiver_interface.h
@@ -128,6 +128,14 @@
   // Must call SetObserver(nullptr) before the observer is destroyed.
   virtual void SetObserver(RtpReceiverObserverInterface* observer) = 0;
 
+  // Sets the jitter buffer minimum delay until media playout. Actual observed
+  // delay may differ depending on the congestion control. |delay_seconds| is a
+  // positive value including 0.0 measured in seconds. |nullopt| means default
+  // value must be used. TODO(kuddai): remove the default implmenetation once
+  // the subclasses in Chromium implement this.
+  virtual void SetJitterBufferMinimumDelay(
+      absl::optional<double> delay_seconds);
+
   // TODO(zhihuang): Remove the default implementation once the subclasses
   // implement this. Currently, the only relevant subclass is the
   // content::FakeRtpReceiver in Chromium.
@@ -163,6 +171,7 @@
 PROXY_CONSTMETHOD0(RtpParameters, GetParameters)
 PROXY_METHOD1(bool, SetParameters, const RtpParameters&)
 PROXY_METHOD1(void, SetObserver, RtpReceiverObserverInterface*)
+PROXY_METHOD1(void, SetJitterBufferMinimumDelay, absl::optional<double>)
 PROXY_CONSTMETHOD0(std::vector<RtpSource>, GetSources)
 PROXY_METHOD1(void,
               SetFrameDecryptor,

api/test/mock_rtpreceiver.h

diff --git a/api/test/mock_rtpreceiver.h b/api/test/mock_rtpreceiver.h
index c585cbf..710f8c5 100644
--- a/api/test/mock_rtpreceiver.h
+++ b/api/test/mock_rtpreceiver.h
@@ -30,6 +30,7 @@
   MOCK_CONST_METHOD0(GetParameters, RtpParameters());
   MOCK_METHOD1(SetParameters, bool(const RtpParameters&));
   MOCK_METHOD1(SetObserver, void(RtpReceiverObserverInterface*));
+  MOCK_METHOD1(SetJitterBufferMinimumDelay, void(absl::optional<double>));
   MOCK_CONST_METHOD0(GetSources, std::vector<RtpSource>());
 };