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

Unified Diff: webkit/support/webkit_support.cc

Issue 10919075: Move android mediaplayer from render process to browser process. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: addressing comments Created 8 years, 3 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: 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));
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());
« media/base/android/cookies_retriever.h ('K') | « webkit/support/platform_support_android.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698