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

Unified Diff: webkit/media/webmediaplayer_proxy.cc

Issue 10123005: Revert 132965 - this change seems to break media_unittests (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 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
« no previous file with comments | « webkit/media/webmediaplayer_proxy.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/media/webmediaplayer_proxy.cc
===================================================================
--- webkit/media/webmediaplayer_proxy.cc (revision 132968)
+++ webkit/media/webmediaplayer_proxy.cc (working copy)
@@ -187,24 +187,15 @@
chunk_demuxer_->FlushData();
}
-media::ChunkDemuxer::Status WebMediaPlayerProxy::DemuxerAddId(
- const std::string& id,
- const std::string& type) {
- return chunk_demuxer_->AddId(id, type);
+bool WebMediaPlayerProxy::DemuxerAppend(const uint8* data, size_t length) {
+ if (chunk_demuxer_.get())
+ return chunk_demuxer_->AppendData(data, length);
+ return false;
}
-bool WebMediaPlayerProxy::DemuxerRemoveId(const std::string& id) {
- return chunk_demuxer_->RemoveId(id);
-}
-
-bool WebMediaPlayerProxy::DemuxerAppend(const std::string& id,
- const uint8* data,
- size_t length) {
- return chunk_demuxer_->AppendData(id, data, length);
-}
-
void WebMediaPlayerProxy::DemuxerEndOfStream(media::PipelineStatus status) {
- chunk_demuxer_->EndOfStream(status);
+ if (chunk_demuxer_.get())
+ chunk_demuxer_->EndOfStream(status);
}
void WebMediaPlayerProxy::DemuxerShutdown() {
« no previous file with comments | « webkit/media/webmediaplayer_proxy.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698