Interface with webrtc through encoded frames

remoting/protocol/webrtc_video_renderer_adapter.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "remoting/protocol/webrtc_video_renderer_adapter.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/callback.h"
@@ skipping 28 matching lines @@
 
   if (video_tracks.size() > 1U) {
     LOG(WARNING) << "Received media stream with multiple video tracks.";
   }
 
   video_tracks[0]->AddOrUpdateSink(this, rtc::VideoSinkWants());
 }
 
 WebrtcVideoRendererAdapter::~WebrtcVideoRendererAdapter() {
   DCHECK(task_runner_->BelongsToCurrentThread());
+
+  webrtc::VideoTrackVector video_tracks = media_stream_->GetVideoTracks();
+  if (video_tracks.empty())

[Sergey Ulanov, 2016/03/31 22:06:19]

I think this can be replaced with a DCHECK()

[Irfan, 2016/04/05 21:23:28]

Done.

+    return;
+  video_tracks[0]->RemoveSink(this);
 }
 
 void WebrtcVideoRendererAdapter::OnFrame(const cricket::VideoFrame& frame) {
   // TODO(sergeyu): WebRTC calls OnFrame on a separate thread it creates.
   // FrameConsumer normally expects to be called on the network thread, so we
   // cannot call FrameConsumer::AllocateFrame() here and instead
   // BasicDesktopFrame is created directly. This will not work correctly with
   // all FrameConsumer implementations. Fix this somehow.
   scoped_ptr<webrtc::DesktopFrame> rgb_frame(new webrtc::BasicDesktopFrame(
       webrtc::DesktopSize(frame.GetWidth(), frame.GetHeight())));
 
   frame.ConvertToRgbBuffer(
       output_format_fourcc_, rgb_frame->data(),
       std::abs(rgb_frame->stride()) * rgb_frame->size().height(),
       rgb_frame->stride());
   rgb_frame->mutable_updated_region()->AddRect(
       webrtc::DesktopRect::MakeSize(rgb_frame->size()));
   task_runner_->PostTask(
       FROM_HERE,
       base::Bind(&WebrtcVideoRendererAdapter::DrawFrame,
                  weak_factory_.GetWeakPtr(), base::Passed(&rgb_frame)));
 }
 
 void WebrtcVideoRendererAdapter::DrawFrame(
     scoped_ptr<webrtc::DesktopFrame> frame) {
   DCHECK(task_runner_->BelongsToCurrentThread());
   frame_consumer_->DrawFrame(std::move(frame), base::Closure());
 }
 
+}  // namespace protocol
 }  // namespace remoting
-}  // namespace protocol