In rampup tests, set start time when starting poller thread.

webrtc/video/rampup_tests.cc

 /*
  *  Copyright (c) 2013 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 37 matching lines @@
     : EndToEndTest(test::CallTest::kLongTimeoutMs),
       event_(false, false),
       clock_(Clock::GetRealTimeClock()),
       num_streams_(num_streams),
       rtx_(rtx),
       red_(red),
       send_stream_(nullptr),
       start_bitrate_bps_(start_bitrate_bps),
       start_bitrate_verified_(false),
       expected_bitrate_bps_(0),
-      test_start_ms_(clock_->TimeInMilliseconds()),
+      test_start_ms_(-1),
       ramp_up_finished_ms_(-1),
       extension_type_(extension_type),
       ssrcs_(GenerateSsrcs(num_streams, 100)),
       rtx_ssrcs_(GenerateSsrcs(num_streams, 200)),
       poller_thread_(ThreadWrapper::CreateThread(&BitrateStatsPollingThread,
                                                  this,
                                                  "BitrateStatsPollingThread")),
       sender_call_(nullptr) {
   if (rtx_) {
     for (size_t i = 0; i < ssrcs_.size(); ++i)
@@ skipping 150 matching lines @@
   *total_sent += stream.rtp_stats.transmitted.TotalBytes() +
                  stream.rtp_stats.retransmitted.TotalBytes() +
                  stream.rtp_stats.fec.TotalBytes();
   *padding_sent += stream.rtp_stats.transmitted.padding_bytes +
                    stream.rtp_stats.retransmitted.padding_bytes +
                    stream.rtp_stats.fec.padding_bytes;
   *media_sent += stream.rtp_stats.MediaPayloadBytes();
 }
 
 void RampUpTester::TriggerTestDone() {
+  RTC_DCHECK_GE(test_start_ms_, 0);
+
   VideoSendStream::Stats send_stats = send_stream_->GetStats();
 
   size_t total_packets_sent = 0;
   size_t total_sent = 0;
   size_t padding_sent = 0;
   size_t media_sent = 0;
   for (uint32_t ssrc : ssrcs_) {
     AccumulateStats(send_stats.substreams[ssrc], &total_packets_sent,
                     &total_sent, &padding_sent, &media_sent);
   }
@@ skipping 16 matching lines @@
   ReportResult("ramp-up-rtx-total-sent", rtx_total_sent, "bytes");
   ReportResult("ramp-up-rtx-media-sent", rtx_media_sent, "bytes");
   ReportResult("ramp-up-rtx-padding-sent", rtx_padding_sent, "bytes");
   if (ramp_up_finished_ms_ >= 0) {
     ReportResult("ramp-up-time", ramp_up_finished_ms_ - test_start_ms_,
                  "milliseconds");
   }
 }
 
 void RampUpTester::PerformTest() {
+  test_start_ms_ = clock_->TimeInMilliseconds();
   poller_thread_->Start();
   if (Wait() != kEventSignaled) {
     printf("Timed out while waiting for ramp-up to complete.");
     return;
   }
   TriggerTestDone();
   poller_thread_->Stop();
 }
 
 RampUpDownUpTester::RampUpDownUpTester(size_t num_streams,
@@ skipping 216 matching lines @@
   RampUpTester test(3, 0, RtpExtension::kTransportSequenceNumber, true, true);
   RunBaseTest(&test);
 }
 
 TEST_F(RampUpTest, TransportSequenceNumberSingleStreamWithHighStartBitrate) {
   RampUpTester test(1, 0.9 * kSingleStreamTargetBps,
                     RtpExtension::kTransportSequenceNumber, false, false);
   RunBaseTest(&test);
 }
 }  // namespace webrtc