commit 0874530b78cbd4a55d16067186ed6b4dc6429c06
author Jiawei Ou <ouj@fb.com> Tue Feb 12 11:36:13 2019 -0800
committer Commit Bot <commit-bot@chromium.org> Tue Feb 12 20:41:25 2019 +0000
tree 4aa186bd186489349eddba2856dd584bf6a1ee43
parent 3329be4f3d3954b99e7fa35fed5155cc48579740

Add gn configs to remove the dependency to audio and video codecs.

Actually, I don't think there is any strong reason to keep these
deps in `webrtc` target except some downstream projects need it.

Making a GN config for now.

Bug: webrtc:10306
Change-Id: Id714faeabf4daaf3cc88d1f6224ae89ca8715e48
Reviewed-on: https://webrtc-review.googlesource.com/c/122420
Commit-Queue: Jiawei Ou <ouj@fb.com>
Reviewed-by: Niels Moller <nisse@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#26653}

BUILD.gn

diff --git a/BUILD.gn b/BUILD.gn
index 1cb920e..a62bb02 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -406,13 +406,19 @@
       "video",
     ]
 
-    # Include audio and video codecs by default.
-    deps += [
-      "api/audio_codecs:builtin_audio_decoder_factory",
-      "api/audio_codecs:builtin_audio_encoder_factory",
-      "api/video_codecs:builtin_video_decoder_factory",
-      "api/video_codecs:builtin_video_encoder_factory",
-    ]
+    if (rtc_include_builtin_audio_codecs) {
+      deps += [
+        "api/audio_codecs:builtin_audio_decoder_factory",
+        "api/audio_codecs:builtin_audio_encoder_factory",
+      ]
+    }
+
+    if (rtc_include_builtin_video_codecs) {
+      deps += [
+        "api/video_codecs:builtin_video_decoder_factory",
+        "api/video_codecs:builtin_video_encoder_factory",
+      ]
+    }
 
     if (build_with_mozilla) {
       deps += [ "api/video:video_frame" ]