From 8a5edb2568e0f21cec3a02c50429dca712910729 Mon Sep 17 00:00:00 2001 From: Alex Loiko Date: Mon, 18 Jun 2018 17:25:32 +0200 Subject: [PATCH] Always enable 'delay-agnostic' in APM fuzzer. This 'fixes' a bug in the non-delay-agnostic code by not fuzzing it. We plan to always enable the delay-agnostic feature. In Chrome, delay-agnostic mode is always on: https://cs.chromium.org/chromium/src/content/renderer/media/stream/media_stream_audio_processor.cc?l=579 Bug: chromium:824638 webrtc:9423 Change-Id: I3d9cac2bc11857fd55549d13c52db4c99dec956c Reviewed-on: https://webrtc-review.googlesource.com/83984 Commit-Queue: Alex Loiko Reviewed-by: Sam Zackrisson Cr-Commit-Position: refs/heads/master@{#23651} --- test/fuzzers/audio_processing_configs_fuzzer.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/fuzzers/audio_processing_configs_fuzzer.cc b/test/fuzzers/audio_processing_configs_fuzzer.cc index d0e58af5db..41b3c716aa 100644 --- a/test/fuzzers/audio_processing_configs_fuzzer.cc +++ b/test/fuzzers/audio_processing_configs_fuzzer.cc @@ -47,7 +47,7 @@ std::unique_ptr CreateApm(test::FuzzDataHelper* fuzz_data, bool bf = fuzz_data->ReadOrDefaultValue(true); bool ef = fuzz_data->ReadOrDefaultValue(true); bool raf = fuzz_data->ReadOrDefaultValue(true); - bool da = fuzz_data->ReadOrDefaultValue(true); + static_cast(fuzz_data->ReadOrDefaultValue(true)); bool ie = fuzz_data->ReadOrDefaultValue(true); bool red = fuzz_data->ReadOrDefaultValue(true); bool hpf = fuzz_data->ReadOrDefaultValue(true); @@ -108,7 +108,7 @@ std::unique_ptr CreateApm(test::FuzzDataHelper* fuzz_data, } config.Set(new ExtendedFilter(ef)); config.Set(new RefinedAdaptiveFilter(raf)); - config.Set(new DelayAgnostic(da)); + config.Set(new DelayAgnostic(true)); config.Set(new Intelligibility(ie)); std::unique_ptr apm(