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

Unified Diff: webrtc/modules/audio_processing/BUILD.gn

Issue 2626163004: GN: Refactor so that WebRTC compiles with rtc_enable_protobuf=false. (Closed)
Patch Set: Created 3 years, 11 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webrtc/logging/BUILD.gn ('k') | webrtc/tools/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/modules/audio_processing/BUILD.gn
diff --git a/webrtc/modules/audio_processing/BUILD.gn b/webrtc/modules/audio_processing/BUILD.gn
index 5cbd97d90a50d2590640965a9fbc0203694e3827..53c350f0ac87cfd922941dd6ef0d5b2cf7d415af 100644
--- a/webrtc/modules/audio_processing/BUILD.gn
+++ b/webrtc/modules/audio_processing/BUILD.gn
@@ -394,12 +394,10 @@ if (rtc_include_tests) {
group("audio_processing_tests") {
testonly = true
public_deps = [
- ":audioproc_f",
":audioproc_test_utils",
":click_annotate",
":nonlinear_beamformer_test",
":transient_suppression_test",
- ":unpack_aecdump",
]
if (rtc_enable_intelligibility_enhancer) {
@@ -407,7 +405,11 @@ if (rtc_include_tests) {
}
if (rtc_enable_protobuf) {
- public_deps += [ ":audioproc_unittest_proto" ]
+ public_deps += [
+ ":audioproc_f",
+ ":audioproc_unittest_proto",
+ ":unpack_aecdump",
+ ]
}
}
@@ -440,51 +442,53 @@ if (rtc_include_tests) {
}
}
- rtc_executable("unpack_aecdump") {
- testonly = true
- sources = [
- "test/unpack.cc",
- ]
+ if (rtc_enable_protobuf) {
+ rtc_executable("unpack_aecdump") {
+ testonly = true
+ sources = [
+ "test/unpack.cc",
+ ]
- deps = [
- ":audio_processing",
- ":audioproc_debug_proto",
- ":audioproc_protobuf_utils",
- ":audioproc_test_utils",
- "../..:webrtc_common",
- "../../base:rtc_base_approved",
- "../../common_audio",
- "../../system_wrappers:system_wrappers_default",
- "//third_party/gflags:gflags",
- ]
- } # unpack_aecdump
+ deps = [
+ ":audio_processing",
+ ":audioproc_debug_proto",
+ ":audioproc_protobuf_utils",
+ ":audioproc_test_utils",
+ "../..:webrtc_common",
+ "../../base:rtc_base_approved",
+ "../../common_audio",
+ "../../system_wrappers:system_wrappers_default",
+ "//third_party/gflags:gflags",
+ ]
+ } # unpack_aecdump
- rtc_executable("audioproc_f") {
- testonly = true
- sources = [
- "test/aec_dump_based_simulator.cc",
- "test/aec_dump_based_simulator.h",
- "test/audio_processing_simulator.cc",
- "test/audio_processing_simulator.h",
- "test/audioproc_float.cc",
- "test/wav_based_simulator.cc",
- "test/wav_based_simulator.h",
- ]
+ rtc_executable("audioproc_f") {
+ testonly = true
+ sources = [
+ "test/aec_dump_based_simulator.cc",
+ "test/aec_dump_based_simulator.h",
+ "test/audio_processing_simulator.cc",
+ "test/audio_processing_simulator.h",
+ "test/audioproc_float.cc",
+ "test/wav_based_simulator.cc",
+ "test/wav_based_simulator.h",
+ ]
- deps = [
- ":audio_processing",
- ":audioproc_debug_proto",
- ":audioproc_protobuf_utils",
- ":audioproc_test_utils",
- "../../base:rtc_base_approved",
- "../../common_audio:common_audio",
- "../../system_wrappers",
- "../../system_wrappers:system_wrappers_default",
- "../../test:test_support",
- "//testing/gtest",
- "//third_party/gflags:gflags",
- ]
- } # audioproc_f
+ deps = [
+ ":audio_processing",
+ ":audioproc_debug_proto",
+ ":audioproc_protobuf_utils",
+ ":audioproc_test_utils",
+ "../../base:rtc_base_approved",
+ "../../common_audio:common_audio",
+ "../../system_wrappers",
+ "../../system_wrappers:system_wrappers_default",
+ "../../test:test_support",
+ "//testing/gtest",
+ "//third_party/gflags:gflags",
+ ]
+ } # audioproc_f
+ }
rtc_source_set("audioproc_test_utils") {
testonly = true
« no previous file with comments | « webrtc/logging/BUILD.gn ('k') | webrtc/tools/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698