Added implementation of four functions in the BBR congestion controller.

webrtc/modules/remote_bitrate_estimator/test/estimators/bbr.h

 /*
  *  Copyright (c) 2017 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  *
  */
@@ skipping 12 matching lines @@
 #include "webrtc/modules/remote_bitrate_estimator/test/bwe.h"
 
 namespace webrtc {
 namespace testing {
 namespace bwe {
 class MaxBandwidthFilter;
 class MinRttFilter;
 class CongestionWindow;
 class BbrBweSender : public BweSender {
  public:
-  BbrBweSender();
+  explicit BbrBweSender(Clock* clock);
   virtual ~BbrBweSender();
   enum Mode {
     // Startup phase.
     STARTUP,
     // Queue draining phase, which where created during startup.
     DRAIN,
     // Cruising, probing new bandwidth.
     PROBE_BW,
     // Temporarily limiting congestion window size in order to measure
     // minimum RTT.
@@ skipping 15 matching lines @@
 
  private:
   void EnterStartup();
   bool UpdateBandwidthAndMinRtt();
   void TryExitingStartup();
   void TryExitingDrain(int64_t now);
   void EnterProbeBw(int64_t now);
   void EnterProbeRtt(int64_t now);
   void TryExitingProbeRtt(int64_t now);
   void TryUpdatingCyclePhase(int64_t now);
+  void TryEnteringOrExitingProbeRtt(int64_t now);
+  Clock* const clock_;
+  Mode mode_;
+  std::unique_ptr<MaxBandwidthFilter> max_bandwidth_filter_;
+  uint64_t rt_count_;

[terelius, 2017/06/07 08:11:22]

round_trip_count_ ?

[gnish1, 2017/06/07 09:25:56]

Should be round_count,updated.

+  uint64_t last_packet_sent_;
+  uint64_t round_trip_end_;
+  float pacing_gain_;
+  float congestion_window_gain_;
+  bool full_bandwidth_reached_;

[terelius, 2017/06/07 08:11:22]

I don't understand what this variable represents. Please either rename of add a
comment.

[gnish1, 2017/06/07 09:25:56]

Done.

 };
 
 class BbrBweReceiver : public BweReceiver {
  public:
   explicit BbrBweReceiver(int flow_id);
   virtual ~BbrBweReceiver();
   void ReceivePacket(int64_t arrival_time_ms,
                      const MediaPacket& media_packet) override;
   FeedbackPacket* GetFeedback(int64_t now_ms) override;
 
  private:
   SimulatedClock clock_;
   std::vector<std::pair<uint64_t, int64_t>> packet_feedbacks_;
 };
 }  // namespace bwe
 }  // namespace testing
 }  // namespace webrtc
 
 #endif  // WEBRTC_MODULES_REMOTE_BITRATE_ESTIMATOR_TEST_ESTIMATORS_BBR_H_