Added implementation of three classes in BBR,with unit-tests.

webrtc/modules/remote_bitrate_estimator/test/estimators/congestion_window.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.
  *
  */
 
 #ifndef WEBRTC_MODULES_REMOTE_BITRATE_ESTIMATOR_TEST_ESTIMATORS_CONGESTION_WINDOW_H_
 #define WEBRTC_MODULES_REMOTE_BITRATE_ESTIMATOR_TEST_ESTIMATORS_CONGESTION_WINDOW_H_
 
+#include <cstddef>
+#include <cstdint>

[philipel, 2017/07/06 12:15:15]

Do you need to include both of these?

[gnish1, 2017/07/07 13:43:34]

Done.

+
+#include "webrtc/modules/remote_bitrate_estimator/test/estimators/bbr.h"
+
 namespace webrtc {
 namespace testing {
 namespace bwe {
 class CongestionWindow {
  public:
-  void set_gain(float gain);
-  size_t data_inflight();
-  int64_t GetCongestionWindow();
-
+  CongestionWindow();
+  ~CongestionWindow();
+  int64_t GetCongestionWindow(BbrBweSender::Mode mode,

[philipel, 2017/07/06 12:15:15]

Why int64_t as return type? Can the congestion window be 0 or negative, and if
so, what does it mean? Maybe we should rtc::Optional instead?

[gnish1, 2017/07/07 13:43:34]

Done.

+                              int64_t bandwidth_estimate,
+                              int64_t min_rtt,
+                              float gain,
+                              size_t bytes_acked,
+                              int multiplier);
+  int64_t GetTargetCongestionWindow(int64_t bandwidth_estimate,
+                                    int64_t min_rtt,
+                                    float gain);
   // Packet sent from sender, meaning it is inflight
   // until we receive it and we should add packet's size to data_inflight.
-  void PacketSent();
+  void PacketSent(size_t sent_packet_size);
 
   // Ack was received by sender, meaning
   // packet is no longer inflight.
-  void AckReceived();
+  void AckReceived(size_t received_packet_size);
+
+  size_t data_inflight() { return data_inflight_; }
+
+ private:
+  size_t data_inflight_;
 };
 }  // namespace bwe
 }  // namespace testing
 }  // namespace webrtc
 
 #endif  // WEBRTC_MODULES_REMOTE_BITRATE_ESTIMATOR_TEST_ESTIMATORS_CONGESTION_WINDOW_H_