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

Unified Diff: content/app/android/child_process_service_impl.cc

Issue 2285593002: Add ScopedSurfaceRequestManager (Closed)
Patch Set: Addressing comments Created 4 years, 4 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/app/android/child_process_service_impl.cc
diff --git a/content/app/android/child_process_service_impl.cc b/content/app/android/child_process_service_impl.cc
index 99ae3f85c76e469ed2af026ac97c22fef3bfa2ce..e48a53dcc63729239d8605bc4670fa20be4e2dcd 100644
--- a/content/app/android/child_process_service_impl.cc
+++ b/content/app/android/child_process_service_impl.cc
@@ -15,6 +15,7 @@
#include "base/posix/global_descriptors.h"
#include "content/child/child_thread_impl.h"
#include "content/public/common/content_descriptors.h"
+#include "gpu/ipc/common/android/scoped_surface_request_conduit.h"
#include "gpu/ipc/common/android/surface_texture_manager.h"
#include "gpu/ipc/common/android/surface_texture_peer.h"
#include "gpu/ipc/common/gpu_surface_lookup.h"
@@ -35,6 +36,7 @@ namespace {
// we're in a renderer or gpu process.
class SurfaceTextureManagerImpl : public gpu::SurfaceTextureManager,
public gpu::SurfaceTexturePeer,
+ public gpu::ScopedSurfaceRequestConduit,
public gpu::GpuSurfaceLookup {
public:
// |service impl| is the instance of
@@ -44,10 +46,12 @@ class SurfaceTextureManagerImpl : public gpu::SurfaceTextureManager,
: service_impl_(service_impl) {
SurfaceTexturePeer::InitInstance(this);
gpu::GpuSurfaceLookup::InitInstance(this);
+ gpu::ScopedSurfaceRequestConduit::SetInstance(this);
}
~SurfaceTextureManagerImpl() override {
SurfaceTexturePeer::InitInstance(NULL);
gpu::GpuSurfaceLookup::InitInstance(NULL);
+ gpu::ScopedSurfaceRequestConduit::SetInstance(NULL);
}
// Overridden from SurfaceTextureManager:
@@ -97,6 +101,17 @@ class SurfaceTextureManagerImpl : public gpu::SurfaceTextureManager,
primary_id, secondary_id);
}
+ // Overriden from ScopedSurfaceRequestConduit:
+ void ForwardSurfaceTextureForSurfaceRequest(
+ uint64_t request_token,
+ gl::SurfaceTexture* surface_texture) override {
+ JNIEnv* env = base::android::AttachCurrentThread();
+ content::
+ Java_ChildProcessServiceImpl_forwardSurfaceTextureForSurfaceRequest(
+ env, service_impl_, request_token,
+ surface_texture->j_surface_texture());
+ }
+
// Overridden from GpuSurfaceLookup:
gfx::AcceleratedWidget AcquireNativeWidget(int surface_id) override {
JNIEnv* env = base::android::AttachCurrentThread();

Powered by Google App Engine
This is Rietveld 408576698