Remove DISABLED_ON_ macros.

webrtc/modules/audio_coding/neteq/neteq_stereo_unittest.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.
  */
 
 // Test to verify correct stereo and multi-channel operation.
 
 #include <algorithm>
 #include <string>
 #include <list>
 
 #include "testing/gtest/include/gtest/gtest.h"
 #include "webrtc/base/scoped_ptr.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/input_audio_file.h"
 #include "webrtc/modules/audio_coding/neteq/tools/rtp_generator.h"
 #include "webrtc/test/testsupport/fileutils.h"
-#include "webrtc/test/testsupport/gtest_disable.h"
 
 namespace webrtc {
 
 struct TestParameters {
   int frame_size;
   int sample_rate;
   int num_channels;
 };
 
 // This is a parameterized test. The test parameters are supplied through a
@@ skipping 234 matching lines @@
  protected:
   NetEqStereoTestNoJitter()
       : NetEqStereoTest() {
     // Start the sender 100 ms before the receiver to pre-fill the buffer.
     // This is to avoid doing preemptive expand early in the test.
     // TODO(hlundin): Mock the decision making instead to control the modes.
     last_arrival_time_ = -100;
   }
 };
 
-TEST_P(NetEqStereoTestNoJitter, DISABLED_ON_ANDROID(RunTest)) {
+#if defined(WEBRTC_ANDROID)
+#define MAYBE_RunTest DISABLED_RunTest
+#else
+#define MAYBE_RunTest RunTest
+#endif
+TEST_P(NetEqStereoTestNoJitter, MAYBE_RunTest) {
   RunTest(8);
 }
 
 class NetEqStereoTestPositiveDrift : public NetEqStereoTest {
  protected:
   NetEqStereoTestPositiveDrift()
       : NetEqStereoTest(),
         drift_factor(0.9) {
     // Start the sender 100 ms before the receiver to pre-fill the buffer.
     // This is to avoid doing preemptive expand early in the test.
     // TODO(hlundin): Mock the decision making instead to control the modes.
     last_arrival_time_ = -100;
   }
   virtual int GetArrivalTime(int send_time) {
     int arrival_time = last_arrival_time_ +
         drift_factor * (send_time - last_send_time_);
     last_send_time_ = send_time;
     last_arrival_time_ = arrival_time;
     return arrival_time;
   }
 
   double drift_factor;
 };
 
-TEST_P(NetEqStereoTestPositiveDrift, DISABLED_ON_ANDROID(RunTest)) {
+TEST_P(NetEqStereoTestPositiveDrift, MAYBE_RunTest) {
   RunTest(100);
 }
 
 class NetEqStereoTestNegativeDrift : public NetEqStereoTestPositiveDrift {
  protected:
   NetEqStereoTestNegativeDrift()
       : NetEqStereoTestPositiveDrift() {
     drift_factor = 1.1;
     last_arrival_time_ = 0;
   }
 };
 
-TEST_P(NetEqStereoTestNegativeDrift, DISABLED_ON_ANDROID(RunTest)) {
+TEST_P(NetEqStereoTestNegativeDrift, MAYBE_RunTest) {
   RunTest(100);
 }
 
 class NetEqStereoTestDelays : public NetEqStereoTest {
  protected:
   static const int kDelayInterval = 10;
   static const int kDelay = 1000;
   NetEqStereoTestDelays()
       : NetEqStereoTest(),
         frame_index_(0) {
   }
 
   virtual int GetArrivalTime(int send_time) {
     // Deliver immediately, unless we have a back-log.
     int arrival_time = std::min(last_arrival_time_, send_time);
     if (++frame_index_ % kDelayInterval == 0) {
       // Delay this packet.
       arrival_time += kDelay;
     }
     last_send_time_ = send_time;
     last_arrival_time_ = arrival_time;
     return arrival_time;
   }
 
   int frame_index_;
 };
 
-TEST_P(NetEqStereoTestDelays, DISABLED_ON_ANDROID(RunTest)) {
+TEST_P(NetEqStereoTestDelays, MAYBE_RunTest) {
   RunTest(1000);
 }
 
 class NetEqStereoTestLosses : public NetEqStereoTest {
  protected:
   static const int kLossInterval = 10;
   NetEqStereoTestLosses()
       : NetEqStereoTest(),
         frame_index_(0) {
   }
 
   virtual bool Lost() {
     return (++frame_index_) % kLossInterval == 0;
   }
 
   int frame_index_;
 };
 
-TEST_P(NetEqStereoTestLosses, DISABLED_ON_ANDROID(RunTest)) {
+// TODO(pbos): Enable on non-Android, this went failing while being accidentally
+// disabled on all platforms and not just Android.
+// https://bugs.chromium.org/p/webrtc/issues/detail?id=5387
+TEST_P(NetEqStereoTestLosses, DISABLED_RunTest) {
   RunTest(100);
 }
 
 
 // Creates a list of parameter sets.
 std::list<TestParameters> GetTestParameters() {
   std::list<TestParameters> l;
   const int sample_rates[] = {8000, 16000, 32000};
   const int num_rates = sizeof(sample_rates) / sizeof(sample_rates[0]);
   // Loop through sample rates.
@@ skipping 39 matching lines @@
 
 INSTANTIATE_TEST_CASE_P(MultiChannel,
                         NetEqStereoTestDelays,
                         ::testing::ValuesIn(GetTestParameters()));
 
 INSTANTIATE_TEST_CASE_P(MultiChannel,
                         NetEqStereoTestLosses,
                         ::testing::ValuesIn(GetTestParameters()));
 
 }  // namespace webrtc