Add generated and hand-written passthrough command handlers with stub Doers.

gpu/command_buffer/common/buffer.cc

 // Copyright 2014 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 "gpu/command_buffer/common/buffer.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include "base/format_macros.h"
@@ skipping 25 matching lines @@
 Buffer::~Buffer() {}
 
 void* Buffer::GetDataAddress(uint32_t data_offset, uint32_t data_size) const {
   base::CheckedNumeric<uint32_t> end = data_offset;
   end += data_size;
   if (!end.IsValid() || end.ValueOrDie() > static_cast<uint32_t>(size_))
     return NULL;
   return static_cast<uint8_t*>(memory_) + data_offset;
 }
 
+void* Buffer::GetDataAddressAndSize(uint32_t data_offset,
+                                    uint32_t* data_size) const {
+  if (data_offset > static_cast<uint32_t>(size_))
+    return NULL;
+  *data_size = GetRemainingSize(data_offset);
+  return static_cast<uint8_t*>(memory_) + data_offset;
+}
+
+uint32_t Buffer::GetRemainingSize(uint32_t data_offset) const {
+  base::CheckedNumeric<uint32_t> offset = data_offset;
+  if (!offset.IsValid() || offset.ValueOrDie() > static_cast<uint32_t>(size_))

[piman, 2016/05/25 23:34:42]

Not sure of the goal of the offset.IsValid() check here. data_offset is a
uint32_t, so moving into a CheckedNumeric<uint32_t> will always be valid.

[Geoff Lang, 2016/05/27 14:31:28]

Whoops, yes, forgot to remove the CheckedNumerics from both functions.

+    return 0;
+  return static_cast<uint32_t>(size_) - data_offset;
+}
+
 base::trace_event::MemoryAllocatorDumpGuid GetBufferGUIDForTracing(
     uint64_t tracing_process_id,
     int32_t buffer_id) {
   return base::trace_event::MemoryAllocatorDumpGuid(base::StringPrintf(
       "gpu-buffer-x-process/%" PRIx64 "/%d", tracing_process_id, buffer_id));
 }
 
 } // namespace gpu