Restart fullscreen video playback when switching back from background

content/browser/android/content_video_view.cc

 // 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.
 
 #include "content/browser/android/content_video_view.h"
 
 #include "base/command_line.h"
 #include "base/logging.h"
+#include "content/browser/android/content_view_core_impl.h"
 #include "content/browser/media/android/browser_media_player_manager.h"
 #include "content/common/android/surface_texture_peer.h"
 #include "content/public/common/content_switches.h"
 #include "jni/ContentVideoView_jni.h"
 
 using base::android::AttachCurrentThread;
 using base::android::CheckException;
 using base::android::ScopedJavaGlobalRef;
 
 namespace content {
 
 namespace {
 // There can only be one content video view at a time, this holds onto that
 // singleton instance.
 ContentVideoView* g_content_video_view = NULL;
 
 }  // namespace
 
 static jobject GetSingletonJavaContentVideoView(JNIEnv*env, jclass) {
   if (g_content_video_view)
     return g_content_video_view->GetJavaObject(env).Release();
   else
     return NULL;
 }
 
 bool ContentVideoView::RegisterContentVideoView(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
-bool ContentVideoView::HasContentVideoView() {
+ContentVideoView* ContentVideoView::GetInstance() {
   return g_content_video_view;
 }
 
 ContentVideoView::ContentVideoView(
-    const ScopedJavaLocalRef<jobject>& context,
-    const ScopedJavaLocalRef<jobject>& client,
     BrowserMediaPlayerManager* manager)
-    : manager_(manager) {
+    : manager_(manager),
+      fullscreen_state_(ENTERED) {
   DCHECK(!g_content_video_view);
-  JNIEnv *env = AttachCurrentThread();
-  j_content_video_view_ = JavaObjectWeakGlobalRef(env,
-      Java_ContentVideoView_createContentVideoView(env, context.obj(),
-          reinterpret_cast<intptr_t>(this), client.obj()).obj());
+  j_content_video_view_ = CreateJavaObject();
   g_content_video_view = this;
 }
 
 ContentVideoView::~ContentVideoView() {
   DCHECK(g_content_video_view);
   DestroyContentVideoView(true);
   g_content_video_view = NULL;
 }
 
 void ContentVideoView::OpenVideo() {
@@ skipping 79 matching lines @@
 
 void ContentVideoView::Play(JNIEnv*, jobject obj) {
   manager_->FullscreenPlayerPlay();
 }
 
 void ContentVideoView::Pause(JNIEnv*, jobject obj) {
   manager_->FullscreenPlayerPause();
 }
 
 void ContentVideoView::ExitFullscreen(
-    JNIEnv*, jobject, jboolean release_media_player) {
+    JNIEnv* env, jobject, jboolean release_media_player) {
+  if (fullscreen_state_ == SUSPENDED)
+    return;
   j_content_video_view_.reset();
   manager_->ExitFullscreen(release_media_player);
 }
 
+void ContentVideoView::SuspendFullscreen() {
+  if (fullscreen_state_ != ENTERED)
+    return;
+  fullscreen_state_ = SUSPENDED;
+  DestroyContentVideoView(false);
+  manager_->SuspendFullscreen();
+}
+
+void ContentVideoView::ResumeFullscreenIfSuspended() {
+  if (fullscreen_state_ != SUSPENDED)
+    return;
+  JNIEnv* env = AttachCurrentThread();
+  DCHECK(!GetJavaObject(env).obj());
+  fullscreen_state_ = RESUME;
+  j_content_video_view_ = CreateJavaObject();
+}
+
 void ContentVideoView::SetSurface(JNIEnv* env, jobject obj,
                                   jobject surface) {
-  manager_->SetVideoSurface(
-      gfx::ScopedJavaSurface::AcquireExternalSurface(surface));
+  gfx::ScopedJavaSurface scoped_surface =
+      gfx::ScopedJavaSurface::AcquireExternalSurface(surface);
+  if (fullscreen_state_ == RESUME) {
+    DCHECK(surface);
+    manager_->ResumeFullscreen(scoped_surface.Pass());
+    fullscreen_state_ = ENTERED;
+  } else {
+    manager_->SetVideoSurface(scoped_surface.Pass());
+  }
 }
 
 void ContentVideoView::UpdateMediaMetadata(JNIEnv* env, jobject obj) {
   media::MediaPlayerAndroid* player = manager_->GetFullscreenPlayer();
   if (player && player->IsPlayerReady())
     Java_ContentVideoView_onUpdateMediaMetadata(
         env, obj, player->GetVideoWidth(), player->GetVideoHeight(),
         player->GetDuration().InMilliseconds(), player->CanPause(),
         player->CanSeekForward(), player->CanSeekBackward());
 }
 
 ScopedJavaLocalRef<jobject> ContentVideoView::GetJavaObject(JNIEnv* env) {
   return j_content_video_view_.get(env);
 }
 
+JavaObjectWeakGlobalRef ContentVideoView::CreateJavaObject() {
+  ContentViewCoreImpl* content_view_core = manager_->GetContentViewCore();
+  JNIEnv *env = AttachCurrentThread();
+  return JavaObjectWeakGlobalRef(
+             env,

[bulach, 2014/01/06 15:44:03]

nit: indent should be just +4 from the beginning of the line.

+             Java_ContentVideoView_createContentVideoView(
+                 env,
+                 content_view_core->GetContext().obj(),
+                 reinterpret_cast<intptr_t>(this),
+                 content_view_core->GetContentVideoViewClient().obj()).obj());
+}
+
 void ContentVideoView::DestroyContentVideoView(bool native_view_destroyed) {
   JNIEnv *env = AttachCurrentThread();
   ScopedJavaLocalRef<jobject> content_video_view = GetJavaObject(env);
   if (!content_video_view.is_null()) {
     Java_ContentVideoView_destroyContentVideoView(env,
         content_video_view.obj(), native_view_destroyed);
     j_content_video_view_.reset();
   }
 }
 }  // namespace content