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

Unified Diff: content/browser/renderer_host/image_transport_factory_android.cc

Issue 11475017: Revert 171569 as it broke some browser_tests on win_aura. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years 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/renderer_host/image_transport_factory_android.cc
===================================================================
--- content/browser/renderer_host/image_transport_factory_android.cc (revision 171661)
+++ content/browser/renderer_host/image_transport_factory_android.cc (working copy)
@@ -11,7 +11,6 @@
#include "content/common/gpu/client/gl_helper.h"
#include "content/common/gpu/client/webgraphicscontext3d_command_buffer_impl.h"
#include "third_party/WebKit/Source/Platform/chromium/public/WebGraphicsContext3D.h"
-#include "third_party/khronos/GLES2/gl2.h"
#include "webkit/gpu/webgraphicscontext3d_in_process_impl.h"
namespace content {
@@ -31,16 +30,6 @@
virtual void DestroySharedSurfaceHandle(
const gfx::GLSurfaceHandle& handle) OVERRIDE {}
virtual uint32_t InsertSyncPoint() OVERRIDE { return 0; }
- virtual uint32_t CreateTexture() OVERRIDE {
- return context_->createTexture();
- }
- virtual void DeleteTexture(uint32_t id) OVERRIDE {
- context_->deleteTexture(id);
- }
- virtual void AcquireTexture(
- uint32 texture_id, const signed char* mailbox_name) OVERRIDE {}
- virtual void ReleaseTexture(
- uint32 texture_id, const signed char* mailbox_name) OVERRIDE {}
virtual WebKit::WebGraphicsContext3D* GetContext3D() OVERRIDE {
return context_.get();
}
@@ -75,12 +64,6 @@
virtual void DestroySharedSurfaceHandle(
const gfx::GLSurfaceHandle& handle) OVERRIDE;
virtual uint32_t InsertSyncPoint() OVERRIDE;
- virtual uint32_t CreateTexture() OVERRIDE;
- virtual void DeleteTexture(uint32_t id) OVERRIDE;
- virtual void AcquireTexture(
- uint32 texture_id, const signed char* mailbox_name) OVERRIDE;
- virtual void ReleaseTexture(
- uint32 texture_id, const signed char* mailbox_name) OVERRIDE;
virtual WebKit::WebGraphicsContext3D* GetContext3D() OVERRIDE {
return context_.get();
}
@@ -122,6 +105,11 @@
gfx::GLSurfaceHandle handle = gfx::GLSurfaceHandle(
gfx::kNullPluginWindow, true);
handle.parent_gpu_process_id = context_->GetGPUProcessID();
+ handle.parent_client_id = context_->GetChannelID();
+ handle.parent_context_id = context_->GetContextID();
+ handle.parent_texture_id[0] = context_->createTexture();
+ handle.parent_texture_id[1] = context_->createTexture();
+ handle.sync_point = context_->insertSyncPoint();
context_->flush();
return handle;
}
@@ -132,33 +120,16 @@
NOTREACHED() << "Failed to make shared graphics context current";
return;
}
+
+ context_->deleteTexture(handle.parent_texture_id[0]);
+ context_->deleteTexture(handle.parent_texture_id[1]);
+ context_->finish();
}
uint32_t CmdBufferImageTransportFactory::InsertSyncPoint() {
return context_->insertSyncPoint();
}
-uint32_t CmdBufferImageTransportFactory::CreateTexture() {
- return context_->createTexture();
-}
-
-void CmdBufferImageTransportFactory::DeleteTexture(uint32_t id) {
- context_->deleteTexture(id);
-}
-
-void CmdBufferImageTransportFactory::AcquireTexture(
- uint32 texture_id, const signed char* mailbox_name) {
- context_->bindTexture(GL_TEXTURE_2D, texture_id);
- context_->consumeTextureCHROMIUM(GL_TEXTURE_2D, mailbox_name);
- context_->flush();
-}
-
-void CmdBufferImageTransportFactory::ReleaseTexture(
- uint32 texture_id, const signed char* mailbox_name) {
- context_->bindTexture(GL_TEXTURE_2D, texture_id);
- context_->produceTextureCHROMIUM(GL_TEXTURE_2D, mailbox_name);
-}
-
GLHelper* CmdBufferImageTransportFactory::GetGLHelper() {
if (!gl_helper_.get())
gl_helper_.reset(new GLHelper(GetContext3D(), NULL));

Powered by Google App Engine
This is Rietveld 408576698