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

Unified Diff: content/browser/media/media_web_contents_observer.h

Issue 1570043002: Implement MediaSession on top of the WebMediaPlayerDelegate. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@media_session
Patch Set: Reorder. Created 4 years, 11 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/browser/media/media_web_contents_observer.h
diff --git a/content/browser/media/media_web_contents_observer.h b/content/browser/media/media_web_contents_observer.h
index 4429d930203a646fa8c2ea515c58e875a6e33e31..d902dc9608d40b80ac856590de24396884c16d09 100644
--- a/content/browser/media/media_web_contents_observer.h
+++ b/content/browser/media/media_web_contents_observer.h
@@ -48,13 +48,16 @@ class CONTENT_EXPORT MediaWebContentsObserver : public WebContentsObserver {
}
private:
- void OnMediaPlayingNotification(RenderFrameHost* render_frame_host,
- int64_t player_cookie,
- bool has_video,
- bool has_audio,
- bool is_remote);
- void OnMediaPausedNotification(RenderFrameHost* render_frame_host,
- int64_t player_cookie);
+ void OnMediaDestroyed(RenderFrameHost* render_frame_host, int delegate_id);
+ void OnMediaPaused(RenderFrameHost* render_frame_host,
+ int delegate_id,
+ bool reached_end_of_stream);
+ void OnMediaPlaying(RenderFrameHost* render_frame_host,
+ int delegate_id,
+ bool has_video,
+ bool has_audio,
+ bool is_remote,
+ base::TimeDelta duration);
// Clear |render_frame_host|'s tracking entry for its power save blockers.
void ClearPowerSaveBlockers(RenderFrameHost* render_frame_host);
@@ -69,7 +72,7 @@ class CONTENT_EXPORT MediaWebContentsObserver : public WebContentsObserver {
void MaybeReleasePowerSaveBlockers();
// Helper methods for adding or removing player entries in |player_map|.
- using PlayerList = std::vector<int64_t>;
+ using PlayerList = std::vector<int>;
using ActiveMediaPlayerMap = std::map<RenderFrameHost*, PlayerList>;
void AddMediaPlayerEntry(const MediaPlayerId& id,
ActiveMediaPlayerMap* player_map);

Powered by Google App Engine
This is Rietveld 408576698