Move android mediaplayer from render process to browser process.

content/common/view_messages.h

Index: content/common/view_messages.h
diff --git a/content/common/view_messages.h b/content/common/view_messages.h
index 9b8ade47245caf20bff70631d516f1fe544e6346..d20ddcbf32a2ea6240b3e454fddd3669975fdf19 100644
--- a/content/common/view_messages.h
+++ b/content/common/view_messages.h
@@ -2302,3 +2302,71 @@ IPC_MESSAGE_ROUTED1(ViewMsg_OrientationChangeEvent,
 // the names of any frames.
 IPC_MESSAGE_ROUTED1(ViewHostMsg_FrameTreeUpdated,
                     std::string /* json encoded frame tree */)
+
+#if defined(OS_ANDROID)
+// Mediaplayer related messages ----------------------------------------------

[scherkus (not reviewing), 2012/09/07 13:17:35]

nit: "mediaplayer" is not a word and you should elaborate what these are used
for

for example, mentioning that this is for controlling media element playback in
the browser process

[qinmin, 2012/09/07 22:48:27]

Done. Split the description into 2, one for controlling the media playback, one
for notifying the render process of status change.

On 2012/09/07 13:17:35, scherkus wrote:

+
+// Media buffering has updated.
+IPC_MESSAGE_ROUTED2(ViewMsg_MediaBufferingUpdate,
+                    int /* player_id */,
+                    int /* percent */)
+
+// A media playback error has occured.
+IPC_MESSAGE_ROUTED2(ViewMsg_MediaError,
+                    int /* player_id */,
+                    int /* error */)
+
+// Playback is completed.
+IPC_MESSAGE_ROUTED1(ViewMsg_MediaPlaybackCompleted,
+                    int /* player_id */)
+
+// Player is prepared.
+IPC_MESSAGE_ROUTED2(ViewMsg_MediaPrepared,
+                    int /* player_id */,
+                    base::TimeDelta /* duration */)
+
+// Media seek is completed.
+IPC_MESSAGE_ROUTED2(ViewMsg_MediaSeekCompleted,
+                    int /* player_id */,
+                    base::TimeDelta /* current_time */)
+
+// Video size has changed.
+IPC_MESSAGE_ROUTED3(ViewMsg_MediaVideoSizeChanged,
+                    int /* player_id */,
+                    int /* width */,
+                    int /* height */)
+
+// The current play time has updated.
+IPC_MESSAGE_ROUTED2(ViewMsg_MediaTimeUpdate,
+                    int /* player_id */,
+                    base::TimeDelta /* current_time */)
+
+// Destroy the media player object.
+IPC_MESSAGE_ROUTED1(ViewHostMsg_DestroyMediaPlayer,
+                    int /* player_id */)
+
+// Clean up all the players.
+IPC_MESSAGE_ROUTED0(ViewHostMsg_CleanUpAllPlayers)

[scherkus (not reviewing), 2012/09/07 13:17:35]

You have DestroyMediaPlayer but CleanUpAllPlayers?

How about DestroyAllMediaPlayers?

[qinmin, 2012/09/07 22:48:27]

Done.

+
+// Initialize a media player object with the given player_id.
+IPC_MESSAGE_ROUTED2(ViewHostMsg_MediaPlayerInitialize,
+                    int /* player_id */,
+                    std::string /* url */)
+
+// Pause the player.
+IPC_MESSAGE_ROUTED1(ViewHostMsg_MediaPlayerPause,
+                    int /* player_id */)
+
+// Release player resources, but keep the object for future usage.
+IPC_MESSAGE_ROUTED1(ViewHostMsg_MediaPlayerRelease,
+                    int /* player_id */)
+
+// Perform a seek.
+IPC_MESSAGE_ROUTED2(ViewHostMsg_MediaPlayerSeek,
+                    int /* player_id */,
+                    base::TimeDelta /* time */)
+
+// Start the player for playback.
+IPC_MESSAGE_ROUTED1(ViewHostMsg_MediaPlayerStart,
+                    int /* player_id */)
+#endif