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

Unified Diff: content/renderer/render_view_impl.cc

Issue 10413015: Upstream implementation for embedded video for WebMediaPlayerAndroid (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: use first party url for cookies Created 8 years, 7 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/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index 8b2b4a54defc52440597e4852fa166245833d136..de450452299e847e1f025095f58f572ad558d2f7 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -183,6 +183,7 @@
#if defined(OS_ANDROID)
#include "webkit/media/android/webmediaplayer_android.h"
+#include "webkit/media/android/webmediaplayer_manager_android.h"
#elif defined(OS_WIN)
// TODO(port): these files are currently Windows only because they concern:
// * theming
@@ -565,6 +566,11 @@ RenderViewImpl::RenderViewImpl(
new TextInputClientObserver(this);
#endif // defined(OS_MACOSX)
+#if defined(OS_ANDROID)
+ media_player_manager_.reset(
+ new webkit_media::WebMediaPlayerManagerAndroid());
+#endif
+
// The next group of objects all implement RenderViewObserver, so are deleted
// along with the RenderView automatically.
devtools_agent_ = new DevToolsAgent(this);
@@ -2207,8 +2213,10 @@ WebMediaPlayer* RenderViewImpl::createMediaPlayer(
RenderViewObserver, observers_, WillCreateMediaPlayer(frame, client));
#if defined(OS_ANDROID)
+ // TODO(qinmin): upstream the implementation of StreamTextureFactoryImpl
+ // to replace the NULL param here.
return new webkit_media::WebMediaPlayerAndroid(
- client, cookieJar(frame));
+ frame, client, cookieJar(frame), media_player_manager_.get(), NULL);
#endif
media::MessageLoopFactory* message_loop_factory =
@@ -4897,6 +4905,11 @@ void RenderViewImpl::DidHandleTouchEvent(const WebTouchEvent& event) {
void RenderViewImpl::OnWasHidden() {
RenderWidget::OnWasHidden();
+#if defined(OS_ANDROID)
+ // Inform WebMediaPlayerManagerAndroid to release all media player resources.
+ media_player_manager_->ReleaseMediaResources();
+#endif
+
if (webview()) {
webview()->settings()->setMinimumTimerInterval(
webkit_glue::kBackgroundTabTimerInterval);

Powered by Google App Engine
This is Rietveld 408576698