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

Unified Diff: content/browser/media/android/media_player_renderer.h

Issue 2471903002: Add MediaServiceThrottler (Closed)
Patch Set: Release resources after 1 min. +2 UTS. Created 4 years, 1 month 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/browser/media/android/media_player_renderer.h
diff --git a/content/browser/media/android/media_player_renderer.h b/content/browser/media/android/media_player_renderer.h
index bd27299df1bd9afe748365769f8f336b04af4e8b..3bfc51e6edde0d1fee789488ba4bcc15ae089418 100644
--- a/content/browser/media/android/media_player_renderer.h
+++ b/content/browser/media/android/media_player_renderer.h
@@ -96,6 +96,8 @@ class CONTENT_EXPORT MediaPlayerRenderer : public media::Renderer,
void OnScopedSurfaceRequestCompleted(gl::ScopedJavaSurface surface);
private:
+ void CreateUnderlyingMediaPlayer(const media::MediaUrlParams& params);
DaleCurtis 2016/11/11 23:32:15 Just CreateMediaPlayer?
tguilbert 2016/11/14 23:18:07 Done.
+
// Used when creating |media_player_|.
void OnDecoderResourcesReleased(int player_id);
@@ -106,6 +108,8 @@ class CONTENT_EXPORT MediaPlayerRenderer : public media::Renderer,
RenderFrameHost* render_frame_host_;
media::RendererClient* renderer_client_;
+ media::PipelineStatusCB pending_init_cb_;
+
std::unique_ptr<media::MediaPlayerBridge> media_player_;
// Current duration of the media.

Powered by Google App Engine
This is Rietveld 408576698