added WebRTC-QuickPerfTest to RampUpTests and CallPerfTests

webrtc/modules/audio_coding/neteq/tools/neteq_performance_test.cc

 /*
  *  Copyright (c) 2014 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.
  */
 
 #include "webrtc/modules/audio_coding/neteq/tools/neteq_performance_test.h"
 
 #include "webrtc/api/audio_codecs/builtin_audio_decoder_factory.h"
 #include "webrtc/base/checks.h"
 #include "webrtc/modules/audio_coding/codecs/pcm16b/pcm16b.h"
 #include "webrtc/modules/audio_coding/neteq/include/neteq.h"
 #include "webrtc/modules/audio_coding/neteq/tools/audio_loop.h"
 #include "webrtc/modules/audio_coding/neteq/tools/rtp_generator.h"
 #include "webrtc/modules/include/module_common_types.h"
 #include "webrtc/system_wrappers/include/clock.h"
+#include "webrtc/system_wrappers/include/field_trial.h"
 #include "webrtc/test/testsupport/fileutils.h"
 #include "webrtc/typedefs.h"
 
 using webrtc::NetEq;
 using webrtc::test::AudioLoop;
 using webrtc::test::RtpGenerator;
 using webrtc::WebRtcRTPHeader;
 
+namespace {
+  const int kQuickTestTimeoutMs = 100000;

[stefan-webrtc, 2017/02/21 09:35:46]

Sounds like a lot to have 100 seconds for a quick test run?

[ilnik, 2017/02/21 09:46:08]

The variable is named runtime_ms, but it is not a realtime duration, but
simulated runtime. No clocks are tied to it. It turns out, simulation is ~300
times faster than realtime. Originally, tests are run with 10 million ms there.
This value of 100000 is already 100 times less than that.

[stefan-webrtc, 2017/02/21 10:26:03]

Ok

I would prefer if you moved this to where the Run() method is being called, so
you can do:

Run(field_trial::FindFullName("WebRTC-QuickPerfTest") == "Enabled" ?
kQuickTestTimeoutMs : kRegularTestTimeoutMs, loss_rate, drift_factor);

That way the caller won't be surprised.

[hlundin-webrtc, 2017/02/21 10:35:04]

Yes, agree with Stefan. This is an unexpected override of the passed parameter.

+}
+
 namespace webrtc {
 namespace test {
 
 int64_t NetEqPerformanceTest::Run(int runtime_ms,
                                   int lossrate,
                                   double drift_factor) {
+  if (field_trial::FindFullName("WebRTC-QuickPerfTest") == "Enabled") {
+    runtime_ms = kQuickTestTimeoutMs;
+  }
   const std::string kInputFileName =
       webrtc::test::ResourcePath("audio_coding/testfile32kHz", "pcm");
   const int kSampRateHz = 32000;
   const webrtc::NetEqDecoder kDecoderType =
       webrtc::NetEqDecoder::kDecoderPCM16Bswb32kHz;
   const std::string kDecoderName = "pcm16-swb32";
   const int kPayloadType = 95;
 
   // Initialize NetEq instance.
   NetEq::Config config;
@@ skipping 79 matching lines @@
       drift_flipped = true;
     }
   }
   int64_t end_time_ms = clock->TimeInMilliseconds();
   delete neteq;
   return end_time_ms - start_time_ms;
 }
 
 }  // namespace test
 }  // namespace webrtc