commit 45340ca824b25320c81e82eb95649da266729fb9
author Anders Carlsson <andersc@webrtc.org> Mon Jan 14 13:23:23 2019
committer Commit Bot <commit-bot@chromium.org> Mon Jan 14 14:56:40 2019
tree 584ab33af748850b70fcd8bfb336eba0e65743dd
parent 959e9b6b57ad4086d3a0b879d8533d2dce5acaf4

Remove legacy video codec factories.

Removes the deprecated video codec factories and the related flag and
helper classes.

Bug: webrtc:7925
Change-Id: I0a6d1666ece9ad074fefc79b626ba241765e1b98
Reviewed-on: https://webrtc-review.googlesource.com/c/113940
Commit-Queue: Anders Carlsson <andersc@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Kári Helgason <kthelgason@webrtc.org>
Reviewed-by: Magnus Jedvert <magjed@webrtc.org>
Reviewed-by: Steve Anton <steveanton@webrtc.org>
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#26245}

media/BUILD.gn

diff --git a/media/BUILD.gn b/media/BUILD.gn
index f6a735f..d846562 100644
--- a/media/BUILD.gn
+++ b/media/BUILD.gn
@@ -248,8 +248,6 @@
     "//third_party/abseil-cpp/absl/memory",
   ]
   sources = [
-    "engine/convert_legacy_video_factory.cc",
-    "engine/convert_legacy_video_factory.h",
     "engine/internal_decoder_factory.cc",
     "engine/internal_decoder_factory.h",
     "engine/internal_encoder_factory.cc",
@@ -259,20 +257,10 @@
     "engine/multiplex_codec_factory.cc",
     "engine/multiplex_codec_factory.h",
     "engine/multiplexcodecfactory.h",
-    "engine/scoped_video_decoder.cc",
-    "engine/scoped_video_decoder.h",
-    "engine/scoped_video_encoder.cc",
-    "engine/scoped_video_encoder.h",
-    "engine/scopedvideodecoder.h",
-    "engine/scopedvideoencoder.h",
 
     # TODO(bugs.webrtc.org/7925): stop exporting this header once downstream
     # targets depend on :rtc_encoder_simulcast_proxy directly.
     "engine/encoder_simulcast_proxy.h",
-    "engine/webrtc_video_decoder_factory.h",
-    "engine/webrtc_video_encoder_factory.h",
-    "engine/webrtcvideodecoderfactory.h",
-    "engine/webrtcvideoencoderfactory.h",
   ]
 
   if (!build_with_chromium && is_clang) {
@@ -306,10 +294,7 @@
 
 rtc_static_library("rtc_audio_video") {
   visibility = [ "*" ]
-  allow_poison = [
-    "audio_codecs",  # TODO(bugs.webrtc.org/8396): Remove.
-    "software_video_codecs",  # TODO(bugs.webrtc.org/7925): Remove.
-  ]
+  allow_poison = [ "audio_codecs" ]  # TODO(bugs.webrtc.org/8396): Remove.
   defines = []
   libs = []
   deps = [
@@ -342,17 +327,11 @@
     "engine/simulcast.h",
     "engine/webrtc_media_engine.cc",
     "engine/webrtc_media_engine.h",
-    "engine/webrtc_video_decoder_factory.cc",
-    "engine/webrtc_video_decoder_factory.h",
-    "engine/webrtc_video_encoder_factory.cc",
-    "engine/webrtc_video_encoder_factory.h",
     "engine/webrtc_video_engine.cc",
     "engine/webrtc_video_engine.h",
     "engine/webrtc_voice_engine.cc",
     "engine/webrtc_voice_engine.h",
     "engine/webrtcmediaengine.h",
-    "engine/webrtcvideodecoderfactory.h",
-    "engine/webrtcvideoencoderfactory.h",
     "engine/webrtcvideoengine.h",
     "engine/webrtcvoiceengine.h",
   ]
@@ -382,9 +361,6 @@
   } else {
     deps += [ "../modules/audio_processing/aec_dump:null_aec_dump_factory" ]
   }
-  if (rtc_use_builtin_sw_codecs) {
-    deps += [ ":rtc_internal_video_codecs" ]
-  }
   deps += [
     ":rtc_constants",
     ":rtc_media_base",
@@ -467,10 +443,7 @@
 
 rtc_source_set("rtc_media") {
   visibility = [ "*" ]
-  allow_poison = [
-    "audio_codecs",  # TODO(bugs.webrtc.org/8396): Remove.
-    "software_video_codecs",  # TODO(bugs.webrtc.org/7925): Remove.
-  ]
+  allow_poison = [ "audio_codecs" ]  # TODO(bugs.webrtc.org/8396): Remove.
   deps = [
     ":rtc_audio_video",
     ":rtc_data",
@@ -634,7 +607,6 @@
       "engine/simulcast_encoder_adapter_unittest.cc",
       "engine/simulcast_unittest.cc",
       "engine/webrtc_media_engine_unittest.cc",
-      "engine/webrtc_video_encoder_factory_unittest.cc",
       "engine/webrtc_video_engine_unittest.cc",
     ]