Add VideoTrackSink interface to content/public

content/renderer/media/video_track.h

+// Copyright 2013 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.
+
+#ifndef CONTENT_RENDERER_MEDIA_VIDEO_TRACK_H_
+#define CONTENT_RENDERER_MEDIA_VIDEO_TRACK_H_
+
+#include "base/compiler_specific.h"
+#include "base/memory/ref_counted.h"
+#include "base/memory/scoped_vector.h"
+#include "content/common/content_export.h"
+#include "content/public/renderer/media/video_track_sink.h"
+#include "content/renderer/media/media_stream_track_extra_data.h"
+
+namespace blink {
+class WebMediaStreamTrack;
+}
+
+namespace webrtc {
+class VideoTrackInterface;
+}
+
+namespace content {
+
+class WebRtcVideoSinkAdapter;
+
+class CONTENT_EXPORT VideoTrack : public MediaStreamTrackExtraData {
+ public:
+  VideoTrack(webrtc::VideoTrackInterface* track);

[no longer working on chromium, 2013/11/25 13:55:07]

nit, explicit

[perkj_chrome, 2013/11/26 09:16:38]

Done.

+  virtual ~VideoTrack();
+  void RegisterSink(VideoTrackSink* sink);
+  void UnRegisterSink(VideoTrackSink* sink);
+
+ private:
+  // The webrtc video track.
+  webrtc::VideoTrackInterface* video_track_;
+  ScopedVector<WebRtcVideoSinkAdapter> sinks_;

[no longer working on chromium, 2013/11/25 13:55:07]

so we are creating one WebRtcVideoSinkAdapter for each sink, I thought we would
only do this for the sink connected to libjingle peer connection. 

[perkj_chrome, 2013/11/26 09:16:38]

Yes- that is the end goal. But that requires huge changes to this is an
intermediate step.
The end goal is to only use this adapter on remote tracks.

+
+  DISALLOW_COPY_AND_ASSIGN(VideoTrack);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_RENDERER_MEDIA_VIDEO_TRACK_H_