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

Unified Diff: content/browser/android/sandboxed_process_launcher.cc

Issue 12390077: Android: Support GPU process (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 10 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/browser/android/sandboxed_process_launcher.cc
diff --git a/content/browser/android/sandboxed_process_launcher.cc b/content/browser/android/sandboxed_process_launcher.cc
index 0bdd88add0fd50f74fa18b4f95f66096cfe518a3..a26451414f0fbff667e538321c5a5d5b4a88803c 100644
--- a/content/browser/android/sandboxed_process_launcher.cc
+++ b/content/browser/android/sandboxed_process_launcher.cc
@@ -8,15 +8,58 @@
#include "base/android/jni_array.h"
#include "base/logging.h"
#include "base/memory/scoped_ptr.h"
+#include "content/browser/android/media_player_manager_android.h"
+#include "content/browser/renderer_host/compositor_impl_android.h"
+#include "content/browser/renderer_host/render_view_host_impl.h"
+#include "content/common/android/scoped_java_surface.h"
+#include "content/public/browser/browser_thread.h"
+#include "content/public/browser/render_process_host.h"
#include "jni/SandboxedProcessLauncher_jni.h"
+#include "media/base/android/media_player_bridge.h"
using base::android::AttachCurrentThread;
using base::android::ToJavaArrayOfStrings;
+using base::android::ScopedJavaGlobalRef;
using base::android::ScopedJavaLocalRef;
using content::StartSandboxedProcessCallback;
namespace content {
+namespace {
+
+// Pass a java surface object to the MediaPlayerBridge object
+// identified by render process handle, render view ID and player ID.
+static void SetSurfacePeer(
+ const base::android::JavaRef<jobject>& surface,
+ base::ProcessHandle render_process_handle,
+ int render_view_id,
+ int player_id) {
+ int renderer_id = 0;
+ RenderProcessHost::iterator it = RenderProcessHost::AllHostsIterator();
+ while (!it.IsAtEnd()) {
+ if (it.GetCurrentValue()->GetHandle() == render_process_handle) {
+ renderer_id = it.GetCurrentValue()->GetID();
+ break;
+ }
+ it.Advance();
+ }
+
+ if (renderer_id) {
+ RenderViewHostImpl* host = RenderViewHostImpl::FromID(
+ renderer_id, render_view_id);
+ if (host) {
+ media::MediaPlayerBridge* player =
+ host->media_player_manager()->GetPlayer(player_id);
+ if (player &&
+ player != host->media_player_manager()->GetFullscreenPlayer()) {
+ ScopedJavaSurface scoped_surface(surface);
+ player->SetVideoSurface(scoped_surface.j_surface().obj());
+ }
+ }
+ }
+}
no sievers 2013/03/05 00:32:33 I guess I should really share this code with Surfa
+
+} // anonymous namespace
bulach 2013/03/05 14:23:58 nit: add a \n
no sievers 2013/03/07 00:51:06 Done.
// Called from SandboxedProcessLauncher.java when the SandboxedProcess was
// started.
// |client_context| is the pointer to StartSandboxedProcessCallback which was
@@ -86,6 +129,27 @@ void StopSandboxedProcess(base::ProcessHandle handle) {
Java_SandboxedProcessLauncher_stop(env, static_cast<jint>(handle));
}
+void EstablishSurfacePeer(
+ JNIEnv* env, jclass clazz,
+ jint pid, jobject surface, jint primary_id, jint secondary_id) {
+ ScopedJavaGlobalRef<jobject> jsurface;
+ jsurface.Reset(env, surface);
+ if (jsurface.is_null())
+ return;
+
+ DCHECK(!BrowserThread::CurrentlyOn(BrowserThread::UI));
+ BrowserThread::PostTask(BrowserThread::UI, FROM_HERE, base::Bind(
+ &SetSurfacePeer, jsurface, pid, primary_id, secondary_id));
+}
+
+jobject GetViewSurface(JNIEnv* env, jclass clazz, jint surface_id) {
+ // This is a synchronous call from the GPU process and is expected to be
+ // handled on a binder thread. Handling this on the UI thread will lead
+ // to deadlocks.
+ DCHECK(!BrowserThread::CurrentlyOn(BrowserThread::UI));
+ return CompositorImpl::GetSurface(surface_id);
+}
+
bool RegisterSandboxedProcessLauncher(JNIEnv* env) {
return RegisterNativesImpl(env);
}

Powered by Google App Engine
This is Rietveld 408576698