create a separate WebMediaPlayer for URL derived from media stream

webkit/media/media_stream_client.h

 // Copyright (c) 2011 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 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;
 
 namespace media {
 class VideoDecoder;
 class MessageLoopFactory;
 }
 
 namespace webkit_media {
 
 // Define an interface for media stream client to get some information about
 // 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;
 
  protected:
   virtual ~MediaStreamClient() {}
 };
 
 }  // namespace webkit_media
 
 #endif  // WEBKIT_MEDIA_MEDIA_STREAM_CLIENT_H_