Remove webrtc deps from AppRTCMobile.
We want the example app to only link agains the framework. This ensures
that we are actually testing the framework, and that AppRTCMobile
doesn't require any other parts of WebRTC not included in the framework.
Bug: webrtc:7759
Change-Id: Ib04aae0bc3ab2a1a508eaf4a4f15c2d37f521598
Reviewed-on: https://chromium-review.googlesource.com/522722
Reviewed-by: Magnus Jedvert <magjed@webrtc.org>
Commit-Queue: Kári Tristan Helgason <kthelgason@webrtc.org>
Cr-Original-Commit-Position: refs/heads/master@{#18470}
Cr-Mirrored-From: https://chromium.googlesource.com/external/webrtc
Cr-Mirrored-Commit: 946923a21fad8e7b98c27d53f13a32d9a8513749
diff --git a/examples/BUILD.gn b/examples/BUILD.gn
index 555a9f9..e6138f1 100644
--- a/examples/BUILD.gn
+++ b/examples/BUILD.gn
@@ -163,10 +163,14 @@
public_configs = [ ":apprtc_common_config" ]
deps = [
- "//webrtc/sdk:objc_common",
"//webrtc/system_wrappers:field_trial_default",
"//webrtc/system_wrappers:metrics_default",
]
+ if (is_ios) {
+ deps += [ ":AppRTCMobile_ios_frameworks" ]
+ } else {
+ deps += [ "//webrtc/sdk:objc_common" ]
+ }
}
config("apprtc_signaling_config") {
@@ -231,9 +235,13 @@
":apprtc_common",
":socketrocket",
]
- public_deps = [
- "//webrtc/sdk:objc_peerconnection",
- ]
+ if (is_ios) {
+ deps += [ ":AppRTCMobile_ios_frameworks" ]
+ } else {
+ public_deps = [
+ "//webrtc/sdk:objc_peerconnection",
+ ]
+ }
libs = [ "QuartzCore.framework" ]
}
@@ -260,9 +268,13 @@
]
deps = [
+ ":AppRTCMobile_ios_frameworks",
":apprtc_common",
":apprtc_signaling",
- "//webrtc/modules/audio_device",
+
+ # TODO(kthelgason): Remove this dep when audio_device has been
+ # moved to sdk.
+ "//webrtc/modules/audio_device:audio_device",
]
}