removed five redundant avsync tests to make webrtc_perf_test faster

webrtc/call/call_perf_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.
  */
 #include <algorithm>
@@ skipping 359 matching lines @@
   voe_base->Release();
   voe_codec->Release();
   voe_network->Release();
   voe_sync->Release();
 
   DestroyCalls();
 
   VoiceEngine::Delete(voice_engine);
 }
 
-TEST_F(CallPerfTest, PlaysOutAudioAndVideoInSyncWithAudioCreatedFirst) {
-  TestAudioVideoSync(false, true, DriftingClock::kNoDrift,
-                     DriftingClock::kNoDrift, DriftingClock::kNoDrift);
-}
-
-TEST_F(CallPerfTest, PlaysOutAudioAndVideoInSyncWithVideoCreatedFirst) {
-  TestAudioVideoSync(false, false, DriftingClock::kNoDrift,
-                     DriftingClock::kNoDrift, DriftingClock::kNoDrift);
-}
-
-TEST_F(CallPerfTest, PlaysOutAudioAndVideoInSyncWithFec) {
-  TestAudioVideoSync(true, false, DriftingClock::kNoDrift,
-                     DriftingClock::kNoDrift, DriftingClock::kNoDrift);
-}
-
 // TODO(danilchap): Reenable after adding support for frame capture clock
 // that is not in sync with local TickTime clock.
 TEST_F(CallPerfTest, DISABLED_PlaysOutAudioAndVideoInSyncWithVideoNtpDrift) {
   TestAudioVideoSync(false, true, DriftingClock::PercentsFaster(10.0f),
                      DriftingClock::kNoDrift, DriftingClock::kNoDrift);
 }
 
-TEST_F(CallPerfTest, PlaysOutAudioAndVideoInSyncWithAudioRtpDrift) {
-  TestAudioVideoSync(false, true, DriftingClock::kNoDrift,
-                     DriftingClock::kNoDrift,
-                     DriftingClock::PercentsFaster(30.0f));
-}
-
-TEST_F(CallPerfTest, PlaysOutAudioAndVideoInSyncWithVideoRtpDrift) {
-  TestAudioVideoSync(false, true, DriftingClock::kNoDrift,
-                     DriftingClock::PercentsFaster(30.0f),
-                     DriftingClock::kNoDrift);
-}
-
 TEST_F(CallPerfTest, PlaysOutAudioAndVideoInSyncWithAudioFasterThanVideoDrift) {
   TestAudioVideoSync(false, true, DriftingClock::kNoDrift,
                      DriftingClock::PercentsSlower(30.0f),
                      DriftingClock::PercentsFaster(30.0f));
 }
 
 TEST_F(CallPerfTest, PlaysOutAudioAndVideoInSyncWithVideoFasterThanAudioDrift) {
-  TestAudioVideoSync(false, true, DriftingClock::kNoDrift,
+  TestAudioVideoSync(true, false, DriftingClock::kNoDrift,

[pbos-webrtc, 2016/02/15 09:18:20]

Put a comment above this test and the one below to say that they are different
in terms of what stream gets added first.

[danilchap, 2016/02/15 09:45:36]

Proper code doesn't need comments
Done with constants instead.

                      DriftingClock::PercentsFaster(30.0f),
                      DriftingClock::PercentsSlower(30.0f));
 }
 
 void CallPerfTest::TestCaptureNtpTime(const FakeNetworkPipe::Config& net_config,
                                       int threshold_ms,
                                       int start_time_ms,
                                       int run_time_ms) {
   class CaptureNtpTimeObserver : public test::EndToEndTest,
                                  public VideoRenderer {
@@ skipping 364 matching lines @@
     int encoder_inits_;
     uint32_t last_set_bitrate_;
     VideoSendStream* send_stream_;
     VideoEncoderConfig encoder_config_;
   } test;
 
   RunBaseTest(&test);
 }
 
 }  // namespace webrtc