create a separate WebMediaPlayer for URL derived from media stream

webkit/media/media_stream_client.h

Index: webkit/media/media_stream_client.h
===================================================================
--- webkit/media/media_stream_client.h	(revision 154837)
+++ webkit/media/media_stream_client.h	(working copy)
@@ -5,7 +5,9 @@
 #ifndef WEBKIT_MEDIA_MEDIA_STREAM_CLIENT_H_
 #define WEBKIT_MEDIA_MEDIA_STREAM_CLIENT_H_
 
+#include "base/callback.h"
 #include "base/memory/ref_counted.h"
+#include "webkit/media/video_frame_provider.h"

[scherkus (not reviewing), 2012/09/07 11:44:03]

instead of #include use a fwd decl

[wjia(left Chromium), 2012/09/13 01:22:07]

This header file is needed because of RepaintCB is defined there.

 
 class GURL;
 
@@ -20,6 +22,13 @@
 // the media stream.
 class MediaStreamClient {
  public:
+  virtual bool IsMediaStream(const GURL& url) = 0;

[scherkus (not reviewing), 2012/09/07 11:44:03]

docs for interface methods?

[wjia(left Chromium), 2012/09/13 01:22:07]

Done.

+
+  virtual scoped_refptr<VideoFrameProvider> GetVideoFrameProvider(
+      const GURL& url,
+      const base::Closure& error_cb,
+      const RepaintCB& repaint_cb) = 0;
+
   virtual scoped_refptr<media::VideoDecoder> GetVideoDecoder(
       const GURL& url,
       media::MessageLoopFactory* message_loop_factory) = 0;