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

Unified Diff: gpu/command_buffer/service/gles2_cmd_decoder_autogen.h

Issue 14456004: GPU client side changes for GpuMemoryBuffers (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@glapi
Patch Set: Implement DeleteImageBuffersHelper and remove unused GetNativeBufferForGpuMemoryBuffer Created 7 years, 8 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: gpu/command_buffer/service/gles2_cmd_decoder_autogen.h
diff --git a/gpu/command_buffer/service/gles2_cmd_decoder_autogen.h b/gpu/command_buffer/service/gles2_cmd_decoder_autogen.h
index 0bc6350ede29a18ef2c0642ab3f3b3cb14b0f484..f8fa692f43f92919aee48932a21f1016e85f0d0c 100644
--- a/gpu/command_buffer/service/gles2_cmd_decoder_autogen.h
+++ b/gpu/command_buffer/service/gles2_cmd_decoder_autogen.h
@@ -593,6 +593,39 @@ error::Error GLES2DecoderImpl::HandleDeleteFramebuffersImmediate(
return error::kNoError;
}
+error::Error GLES2DecoderImpl::HandleDeleteImageBuffers(
+ uint32 immediate_data_size, const gles2::cmds::DeleteImageBuffers& c) {
+ GLsizei n = static_cast<GLsizei>(c.n);
+ uint32 data_size;
+ if (!SafeMultiplyUint32(n, sizeof(GLuint), &data_size)) {
+ return error::kOutOfBounds;
+ }
+ const GLuint* imagebuffers = GetSharedMemoryAs<const GLuint*>(
+ c.imagebuffers_shm_id, c.imagebuffers_shm_offset, data_size);
+ if (imagebuffers == NULL) {
+ return error::kOutOfBounds;
+ }
+ DeleteImageBuffersHelper(n, imagebuffers);
+ return error::kNoError;
+}
+
+error::Error GLES2DecoderImpl::HandleDeleteImageBuffersImmediate(
+ uint32 immediate_data_size,
+ const gles2::cmds::DeleteImageBuffersImmediate& c) {
+ GLsizei n = static_cast<GLsizei>(c.n);
+ uint32 data_size;
+ if (!SafeMultiplyUint32(n, sizeof(GLuint), &data_size)) {
+ return error::kOutOfBounds;
+ }
+ const GLuint* imagebuffers = GetImmediateDataAs<const GLuint*>(
+ c, data_size, immediate_data_size);
+ if (imagebuffers == NULL) {
+ return error::kOutOfBounds;
+ }
+ DeleteImageBuffersHelper(n, imagebuffers);
+ return error::kNoError;
+}
+
error::Error GLES2DecoderImpl::HandleDeleteRenderbuffers(
uint32 immediate_data_size, const gles2::cmds::DeleteRenderbuffers& c) {
GLsizei n = static_cast<GLsizei>(c.n);
@@ -906,6 +939,43 @@ error::Error GLES2DecoderImpl::HandleGenFramebuffersImmediate(
return error::kNoError;
}
+error::Error GLES2DecoderImpl::HandleGenImageBuffers(
+ uint32 immediate_data_size, const gles2::cmds::GenImageBuffers& c) {
+ GLsizei n = static_cast<GLsizei>(c.n);
+ uint32 data_size;
+ if (!SafeMultiplyUint32(n, sizeof(GLuint), &data_size)) {
+ return error::kOutOfBounds;
+ }
+ GLuint* imagebuffers = GetSharedMemoryAs<GLuint*>(
+ c.imagebuffers_shm_id, c.imagebuffers_shm_offset, data_size);
+ if (imagebuffers == NULL) {
+ return error::kOutOfBounds;
+ }
+ if (!GenImageBuffersHelper(n, imagebuffers)) {
+ return error::kInvalidArguments;
+ }
+ return error::kNoError;
+}
+
+error::Error GLES2DecoderImpl::HandleGenImageBuffersImmediate(
+ uint32 immediate_data_size,
+ const gles2::cmds::GenImageBuffersImmediate& c) {
+ GLsizei n = static_cast<GLsizei>(c.n);
+ uint32 data_size;
+ if (!SafeMultiplyUint32(n, sizeof(GLuint), &data_size)) {
+ return error::kOutOfBounds;
+ }
+ GLuint* imagebuffers = GetImmediateDataAs<GLuint*>(
+ c, data_size, immediate_data_size);
+ if (imagebuffers == NULL) {
+ return error::kOutOfBounds;
+ }
+ if (!GenImageBuffersHelper(n, imagebuffers)) {
+ return error::kInvalidArguments;
+ }
+ return error::kNoError;
+}
+
error::Error GLES2DecoderImpl::HandleGenRenderbuffers(
uint32 immediate_data_size, const gles2::cmds::GenRenderbuffers& c) {
GLsizei n = static_cast<GLsizei>(c.n);

Powered by Google App Engine
This is Rietveld 408576698