Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(63)

Unified Diff: webkit/media/webmediaplayer_impl.h

Issue 13813016: Remove reference counting from media::Demuxer and friends. (Closed) Base URL: http://git.chromium.org/chromium/src.git@vd_scoped
Patch Set: fix tools Created 7 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webkit/media/webmediaplayer_impl.h
diff --git a/webkit/media/webmediaplayer_impl.h b/webkit/media/webmediaplayer_impl.h
index 46f998588e3566fc94952d34e6157eec24585419..ff63ae195c027bb7d23aa36b3b56c2b1de0af9ce 100644
--- a/webkit/media/webmediaplayer_impl.h
+++ b/webkit/media/webmediaplayer_impl.h
@@ -53,6 +53,7 @@ class MessageLoopProxy;
namespace media {
class ChunkDemuxer;
+class FFmpegDemuxer;
class MediaLog;
}
@@ -230,7 +231,9 @@ class WebMediaPlayerImpl
void NotifyDownloading(bool is_downloading);
// Finishes starting the pipeline due to a call to load().
- void StartPipeline();
+ //
+ // A non-null |media_source| will construct a Media Source pipeline.
+ void StartPipeline(WebKit::WebMediaSource* media_source);
// Helpers that set the network/ready state and notifies the client if
// they've changed.
@@ -272,10 +275,6 @@ class WebMediaPlayerImpl
// painted.
void FrameReady(const scoped_refptr<media::VideoFrame>& frame);
- // Builds a FilterCollection based on the current configuration of
- // WebMediaPlayerImpl.
- scoped_ptr<media::FilterCollection> BuildFilterCollection();
-
WebKit::WebFrame* frame_;
// TODO(hclam): get rid of these members and read from the pipeline directly.
@@ -350,7 +349,8 @@ class WebMediaPlayerImpl
// |data_source_| is used for regular resource loads.
// |chunk_demuxer_| is used for Media Source resource loads.
xhwang 2013/04/20 00:09:54 You have 3 now, update the comment a bit?
scherkus (not reviewing) 2013/04/22 04:02:08 Done.
scoped_refptr<BufferedDataSource> data_source_;
- scoped_refptr<media::ChunkDemuxer> chunk_demuxer_;
+ scoped_ptr<media::Demuxer> demuxer_;
+ media::ChunkDemuxer* chunk_demuxer_;
// Temporary for EME v0.1. In the future the init data type should be passed
// through GenerateKeyRequest() directly from WebKit.

Powered by Google App Engine
This is Rietveld 408576698