commit 795dbe4e0fa78333d52fe39edb08a0b13a281c34
author Peter Boström <pbos@webrtc.org> Fri Nov 27 14:09:07 2015 +0100
committer Peter Boström <pbos@webrtc.org> Fri Nov 27 13:09:14 2015 +0000
tree 950f90bc66966784070066f33189a8935e3768cd
parent 34873b5bb0dd11385c1aa3964113747f8b3b460c

Remove RegisterExternal{De,En}coder error codes.

Also adds a RTC_CHECK in VideoReceiveStream that verifies that decoders
aren't null, since this will attempt to deregister a codec which would
previously fail with an obscure stack trace not indicating what actually
was wrong.

BUG=webrtc:5249
R=stefan@webrtc.org

Review URL: https://codereview.webrtc.org/1479793002 .

Cr-Commit-Position: refs/heads/master@{#10821}

webrtc/modules/video_coding/video_coding_impl.cc

diff --git a/webrtc/modules/video_coding/video_coding_impl.cc b/webrtc/modules/video_coding/video_coding_impl.cc
index 1b6a893..c9ed6f4 100644
--- a/webrtc/modules/video_coding/video_coding_impl.cc
+++ b/webrtc/modules/video_coding/video_coding_impl.cc
@@ -127,8 +127,9 @@
   int32_t RegisterExternalEncoder(VideoEncoder* externalEncoder,
                                   uint8_t payloadType,
                                   bool internalSource) override {
-    return sender_.RegisterExternalEncoder(externalEncoder, payloadType,
-                                           internalSource);
+    sender_.RegisterExternalEncoder(externalEncoder, payloadType,
+                                    internalSource);
+    return 0;
   }
 
   int Bitrate(unsigned int* bitrate) const override {
@@ -195,11 +196,11 @@
                                           requireKeyFrame);
   }
 
-  int32_t RegisterExternalDecoder(VideoDecoder* externalDecoder,
-                                  uint8_t payloadType,
-                                  bool internalRenderTiming) override {
-    return receiver_.RegisterExternalDecoder(externalDecoder, payloadType,
-                                             internalRenderTiming);
+  void RegisterExternalDecoder(VideoDecoder* externalDecoder,
+                               uint8_t payloadType,
+                               bool internalRenderTiming) override {
+    receiver_.RegisterExternalDecoder(externalDecoder, payloadType,
+                                      internalRenderTiming);
   }
 
   int32_t RegisterReceiveCallback(