Move android mediaplayer from render process to browser process.

content/common/view_messages.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // IPC messages for page rendering.
 // Multiply-included message file, hence no include guard.
 
 #include "base/process.h"
 #include "base/shared_memory.h"
 #include "content/common/content_export.h"
@@ skipping 2284 matching lines @@
 // * id - (integer) the frame identifier in this RenderView
 // * name - (string) the name of the frame, if one has been assigned
 // * subtree - an array of the same type of objects for each frame that is a
 //     direct child of the current frame. This property can be omitted if
 //     there are no direct child frames, so less data is transferred.
 //
 // This message must be sent on any events that modify the tree structure or
 // 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