Add external surface rendering mode

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/android/jni_android.h"
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "content/browser/android/media_player_manager_android.h"
 #include "content/common/android/surface_callback.h"
 #include "content/common/android/surface_texture_peer.h"
 #include "content/public/browser/browser_thread.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 {
 
 bool ContentVideoView::RegisterContentVideoView(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 ContentVideoView::ContentVideoView(MediaPlayerManagerAndroid* manager)
-    : manager_(manager) {
+    : manager_(manager), personality_(PERSONALITY_FULL_SCREEN) {
 }
 
 ContentVideoView::~ContentVideoView() {
   DestroyContentVideoView();
 }
 
-void ContentVideoView::CreateContentVideoView() {
+void ContentVideoView::CreateContentVideoView(int personality) {
+  if (!j_content_video_view_.is_null() && personality_ != personality) {

[qinmin, 2012/12/17 19:34:06]

is this going to support both playing an embedded and a fullscreen video at the
same time? Since we only have one contentVideoView per render View, I don't
think this is going to happen.

[wonsik2, 2013/02/14 14:56:38]

Do you think having a separate class/SurfaceView for embedded video makes more
sense? Please let me know. Thanks!

+    DestroyContentVideoView();
+  }
+  personality_ = personality;
   if (j_content_video_view_.is_null()) {
     JNIEnv* env = AttachCurrentThread();
     j_content_video_view_.Reset(Java_ContentVideoView_createContentVideoView(
-        env, reinterpret_cast<jint>(this)));
+        env, reinterpret_cast<jint>(this), static_cast<jint>(personality)));
   } else {
     // Just ask video view to reopen the video.
     Java_ContentVideoView_openVideo(AttachCurrentThread(),
                                     j_content_video_view_.obj());
   }
 }
 
 void ContentVideoView::DestroyContentVideoView() {
   if (!j_content_video_view_.is_null()) {
     Java_ContentVideoView_destroyContentVideoView(AttachCurrentThread());
@@ skipping 26 matching lines @@
   }
 }
 
 void ContentVideoView::OnPlaybackComplete() {
   if (!j_content_video_view_.is_null()) {
     Java_ContentVideoView_onPlaybackComplete(AttachCurrentThread(),
                                              j_content_video_view_.obj());
   }
 }
 
+int ContentVideoView::getPersonality() {
+  return personality_;
+}
+
 void ContentVideoView::UpdateMediaMetadata() {
   if (!j_content_video_view_.is_null())
     UpdateMediaMetadata(AttachCurrentThread(), j_content_video_view_.obj());
 }
 
 int ContentVideoView::GetVideoWidth(JNIEnv*, jobject obj) const {
   media::MediaPlayerBridge* player = manager_->GetFullscreenPlayer();
   return player ? player->GetVideoWidth() : 0;
 }
 
@@ skipping 43 matching lines @@
 void ContentVideoView::UpdateMediaMetadata(JNIEnv* env, jobject obj) {
   media::MediaPlayerBridge* player = manager_->GetFullscreenPlayer();
   if (player && player->prepared())
     Java_ContentVideoView_updateMediaMetadata(
         env, obj, player->GetVideoWidth(), player->GetVideoHeight(),
         player->GetDuration().InMilliseconds(), player->can_pause(),
         player->can_seek_forward(), player->can_seek_backward());
 }
 
 } // namespace content