Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(77)

Side by Side Diff: webrtc/modules/audio_processing/test/debug_dump_test.cc

Issue 1937153002: Revert of Disable failing modules_unittests for UBSan. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « webrtc/modules/audio_processing/audio_processing_unittest.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 244 matching lines...) Expand 10 before | Expand all | Expand 10 after
255 msg->output_channel(0).size()); 255 msg->output_channel(0).size());
256 for (int i = 0; i < msg->output_channel_size(); ++i) { 256 for (int i = 0; i < msg->output_channel_size(); ++i) {
257 ASSERT_EQ(0, memcmp(output->channels()[i], 257 ASSERT_EQ(0, memcmp(output->channels()[i],
258 msg->output_channel(i).data(), 258 msg->output_channel(i).data(),
259 msg->output_channel(i).size())); 259 msg->output_channel(i).size()));
260 } 260 }
261 } 261 }
262 } 262 }
263 } 263 }
264 264
265 // Disabled for UBSan: https://bugs.chromium.org/p/webrtc/issues/detail?id=5820
266 #ifndef UNDEFINED_SANITIZER
267 TEST_F(DebugDumpTest, SimpleCase) { 265 TEST_F(DebugDumpTest, SimpleCase) {
268 Config config; 266 Config config;
269 DebugDumpGenerator generator(config); 267 DebugDumpGenerator generator(config);
270 generator.StartRecording(); 268 generator.StartRecording();
271 generator.Process(100); 269 generator.Process(100);
272 generator.StopRecording(); 270 generator.StopRecording();
273 VerifyDebugDump(generator.dump_file_name()); 271 VerifyDebugDump(generator.dump_file_name());
274 } 272 }
275 273
276 TEST_F(DebugDumpTest, ChangeInputFormat) { 274 TEST_F(DebugDumpTest, ChangeInputFormat) {
(...skipping 243 matching lines...) Expand 10 before | Expand all | Expand 10 after
520 TEST_F(DebugDumpTest, TransientSuppressionOn) { 518 TEST_F(DebugDumpTest, TransientSuppressionOn) {
521 Config config; 519 Config config;
522 config.Set<ExperimentalNs>(new ExperimentalNs(true)); 520 config.Set<ExperimentalNs>(new ExperimentalNs(true));
523 DebugDumpGenerator generator(config); 521 DebugDumpGenerator generator(config);
524 generator.StartRecording(); 522 generator.StartRecording();
525 generator.Process(100); 523 generator.Process(100);
526 generator.StopRecording(); 524 generator.StopRecording();
527 VerifyDebugDump(generator.dump_file_name()); 525 VerifyDebugDump(generator.dump_file_name());
528 } 526 }
529 527
530 #endif // !UNDEFINED_SANITIZER
531
532 } // namespace test 528 } // namespace test
533 } // namespace webrtc 529 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/modules/audio_processing/audio_processing_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698