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

Unified Diff: webrtc/audio/BUILD.gn

Issue 2694203002: Low-bandwidth audio testing (Closed)
Patch Set: Move BoundedWavFileWriter and add unittests for it Created 3 years, 9 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 | « no previous file | webrtc/audio/DEPS » ('j') | webrtc/audio/test/low_bandwidth_audio_test.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/audio/BUILD.gn
diff --git a/webrtc/audio/BUILD.gn b/webrtc/audio/BUILD.gn
index a6a22e652cce5cad47874baebe00acf9c1649cc2..6095daa39ba5962e9bf28205cb6213f8129f464d 100644
--- a/webrtc/audio/BUILD.gn
+++ b/webrtc/audio/BUILD.gn
@@ -78,6 +78,10 @@ if (rtc_include_tests) {
"//testing/gmock",
"//testing/gtest",
]
+ if (rtc_enable_protobuf) {
+ deps += [ ":low_bandwidth_audio_test" ]
+ }
+
if (!build_with_chromium && is_clang) {
# Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
@@ -90,12 +94,30 @@ if (rtc_include_tests) {
sources = [
"test/low_bandwidth_audio_test.cc",
+ "test/low_bandwidth_audio_test.h",
]
- deps = []
-
+ deps = [
+ "../common_audio",
+ "../system_wrappers",
+ "../test:fake_audio_device",
+ "../test:run_test",
+ "../test:test_common",
+ "../test:test_main",
+ ]
if (is_android) {
- deps += [ "//testing/android/native_test:native_test_support" ]
+ deps += [ "//testing/android/native_test:native_test_native_code" ]
+ } else {
+ deps += [ "../modules/video_capture:video_capture_internal_impl" ]
stefan-webrtc 2017/03/20 14:40:10 Why do we need a video capturer for the audio test
oprypin_webrtc 2017/03/20 15:09:16 I suspect that call_test pulls that in, since it a
stefan-webrtc 2017/03/20 15:42:32 Shouldn't it be added as a dependency on the targe
oprypin_webrtc 2017/03/20 16:02:35 Done. Thanks.
+ }
+
+ data = [
+ "//resources/voice_engine/audio_tiny16.wav",
+ ]
+
+ if (!build_with_chromium && is_clang) {
+ # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163)
+ suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
}
}
}
« no previous file with comments | « no previous file | webrtc/audio/DEPS » ('j') | webrtc/audio/test/low_bandwidth_audio_test.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698