Remove dependencies on modules:module_api from AudioProcessing.
- Directly include api/audio/audio_frame.h everywhere AudioFrame is used. - This *will* remove transient dependencies on libjpeg and a bunch of other things from the e.g. APM. - audio_frame.h still included from module_common_types.h for backwards compatibility with clients. Bug: webrtc:9139, webrtc:7504 Change-Id: Id96f9268c01667fbcc29a01f5c1dd25a37836897 Reviewed-on: https://webrtc-review.googlesource.com/62464 Commit-Queue: Fredrik Solenberg <solenberg@webrtc.org> Reviewed-by: Karl Wiberg <kwiberg@webrtc.org> Cr-Commit-Position: refs/heads/master@{#22845}
This commit is contained in:
committed by
Commit Bot
parent
0ab56511f1
commit
bbf21a3fd6
@ -11,7 +11,7 @@
|
||||
#ifndef MODULES_AUDIO_CODING_NETEQ_TOOLS_AUDIO_SINK_H_
|
||||
#define MODULES_AUDIO_CODING_NETEQ_TOOLS_AUDIO_SINK_H_
|
||||
|
||||
#include "modules/include/module_common_types.h"
|
||||
#include "api/audio/audio_frame.h"
|
||||
#include "rtc_base/constructormagic.h"
|
||||
#include "typedefs.h" // NOLINT(build/include)
|
||||
|
||||
|
||||
@ -15,7 +15,6 @@
|
||||
|
||||
#include "api/audio_codecs/audio_encoder.h"
|
||||
#include "modules/audio_coding/neteq/tools/neteq_input.h"
|
||||
#include "modules/include/module_common_types.h"
|
||||
|
||||
namespace webrtc {
|
||||
namespace test {
|
||||
|
||||
@ -17,6 +17,7 @@
|
||||
#include <limits>
|
||||
#include <utility>
|
||||
|
||||
#include "modules/include/module_common_types.h"
|
||||
#include "rtc_base/checks.h"
|
||||
|
||||
namespace webrtc {
|
||||
|
||||
@ -11,6 +11,7 @@
|
||||
|
||||
#include "modules/audio_coding/neteq/tools/neteq_external_decoder_test.h"
|
||||
|
||||
#include "api/audio/audio_frame.h"
|
||||
#include "api/audio_codecs/builtin_audio_decoder_factory.h"
|
||||
#include "rtc_base/format_macros.h"
|
||||
#include "test/gtest.h"
|
||||
|
||||
@ -17,7 +17,6 @@
|
||||
#include "api/audio_codecs/audio_decoder.h"
|
||||
#include "common_types.h" // NOLINT(build/include)
|
||||
#include "modules/audio_coding/neteq/include/neteq.h"
|
||||
#include "modules/include/module_common_types.h"
|
||||
|
||||
namespace webrtc {
|
||||
namespace test {
|
||||
|
||||
@ -10,13 +10,13 @@
|
||||
|
||||
#include "modules/audio_coding/neteq/tools/neteq_performance_test.h"
|
||||
|
||||
#include "api/audio/audio_frame.h"
|
||||
#include "api/audio_codecs/builtin_audio_decoder_factory.h"
|
||||
#include "common_types.h" // NOLINT(build/include)
|
||||
#include "modules/audio_coding/codecs/pcm16b/pcm16b.h"
|
||||
#include "modules/audio_coding/neteq/include/neteq.h"
|
||||
#include "modules/audio_coding/neteq/tools/audio_loop.h"
|
||||
#include "modules/audio_coding/neteq/tools/rtp_generator.h"
|
||||
#include "modules/include/module_common_types.h"
|
||||
#include "rtc_base/checks.h"
|
||||
#include "system_wrappers/include/clock.h"
|
||||
#include "test/testsupport/fileutils.h"
|
||||
@ -103,7 +103,7 @@ int64_t NetEqPerformanceTest::Run(int runtime_ms,
|
||||
return -1;
|
||||
payload_len = WebRtcPcm16b_Encode(input_samples.data(),
|
||||
input_samples.size(), input_payload);
|
||||
assert(payload_len == kInputBlockSizeSamples * sizeof(int16_t));
|
||||
RTC_DCHECK_EQ(payload_len, kInputBlockSizeSamples * sizeof(int16_t));
|
||||
}
|
||||
|
||||
// Get output audio, but don't do anything with it.
|
||||
@ -113,8 +113,7 @@ int64_t NetEqPerformanceTest::Run(int runtime_ms,
|
||||
if (error != NetEq::kOK)
|
||||
return -1;
|
||||
|
||||
assert(out_frame.samples_per_channel_ ==
|
||||
static_cast<size_t>(kSampRateHz * 10 / 1000));
|
||||
RTC_DCHECK_EQ(out_frame.samples_per_channel_, (kSampRateHz * 10) / 1000);
|
||||
|
||||
static const int kOutputBlockSizeMs = 10;
|
||||
time_now_ms += kOutputBlockSizeMs;
|
||||
|
||||
@ -19,7 +19,6 @@
|
||||
#include "modules/audio_coding/neteq/tools/audio_sink.h"
|
||||
#include "modules/audio_coding/neteq/tools/input_audio_file.h"
|
||||
#include "modules/audio_coding/neteq/tools/rtp_generator.h"
|
||||
#include "modules/include/module_common_types.h"
|
||||
#include "rtc_base/flags.h"
|
||||
#include "test/gtest.h"
|
||||
#include "typedefs.h" // NOLINT(build/include)
|
||||
|
||||
@ -32,7 +32,6 @@
|
||||
#include "modules/audio_coding/neteq/tools/output_audio_file.h"
|
||||
#include "modules/audio_coding/neteq/tools/output_wav_file.h"
|
||||
#include "modules/audio_coding/neteq/tools/rtp_file_source.h"
|
||||
#include "modules/include/module_common_types.h"
|
||||
#include "rtc_base/checks.h"
|
||||
#include "rtc_base/flags.h"
|
||||
#include "test/testsupport/fileutils.h"
|
||||
|
||||
@ -14,7 +14,6 @@
|
||||
|
||||
#include <memory>
|
||||
|
||||
#include "modules/include/module_common_types.h"
|
||||
#include "modules/rtp_rtcp/include/rtp_header_parser.h"
|
||||
#include "rtc_base/checks.h"
|
||||
|
||||
|
||||
@ -21,6 +21,7 @@
|
||||
#include <map>
|
||||
#include <string>
|
||||
|
||||
#include "api/audio/audio_frame.h"
|
||||
#include "api/audio_codecs/L16/audio_encoder_L16.h"
|
||||
#include "api/audio_codecs/g711/audio_encoder_g711.h"
|
||||
#include "api/audio_codecs/g722/audio_encoder_g722.h"
|
||||
|
||||
Reference in New Issue
Block a user