commit 1f433e46dba79d616b444ede00534751030d3cd7
author Anders Carlsson <andersc@webrtc.org> Tue Apr 24 16:39:05 2018 +0200
committer Commit Bot <commit-bot@chromium.org> Wed Apr 25 11:34:33 2018 +0000
tree 4826cb0e0eef2bfa5d2adba3c015a55e14c73fdd
parent cd7da92012f8a7d149eae4669a7e8bb97c9d8a22

Mark built-in software video codecs as poisonous.

The goal is to make these injectable, and only VP8 and VP9 specific
targets should depend on them.

Bug: webrtc:7925
Change-Id: Ie9239a54d197fe70c93de0582797211fef6997a2
Reviewed-on: https://webrtc-review.googlesource.com/72082
Commit-Queue: Anders Carlsson <andersc@webrtc.org>
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#23021}

sdk/android/BUILD.gn

diff --git a/sdk/android/BUILD.gn b/sdk/android/BUILD.gn
index 1d9bb86..5717036 100644
--- a/sdk/android/BUILD.gn
+++ b/sdk/android/BUILD.gn
@@ -607,7 +607,10 @@
 
 rtc_static_library("libjingle_peerconnection_jni") {
   visibility = [ "*" ]
-  allow_poison = [ "audio_codecs" ]  # TODO(bugs.webrtc.org/8396): Remove.
+  allow_poison = [
+    "audio_codecs",  # TODO(bugs.webrtc.org/8396): Remove.
+    "software_video_codecs",  # TODO(bugs.webrtc.org/7925): Remove.
+  ]
   public_deps = [
     ":audio_jni",
     ":base_jni",
@@ -1174,7 +1177,10 @@
 # objects.
 rtc_static_library("native_api_codecs") {
   visibility = [ "*" ]
-  allow_poison = [ "audio_codecs" ]  # TODO(bugs.webrtc.org/8396): Remove.
+  allow_poison = [
+    "audio_codecs",  # TODO(bugs.webrtc.org/8396): Remove.
+    "software_video_codecs",  # TODO(bugs.webrtc.org/7925): Remove.
+  ]
   sources = [
     "native_api/codecs/wrapper.cc",
     "native_api/codecs/wrapper.h",
@@ -1219,7 +1225,10 @@
 # video interfaces from their Java equivalents.
 rtc_static_library("native_api_video") {
   visibility = [ "*" ]
-  allow_poison = [ "audio_codecs" ]  # TODO(bugs.webrtc.org/8396): Remove.
+  allow_poison = [
+    "audio_codecs",  # TODO(bugs.webrtc.org/8396): Remove.
+    "software_video_codecs",  # TODO(bugs.webrtc.org/7925): Remove.
+  ]
   sources = [
     "native_api/video/wrapper.cc",
     "native_api/video/wrapper.h",