Removed all RTP dependencies from ViEChannel and renamed class.

webrtc/video/video_stream_decoder.cc

+/*
+ *  Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
+ *
+ *  Use of this source code is governed by a BSD-style license
+ *  that can be found in the LICENSE file in the root of the source
+ *  tree. An additional intellectual property rights grant can be found
+ *  in the file PATENTS.  All contributing project authors may
+ *  be found in the AUTHORS file in the root of the source tree.
+ */
+
+#include "webrtc/video/video_stream_decoder.h"
+
+#include <algorithm>
+#include <map>
+#include <vector>
+
+#include "webrtc/base/checks.h"
+#include "webrtc/base/logging.h"
+#include "webrtc/common_video/include/frame_callback.h"
+#include "webrtc/common_video/include/incoming_video_stream.h"
+#include "webrtc/modules/video_coding/video_coding_impl.h"
+#include "webrtc/modules/video_processing/include/video_processing.h"
+#include "webrtc/system_wrappers/include/metrics.h"
+#include "webrtc/video/call_stats.h"
+#include "webrtc/video/payload_router.h"
+#include "webrtc/video/receive_statistics_proxy.h"
+
+namespace webrtc {
+
+VideoStreamDecoder::VideoStreamDecoder(
+    vcm::VideoReceiver* video_receiver,
+    VCMFrameTypeCallback* vcm_frame_type_callback,
+    VCMPacketRequestCallback* vcm_packet_request_callback,
+    bool enable_nack,
+    ReceiveStatisticsProxy* receive_statistics_proxy,
+    IncomingVideoStream* incoming_video_stream,
+    I420FrameCallback* pre_render_callback)
+    : video_receiver_(video_receiver),
+      receive_stats_callback_(receive_statistics_proxy),
+      incoming_video_stream_(incoming_video_stream),
+      pre_render_callback_(pre_render_callback),
+      last_rtt_ms_(0) {
+  RTC_DCHECK(video_receiver_);
+
+  static const int kMaxPacketAgeToNack = 450;
+  static const int kMaxNackListSize = 250;
+  video_receiver_->SetNackSettings(kMaxNackListSize,
+                                   kMaxPacketAgeToNack, 0);
+  video_receiver_->RegisterReceiveCallback(this);
+  video_receiver_->RegisterFrameTypeCallback(vcm_frame_type_callback);
+  video_receiver_->RegisterReceiveStatisticsCallback(this);
+  video_receiver_->RegisterDecoderTimingCallback(this);
+  static const int kDefaultRenderDelayMs = 10;
+  video_receiver_->SetRenderDelay(kDefaultRenderDelayMs);
+
+  VCMVideoProtection video_protection = enable_nack ? kProtectionNack
+                                                    : kProtectionNone;

[stefan-webrtc, 2016/05/03 07:48:08]

We should inform about FEC here as well if both NACK and FEC is enabled.

philipel has a CL which does that here https://codereview.webrtc.org/1942683003/
which will conflict with your CL.

[mflodman, 2016/05/03 07:56:52]

Acknowledged.

+  VCMDecodeErrorMode decode_error_mode = enable_nack ? kNoErrors : kWithErrors;
+  video_receiver_->SetVideoProtection(video_protection, true);
+  video_receiver_->SetDecodeErrorMode(decode_error_mode);
+  VCMPacketRequestCallback* packet_request_callback =
+      enable_nack ? vcm_packet_request_callback : nullptr;
+  video_receiver_->RegisterPacketRequestCallback(packet_request_callback);
+}
+
+VideoStreamDecoder::~VideoStreamDecoder() {}
+
+// Do not acquire the lock of |video_receiver_| in this function. Decode
+// callback won't necessarily be called from the decoding thread. The decoding
+// thread may have held the lock when calling VideoDecoder::Decode, Reset, or
+// Release. Acquiring the same lock in the path of decode callback can deadlock.
+int32_t VideoStreamDecoder::FrameToRender(VideoFrame& video_frame) {  // NOLINT
+  if (pre_render_callback_)
+    pre_render_callback_->FrameCallback(&video_frame);
+
+  // TODO(pbos): Remove stream id argument.
+  incoming_video_stream_->RenderFrame(0xFFFFFFFF, video_frame);
+  return 0;
+}
+
+int32_t VideoStreamDecoder::ReceivedDecodedReferenceFrame(
+  const uint64_t picture_id) {
+  RTC_NOTREACHED();
+  return 0;
+}
+
+void VideoStreamDecoder::OnIncomingPayloadType(int payload_type) {
+  receive_stats_callback_->OnIncomingPayloadType(payload_type);
+}
+
+void VideoStreamDecoder::OnDecoderImplementationName(
+    const char* implementation_name) {
+  receive_stats_callback_->OnDecoderImplementationName(implementation_name);
+}
+
+void VideoStreamDecoder::OnReceiveRatesUpdated(uint32_t bit_rate,
+                                               uint32_t frame_rate) {
+  receive_stats_callback_->OnIncomingRate(frame_rate, bit_rate);
+}
+
+void VideoStreamDecoder::OnDiscardedPacketsUpdated(int discarded_packets) {
+  receive_stats_callback_->OnDiscardedPacketsUpdated(discarded_packets);
+}
+
+void VideoStreamDecoder::OnFrameCountsUpdated(const FrameCounts& frame_counts) {
+  receive_stats_callback_->OnFrameCountsUpdated(frame_counts);
+}
+
+void VideoStreamDecoder::OnDecoderTiming(int decode_ms,
+                                         int max_decode_ms,
+                                         int current_delay_ms,
+                                         int target_delay_ms,
+                                         int jitter_buffer_ms,
+                                         int min_playout_delay_ms,
+                                         int render_delay_ms) {
+  int last_rtt = -1;
+  {
+    rtc::CritScope lock(&crit_);
+    last_rtt = last_rtt_ms_;
+  }
+
+  receive_stats_callback_->OnDecoderTiming(
+      decode_ms, max_decode_ms, current_delay_ms, target_delay_ms,
+      jitter_buffer_ms, min_playout_delay_ms, render_delay_ms, last_rtt);
+}
+
+void VideoStreamDecoder::OnRttUpdate(int64_t avg_rtt_ms, int64_t max_rtt_ms) {
+  video_receiver_->SetReceiveChannelParameters(max_rtt_ms);
+
+  rtc::CritScope lock(&crit_);
+  last_rtt_ms_ = avg_rtt_ms;
+}
+}  // namespace webrtc