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

Side by Side Diff: webrtc/modules/audio_device/BUILD.gn

Issue 2629923002: GN: Refactor modules_unittests to eliminate package boundary violations. (Closed)
Patch Set: Addressing comments. 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 unified diff | Download patch
« no previous file with comments | « webrtc/modules/audio_conference_mixer/BUILD.gn ('k') | webrtc/modules/audio_mixer/BUILD.gn » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2014 The WebRTC project authors. All Rights Reserved. 1 # Copyright (c) 2014 The WebRTC project authors. All Rights Reserved.
2 # 2 #
3 # Use of this source code is governed by a BSD-style license 3 # Use of this source code is governed by a BSD-style license
4 # that can be found in the LICENSE file in the root of the source 4 # that can be found in the LICENSE file in the root of the source
5 # tree. An additional intellectual property rights grant can be found 5 # tree. An additional intellectual property rights grant can be found
6 # in the file PATENTS. All contributing project authors may 6 # in the file PATENTS. All contributing project authors may
7 # be found in the AUTHORS file in the root of the source tree. 7 # be found in the AUTHORS file in the root of the source tree.
8 8
9 import("../../build/webrtc.gni") 9 import("../../build/webrtc.gni")
10 10
(...skipping 247 matching lines...) Expand 10 before | Expand all | Expand 10 after
258 cflags = [ 258 cflags = [
259 # TODO(phoglund): get rid of 4373 supression when 259 # TODO(phoglund): get rid of 4373 supression when
260 # http://code.google.com/p/webrtc/issues/detail?id=261 is solved. 260 # http://code.google.com/p/webrtc/issues/detail?id=261 is solved.
261 # legacy warning for ignoring const / volatile in signatures. 261 # legacy warning for ignoring const / volatile in signatures.
262 "/wd4373", 262 "/wd4373",
263 ] 263 ]
264 } 264 }
265 } 265 }
266 266
267 if (rtc_include_tests) { 267 if (rtc_include_tests) {
268 rtc_source_set("audio_device_unittests") {
269 testonly = true
270 sources = [
271 "fine_audio_buffer_unittest.cc",
272 ]
273 deps = [
274 ":audio_device",
275 ":mock_audio_device",
276 "../../base:rtc_base_approved",
277 "../../system_wrappers:system_wrappers",
278 "../../test:test_support",
279 "../utility:utility",
280 "//testing/gmock",
281 ]
282 if (is_android) {
283 # Need to disable error due to the line in
284 # base/android/jni_android.h triggering it:
285 # const BASE_EXPORT jobject GetApplicationContext()
286 # error: type qualifiers ignored on function return type
287 cflags = [ "-Wno-ignored-qualifiers" ]
288 sources += [
289 "android/audio_device_unittest.cc",
290 "android/audio_manager_unittest.cc",
291 "android/ensure_initialized.cc",
292 "android/ensure_initialized.h",
293 ]
294 deps += [
295 "../../../base",
296 "//webrtc/sdk/android:libjingle_peerconnection_java",
297 ]
298 }
299 if (is_ios) {
300 sources += [ "ios/objc/RTCAudioSessionTest.mm" ]
301 configs += [ "//build/config/compiler:enable_arc" ]
302 if (target_cpu != "x64") {
303 sources += [ "ios/audio_device_unittest_ios.cc" ]
304 }
305 }
306 if (!build_with_chromium && is_clang) {
307 # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
308 suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
309 }
310 }
311
268 rtc_source_set("mock_audio_device") { 312 rtc_source_set("mock_audio_device") {
269 testonly = true 313 testonly = true
270 sources = [ 314 sources = [
271 "include/mock_audio_device.h", 315 "include/mock_audio_device.h",
272 "include/mock_audio_transport.h", 316 "include/mock_audio_transport.h",
273 ] 317 ]
274 deps = [ 318 deps = [
275 ":audio_device", 319 ":audio_device",
276 "../../test:test_support", 320 "../../test:test_support",
277 ] 321 ]
278 all_dependent_configs = [ ":mock_audio_device_config" ] 322 all_dependent_configs = [ ":mock_audio_device_config" ]
279 } 323 }
280 }
281 324
282 # These tests do not work on ios, see 325 if (!is_ios) {
283 # https://bugs.chromium.org/p/webrtc/issues/detail?id=4755 326 # These tests do not work on ios, see
284 if (rtc_include_tests && !is_ios) { 327 # https://bugs.chromium.org/p/webrtc/issues/detail?id=4755
285 rtc_executable("audio_device_tests") { 328 rtc_executable("audio_device_tests") {
286 testonly = true 329 testonly = true
287 sources = [ 330 sources = [
288 "test/audio_device_test_api.cc", 331 "test/audio_device_test_api.cc",
289 "test/audio_device_test_defines.h", 332 "test/audio_device_test_defines.h",
290 ] 333 ]
291 deps = [ 334 deps = [
292 ":audio_device", 335 ":audio_device",
293 "../..:webrtc_common", 336 "../..:webrtc_common",
294 "../../system_wrappers", 337 "../../system_wrappers",
295 "../../test:test_main", 338 "../../test:test_main",
296 "../../test:test_support", 339 "../../test:test_support",
297 "../rtp_rtcp", 340 "../rtp_rtcp",
298 "../utility", 341 "../utility",
299 "//testing/gtest", 342 "//testing/gtest",
300 ] 343 ]
301 public_configs = [ ":audio_device_config" ] 344 public_configs = [ ":audio_device_config" ]
345 }
302 } 346 }
303 } 347 }
304 348
305 if (!build_with_chromium && is_android) { 349 if (!build_with_chromium && is_android) {
306 android_shared_srcjar("audio_device_java") { 350 android_shared_srcjar("audio_device_java") {
307 sources = [ 351 sources = [
308 "android/java/src/org/webrtc/voiceengine/BuildInfo.java", 352 "android/java/src/org/webrtc/voiceengine/BuildInfo.java",
309 "android/java/src/org/webrtc/voiceengine/WebRtcAudioEffects.java", 353 "android/java/src/org/webrtc/voiceengine/WebRtcAudioEffects.java",
310 "android/java/src/org/webrtc/voiceengine/WebRtcAudioManager.java", 354 "android/java/src/org/webrtc/voiceengine/WebRtcAudioManager.java",
311 "android/java/src/org/webrtc/voiceengine/WebRtcAudioRecord.java", 355 "android/java/src/org/webrtc/voiceengine/WebRtcAudioRecord.java",
312 "android/java/src/org/webrtc/voiceengine/WebRtcAudioTrack.java", 356 "android/java/src/org/webrtc/voiceengine/WebRtcAudioTrack.java",
313 "android/java/src/org/webrtc/voiceengine/WebRtcAudioUtils.java", 357 "android/java/src/org/webrtc/voiceengine/WebRtcAudioUtils.java",
314 ] 358 ]
315 } 359 }
316 } 360 }
OLDNEW
« no previous file with comments | « webrtc/modules/audio_conference_mixer/BUILD.gn ('k') | webrtc/modules/audio_mixer/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698