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

Unified Diff: content/common/media/media_player_messages_android.h

Issue 302453012: Support casting for embedded YT videos (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Merged parameters into struct to avoid 2 IPC calls, updated nits Created 6 years, 6 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/common/media/media_player_messages_android.h
diff --git a/content/common/media/media_player_messages_android.h b/content/common/media/media_player_messages_android.h
index 12fdcf19cfc8973c7adce866dcc460e9837b89c7..6182b003cef3ec4a3ca95b3aaddbbcbf45425d85 100644
--- a/content/common/media/media_player_messages_android.h
+++ b/content/common/media/media_player_messages_android.h
@@ -62,6 +62,16 @@ IPC_STRUCT_TRAITS_END()
IPC_ENUM_TRAITS(MediaPlayerHostMsg_Initialize_Type)
+// Parameters to describe a media player
+IPC_STRUCT_BEGIN(MediaPlayerHostMsg_Initialize_Params)
+ IPC_STRUCT_MEMBER(base::SharedMemoryHandle, metafile_data_handle)
+ IPC_STRUCT_MEMBER(int, player_id)
+ IPC_STRUCT_MEMBER(int, demuxer_client_id)
+ IPC_STRUCT_MEMBER(GURL, url)
+ IPC_STRUCT_MEMBER(GURL, first_party_for_cookies)
+ IPC_STRUCT_MEMBER(GURL, frame_url)
+IPC_STRUCT_END()
+
// Chrome for Android seek message sequence is:
// 1. Renderer->Browser MediaPlayerHostMsg_Seek
// This is the beginning of actual seek flow in response to web app requests
@@ -207,13 +217,16 @@ IPC_MESSAGE_ROUTED0(MediaPlayerHostMsg_DestroyAllMediaPlayers)
//
// demuxer_client_id: the demuxer associated with this player when initializing
// a media source player.
-IPC_MESSAGE_ROUTED5(
+IPC_MESSAGE_ROUTED2(
MediaPlayerHostMsg_Initialize,
MediaPlayerHostMsg_Initialize_Type /* type */,
+ MediaPlayerHostMsg_Initialize_Params);
+
+// Sets the URL of the frame containing the video with the given player_id
+IPC_MESSAGE_ROUTED2(
acolwell GONE FROM CHROMIUM 2014/06/09 20:02:09 Is this still needed?
May 2014/06/10 18:22:19 Removed. Good catch.
+ MediaPlayerHostMsg_SetFrameUrl,
int /* player_id */,
- GURL /* url */,
- GURL /* first_party_for_cookies */,
- int /* demuxer_client_id */)
+ GURL /* frame_source_url */);
// Pause the player.
IPC_MESSAGE_ROUTED2(MediaPlayerHostMsg_Pause,

Powered by Google App Engine
This is Rietveld 408576698