commit c0e4d45ce0468487b20bed908484d3ab6b16bbbd
author Sebastian Jansson <srte@webrtc.org> Thu Oct 25 13:08:32 2018
committer Commit Bot <commit-bot@chromium.org> Thu Oct 25 13:51:11 2018
tree e84a00359bdd2d345cff424250186be474c66313
parent 4ba6c26623a2ba1c59adf2171308b92d04f6f7a1

Adds BitrateAllocation struct to OnBitrateUpdated.

This prepares for adding parameters to OnBitrateUpdated. By using a
struct, additional fields doesn't require a change in the signature and
only the obeservers that use the new fields will be affected by the
change.

Bug: webrtc:9718
Change-Id: I7dd6c9577afd77af06da5f56aea312356f80f9c0
Reviewed-on: https://webrtc-review.googlesource.com/c/107727
Commit-Queue: Sebastian Jansson <srte@webrtc.org>
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25366}

call/bitrate_allocator_unittest.cc

diff --git a/call/bitrate_allocator_unittest.cc b/call/bitrate_allocator_unittest.cc
index 2961fd4..1fc6cd6 100644
--- a/call/bitrate_allocator_unittest.cc
+++ b/call/bitrate_allocator_unittest.cc
@@ -59,15 +59,12 @@
     protection_ratio_ = protection_ratio;
   }
 
-  uint32_t OnBitrateUpdated(uint32_t bitrate_bps,
-                            uint8_t fraction_loss,
-                            int64_t rtt,
-                            int64_t probing_interval_ms) override {
-    last_bitrate_bps_ = bitrate_bps;
-    last_fraction_loss_ = fraction_loss;
-    last_rtt_ms_ = rtt;
-    last_probing_interval_ms_ = probing_interval_ms;
-    return bitrate_bps * protection_ratio_;
+  uint32_t OnBitrateUpdated(BitrateAllocationUpdate update) override {
+    last_bitrate_bps_ = update.bitrate_bps;
+    last_fraction_loss_ = update.fraction_loss;
+    last_rtt_ms_ = update.rtt;
+    last_probing_interval_ms_ = update.bwe_period_ms;
+    return update.bitrate_bps * protection_ratio_;
   }
   uint32_t last_bitrate_bps_;
   uint8_t last_fraction_loss_;