Revert of Disable failing modules_unittests for UBSan.

webrtc/modules/audio_processing/audio_processing_unittest.cc

 /*
  *  Copyright (c) 2012 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 1829 matching lines @@
   EXPECT_NE(0, feof(out_file));
   EXPECT_NE(0, feof(limited_file));
   ASSERT_EQ(0, fclose(ref_file));
   ASSERT_EQ(0, fclose(out_file));
   ASSERT_EQ(0, fclose(limited_file));
   remove(ref_filename.c_str());
   remove(out_filename.c_str());
   remove(limited_filename.c_str());
 }
 
-// Disabled for UBSan: https://bugs.chromium.org/p/webrtc/issues/detail?id=5820
-#ifdef UNDEFINED_SANITIZER
-#define MAYBE_VerifyDebugDumpInt DISABLED_VerifyDebugDumpInt
-#else
-#define MAYBE_VerifyDebugDumpInt VerifyDebugDumpInt
-#endif
-TEST_F(ApmTest, MAYBE_VerifyDebugDumpInt) {
+TEST_F(ApmTest, VerifyDebugDumpInt) {
   VerifyDebugDumpTest(kIntFormat);
 }
 
-// Disabled for UBSan: https://bugs.chromium.org/p/webrtc/issues/detail?id=5820
-#ifdef UNDEFINED_SANITIZER
-#define MAYBE_VerifyDebugDumpFloat DISABLED_VerifyDebugDumpFloat
-#else
-#define MAYBE_VerifyDebugDumpFloat VerifyDebugDumpFloat
-#endif
-TEST_F(ApmTest, MAYBE_VerifyDebugDumpFloat) {
+TEST_F(ApmTest, VerifyDebugDumpFloat) {
   VerifyDebugDumpTest(kFloatFormat);
 }
 #endif
 
 // TODO(andrew): expand test to verify output.
-// Disabled for UBSan: https://bugs.chromium.org/p/webrtc/issues/detail?id=5820
-#ifdef UNDEFINED_SANITIZER
-#define MAYBE_DebugDump DISABLED_DebugDump
-#else
-#define MAYBE_DebugDump DebugDump
-#endif
-TEST_F(ApmTest, MAYBE_DebugDump) {
+TEST_F(ApmTest, DebugDump) {
   const std::string filename =
       test::TempFilename(test::OutputPath(), "debug_aec");
   EXPECT_EQ(apm_->kNullPointerError,
             apm_->StartDebugRecording(static_cast<const char*>(NULL), -1));
 
 #ifdef WEBRTC_AUDIOPROC_DEBUG_DUMP
   // Stopping without having started should be OK.
   EXPECT_EQ(apm_->kNoError, apm_->StopDebugRecording());
 
   EXPECT_EQ(apm_->kNoError, apm_->StartDebugRecording(filename.c_str(), -1));
@@ skipping 12 matching lines @@
   EXPECT_EQ(apm_->kUnsupportedFunctionError,
             apm_->StartDebugRecording(filename.c_str(), -1));
   EXPECT_EQ(apm_->kUnsupportedFunctionError, apm_->StopDebugRecording());
 
   // Verify the file has NOT been written.
   ASSERT_TRUE(fopen(filename.c_str(), "r") == NULL);
 #endif  // WEBRTC_AUDIOPROC_DEBUG_DUMP
 }
 
 // TODO(andrew): expand test to verify output.
-// Disabled for UBSan: https://bugs.chromium.org/p/webrtc/issues/detail?id=5820
-#ifdef UNDEFINED_SANITIZER
-#define MAYBE_DebugDumpFromFileHandle DISABLED_DebugDumpFromFileHandle
-#else
-#define MAYBE_DebugDumpFromFileHandle DebugDumpFromFileHandle
-#endif
-TEST_F(ApmTest, MAYBE_DebugDumpFromFileHandle) {
+TEST_F(ApmTest, DebugDumpFromFileHandle) {
   FILE* fid = NULL;
   EXPECT_EQ(apm_->kNullPointerError, apm_->StartDebugRecording(fid, -1));
   const std::string filename =
       test::TempFilename(test::OutputPath(), "debug_aec");
   fid = fopen(filename.c_str(), "w");
   ASSERT_TRUE(fid);
 
 #ifdef WEBRTC_AUDIOPROC_DEBUG_DUMP
   // Stopping without having started should be OK.
   EXPECT_EQ(apm_->kNoError, apm_->StopDebugRecording());
@@ skipping 848 matching lines @@
                     std::tr1::make_tuple(16000, 32000, 48000, 32000, 25, 35),
                     std::tr1::make_tuple(16000, 32000, 32000, 32000, 25, 0),
                     std::tr1::make_tuple(16000, 32000, 16000, 32000, 25, 20),
                     std::tr1::make_tuple(16000, 16000, 48000, 16000, 35, 20),
                     std::tr1::make_tuple(16000, 16000, 32000, 16000, 35, 20),
                     std::tr1::make_tuple(16000, 16000, 16000, 16000, 0, 0)));
 #endif
 
 }  // namespace
 }  // namespace webrtc