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

Unified Diff: content/renderer/media/webmediaplayer_impl.h

Issue 206103004: Remove HasAudio(), HasVideo(), GetInitialNaturalSize() from media::Pipeline. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 9 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: content/renderer/media/webmediaplayer_impl.h
diff --git a/content/renderer/media/webmediaplayer_impl.h b/content/renderer/media/webmediaplayer_impl.h
index 7f765a82c036193401f1bbcf589d95804527cb24..b7a951899be58776b28106b1797eb86b9d6d82d1 100644
--- a/content/renderer/media/webmediaplayer_impl.h
+++ b/content/renderer/media/webmediaplayer_impl.h
@@ -174,6 +174,7 @@ class WebMediaPlayerImpl
void OnPipelineSeek(media::PipelineStatus status);
void OnPipelineEnded();
void OnPipelineError(media::PipelineStatus error);
+ void OnPipelineHasTrack(media::Pipeline::TrackType track);
void OnPipelineBufferingState(
media::Pipeline::BufferingState buffering_state);
void OnDemuxerOpened();
@@ -269,6 +270,10 @@ class WebMediaPlayerImpl
// The LoadType passed in the |load_type| parameter of the load() call.
LoadType load_type_;
+ // Cache of available tracks for answering hasAudio() and hasVideo().
+ bool has_audio_;
+ bool has_video_;
+
// Playback state.
//
// TODO(scherkus): we have these because Pipeline favours the simplicity of a
« no previous file with comments | « no previous file | content/renderer/media/webmediaplayer_impl.cc » ('j') | content/renderer/media/webmediaplayer_impl.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698