Move android mediaplayer from render process to browser process.

webkit/support/webkit_support.cc

Index: webkit/support/webkit_support.cc
diff --git a/webkit/support/webkit_support.cc b/webkit/support/webkit_support.cc
index 6b8c4e3afab0f5d4f411490c0545bd9a46617dec..18bdbeb66b27cb14027ec7d5912928111d5fed17 100644
--- a/webkit/support/webkit_support.cc
+++ b/webkit/support/webkit_support.cc
@@ -58,7 +58,8 @@
 #include "webkit/gpu/webgraphicscontext3d_in_process_command_buffer_impl.h"
 #include "webkit/gpu/webgraphicscontext3d_in_process_impl.h"
 #if defined(OS_ANDROID)
-#include "webkit/media/android/webmediaplayer_android.h"
+#include "webkit/media/android/media_player_bridge_manager_impl.h"
+#include "webkit/media/android/webmediaplayer_in_process_android.h"
 #include "webkit/media/android/webmediaplayer_manager_android.h"
 #endif
 #include "webkit/media/webmediaplayer_impl.h"
@@ -166,6 +167,8 @@ class TestEnvironment {
                                       shadow_platform_delegate));
 
 #if defined(OS_ANDROID)
+    media_bridge_manager_.reset(
+        new webkit_media::MediaPlayerBridgeManagerImpl(8));

[tkent, 2012/09/18 06:09:47]

Why is it 8?

We need a comment or a const symbol to explain 8.

[qinmin, 2012/09/18 16:26:05]

Comment added.
8 is the current maximum number of media elements in a layout tests. 
This is for us to manage the decoding resources. When we have more than 8
decoders allocated, we will start freeing them as the resource is limited on
android.

On 2012/09/18 06:09:47, Kent Tamura wrote:

     media_player_manager_.reset(
         new webkit_media::WebMediaPlayerManagerAndroid());
 #endif
@@ -206,6 +209,10 @@ class TestEnvironment {
   webkit_media::WebMediaPlayerManagerAndroid* media_player_manager() {
     return media_player_manager_.get();
   }
+
+  webkit_media::MediaPlayerBridgeManagerImpl* media_bridge_manager() {
+    return media_bridge_manager_.get();
+  }
 #endif
 
  private:
@@ -218,6 +225,7 @@ class TestEnvironment {
 #if defined(OS_ANDROID)
   FilePath mock_current_directory_;
   scoped_ptr<webkit_media::WebMediaPlayerManagerAndroid> media_player_manager_;
+  scoped_ptr<webkit_media::MediaPlayerBridgeManagerImpl> media_bridge_manager_;
 #endif
 };
 
@@ -387,12 +395,14 @@ WebKit::WebMediaPlayer* CreateMediaPlayer(
     WebMediaPlayerClient* client,
     webkit_media::MediaStreamClient* media_stream_client) {
 #if defined(OS_ANDROID)
-  return new webkit_media::WebMediaPlayerAndroid(
+  return new webkit_media::WebMediaPlayerInProcessAndroid(
       frame,
       client,
       GetWebKitPlatformSupport()->cookieJar(),
       test_environment->media_player_manager(),
-      new webkit_support::TestStreamTextureFactory());
+      test_environment->media_bridge_manager(),
+      new webkit_support::TestStreamTextureFactory(),
+      true);
 #else
   scoped_ptr<media::MessageLoopFactory> message_loop_factory(
       new media::MessageLoopFactory());