Allow FakeNetworkPipe to drop packets in bursts.

webrtc/test/fake_network_pipe.h

Index: webrtc/test/fake_network_pipe.h
diff --git a/webrtc/test/fake_network_pipe.h b/webrtc/test/fake_network_pipe.h
index 608ff008c9919c4e23bc418806cd56f59fd58380..2dcbcf59d128fbf732adba4e373aeb352741c5e1 100644
--- a/webrtc/test/fake_network_pipe.h
+++ b/webrtc/test/fake_network_pipe.h
@@ -80,6 +80,8 @@ class FakeNetworkPipe {
     int loss_percent = 0;
     // If packets are allowed to be reordered.
     bool allow_reordering = false;
+    // The average length of a burst of lost packets.
+    int avg_burst_loss_length = -1;

[terelius, 2016/05/30 16:26:32]

I think it would be good to make it a double. There doesn't seem to be any
particular reason why the average burst loss should be an integer.

[philipel, 2016/05/30 16:29:25]

I guess it makes more sense to use a double here instead, but I think it would
be better with a follow-up CL where I change that (and maybe loss_percent as
well) instead.

[mflodman, 2016/05/31 07:42:02]

Do we really need this as a double? I think int precision is fine in this case.

[philipel, 2016/05/31 08:39:00]

Don't think we need this as a double, just something that might have been nice.
If anyone needs it they can easily change it...

   };
 
   FakeNetworkPipe(Clock* clock, const FakeNetworkPipe::Config& config);
@@ -133,6 +135,16 @@ class FakeNetworkPipe {
   size_t sent_packets_;
   int64_t total_packet_delay_;
 
+  // Are we currently dropping a burst of packets?
+  bool bursting_;
+
+  // The probability to drop the packet if we are currently dropping a
+  // burst of packet
+  double prob_loss_bursting_;
+
+  // The probability to drop a burst of packets.

[mflodman, 2016/05/31 07:42:02]

It would be good to add something telling the model, e.g. "according to
gilbert-elliot model."

[philipel, 2016/05/31 08:39:00]

Done kind of. I added a comment to the .cc file since we configure
|prob_loss_bursting_| and |prob_start_bursting_| differently depending on which
type of loss we want.

+  double prob_start_bursting_;
+
   int64_t next_process_time_;
 
   RTC_DISALLOW_COPY_AND_ASSIGN(FakeNetworkPipe);