Chromium Code Reviews| 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" ] | 
| } | 
| } | 
| } |