Add VideoTrackSink interface to content/public

content/public/renderer/media_stream_sink.h

Index: content/public/renderer/media_stream_sink.h
diff --git a/content/public/renderer/media_stream_sink.h b/content/public/renderer/media_stream_sink.h
new file mode 100644
index 0000000000000000000000000000000000000000..d910e9c7e69a51b24a4607aa3944add9896f6187
--- /dev/null
+++ b/content/public/renderer/media_stream_sink.h
@@ -0,0 +1,25 @@
+// 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_PUBLIC_RENDERER_MEDIA_STREAM_SINK_H_
+#define CONTENT_PUBLIC_RENDERER_MEDIA_STREAM_SINK_H_
+
+#include "base/compiler_specific.h"
+#include "content/common/content_export.h"
+#include "third_party/WebKit/public/platform/WebMediaStreamSource.h"
+
+namespace content {
+
+class CONTENT_EXPORT MediaStreamSink {

[tommi (sloooow) - chröme, 2013/11/27 16:53:07]

t'would be nice to have some comments

[perkj_chrome, 2013/11/28 13:32:45]

Done.

+ public:
+  virtual void OnReadyStateChanged(
+      blink::WebMediaStreamSource::ReadyState state) {}

[tommi (sloooow) - chröme, 2013/11/27 16:53:07]

= 0?
since there's only one method in the interface currently, it seems like nobody
would want to inherit from it without overriding this function.

[Jói, 2013/11/27 18:35:51]

The interfaces MediaStreamVideoSink and MediaStreamAudioSink which inherit from
this one have more methods and somebody implementing them might not want to
implement OnReadyStateChanged.

[perkj_chrome, 2013/11/28 13:32:45]

I agree with Joi. This is not needed by all implementations but belong to a
common base class. 

[tommi (sloooow) - chröme, 2013/11/28 14:32:21]

OK, sounds good.

+
+ protected:
+  virtual ~MediaStreamSink() {}
+};
+
+}  // namespace content
+
+#endif  // CONTENT_PUBLIC_RENDERER_MEDIA_STREAM_SINK_H_