More work in Command Buffers...

command_buffer/service/cross/gapi_decoder.cc

 /*
  * Copyright 2009, Google Inc.
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
@@ skipping 134 matching lines @@
   gapi_->EndFrame();
   return BufferSyncInterface::kParseNoError;
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleClear(
     uint32 arg_count,
     const cmd::Clear& args) {
   // Pull out some values so they can't be changed by another thread after we've
   // validated them.
   uint32 buffers = args.buffers;
-  if (buffers & ~GAPIInterface::ALL_BUFFERS)
+  if (buffers & ~command_buffer::kAllBuffers)
     return BufferSyncInterface::kParseInvalidArguments;
   RGBA rgba;
   rgba.red = args.red;
   rgba.green = args.green;
   rgba.blue = args.blue;
   rgba.alpha = args.alpha;
   gapi_->Clear(buffers, rgba, args.depth, args.stencil);
   return BufferSyncInterface::kParseNoError;
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetViewport(
     uint32 arg_count,
     const cmd::SetViewport& args) {
   gapi_->SetViewport(args.left,
                      args.top,
                      args.width,
                      args.height,
                      args.z_min,
                      args.z_max);
   return BufferSyncInterface::kParseNoError;
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleCreateVertexBuffer(
     uint32 arg_count,
     const cmd::CreateVertexBuffer& args) {
-  return gapi_->CreateVertexBuffer(args.id, args.size, args.flags);
+  return gapi_->CreateVertexBuffer(
+      args.vertex_buffer_id, args.size, args.flags);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleDestroyVertexBuffer(
     uint32 arg_count,
     const cmd::DestroyVertexBuffer& args) {
-  return gapi_->DestroyVertexBuffer(args.id);
+  return gapi_->DestroyVertexBuffer(args.vertex_buffer_id);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetVertexBufferDataImmediate(
     uint32 arg_count,
     const cmd::SetVertexBufferDataImmediate& args) {
-  return gapi_->SetVertexBufferData(args.id, args.offset,
+  return gapi_->SetVertexBufferData(args.vertex_buffer_id, args.offset,
                                     ImmediateDataSize(arg_count, args),
                                     AddressAfterStruct(args));
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetVertexBufferData(
     uint32 arg_count,
     const cmd::SetVertexBufferData& args) {
   // Pull out some values so they can't be changed by another thread after we've
   // validated them.
   uint32 size = args.size;
   void *data = GetAddressAndCheckSize(args.shared_memory.id,
                                       args.shared_memory.offset,
                                       size);
   if (!data) return BufferSyncInterface::kParseInvalidArguments;
-  return gapi_->SetVertexBufferData(args.id, args.offset, size, data);
+  return gapi_->SetVertexBufferData(
+      args.vertex_buffer_id, args.offset, size, data);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleGetVertexBufferData(
     uint32 arg_count,
     const cmd::GetVertexBufferData& args) {
   // Pull out some values so they can't be changed by another thread after we've
   // validated them.
   uint32 size = args.size;
   void *data = GetAddressAndCheckSize(args.shared_memory.id,
                                       args.shared_memory.offset,
                                       size);
   if (!data) return BufferSyncInterface::kParseInvalidArguments;
-  return gapi_->GetVertexBufferData(args.id, args.offset, size, data);
+  return gapi_->GetVertexBufferData(
+      args.vertex_buffer_id, args.offset, size, data);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleCreateIndexBuffer(
     uint32 arg_count,
     const cmd::CreateIndexBuffer& args) {
-  return gapi_->CreateIndexBuffer(args.id, args.size, args.flags);
+  return gapi_->CreateIndexBuffer(args.index_buffer_id, args.size, args.flags);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleDestroyIndexBuffer(
     uint32 arg_count,
     const cmd::DestroyIndexBuffer& args) {
-  return gapi_->DestroyIndexBuffer(args.id);
+  return gapi_->DestroyIndexBuffer(args.index_buffer_id);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetIndexBufferDataImmediate(
     uint32 arg_count,
     const cmd::SetIndexBufferDataImmediate& args) {
-  return gapi_->SetIndexBufferData(args.id, args.offset,
+  return gapi_->SetIndexBufferData(args.index_buffer_id, args.offset,
                                    ImmediateDataSize(arg_count, args),
                                    AddressAfterStruct(args));
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetIndexBufferData(
     uint32 arg_count,
     const cmd::SetIndexBufferData& args) {
   // Pull out some values so they can't be changed by another thread after we've
   // validated them.
   uint32 size = args.size;
   void *data = GetAddressAndCheckSize(args.shared_memory.id,
                                       args.shared_memory.offset,
                                       size);
   if (!data) return BufferSyncInterface::kParseInvalidArguments;
-  return gapi_->SetIndexBufferData(args.id, args.offset, size, data);
+  return gapi_->SetIndexBufferData(
+      args.index_buffer_id, args.offset, size, data);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleGetIndexBufferData(
     uint32 arg_count,
     const cmd::GetIndexBufferData& args) {
   // Pull out some values so they can't be changed by another thread after we've
   // validated them.
   uint32 size = args.size;
   void *data = GetAddressAndCheckSize(args.shared_memory.id,
                                       args.shared_memory.offset,
                                       size);
   if (!data) return BufferSyncInterface::kParseInvalidArguments;
-  return gapi_->GetIndexBufferData(args.id, args.offset, size, data);
+  return gapi_->GetIndexBufferData(
+      args.index_buffer_id, args.offset, size, data);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleCreateVertexStruct(
     uint32 arg_count,
     const cmd::CreateVertexStruct& args) {
-  return gapi_->CreateVertexStruct(args.id, args.input_count);
+  return gapi_->CreateVertexStruct(args.vertex_struct_id, args.input_count);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleDestroyVertexStruct(
     uint32 arg_count,
     const cmd::DestroyVertexStruct& args) {
-  return gapi_->DestroyVertexStruct(args.id);
+  return gapi_->DestroyVertexStruct(args.vertex_struct_id);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetVertexInput(
     uint32 arg_count,
     const cmd::SetVertexInput& args) {
-  namespace cmd = set_vertex_input_cmd;
-  unsigned int type_stride_semantic = args.fixme4;
-  unsigned int semantic_index = cmd::SemanticIndex::Get(type_stride_semantic);
-  unsigned int semantic = cmd::Semantic::Get(type_stride_semantic);
-  unsigned int type = cmd::Type::Get(type_stride_semantic);
-  unsigned int stride = cmd::Stride::Get(type_stride_semantic);
-  if (semantic >= vertex_struct::NUM_SEMANTICS ||
-      type >= vertex_struct::NUM_TYPES || stride == 0)
+  unsigned int type_stride_semantic = args.type_stride_semantic;
+  unsigned int semantic_index =
+      cmd::SetVertexInput::SemanticIndex::Get(type_stride_semantic);
+  unsigned int semantic =
+      cmd::SetVertexInput::Semantic::Get(type_stride_semantic);
+  unsigned int type =
+      cmd::SetVertexInput::Type::Get(type_stride_semantic);
+  unsigned int stride =
+      cmd::SetVertexInput::Stride::Get(type_stride_semantic);
+  if (semantic >= vertex_struct::kNumSemantics ||
+      type >= vertex_struct::kNumTypes || stride == 0)
     return BufferSyncInterface::kParseInvalidArguments;
   return gapi_->SetVertexInput(
       args.vertex_struct_id, args.input_index, args.vertex_buffer_id,
       args.offset, stride,
       static_cast<vertex_struct::Type>(type),
       static_cast<vertex_struct::Semantic>(semantic),
       semantic_index);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetVertexStruct(
     uint32 arg_count,
     const cmd::SetVertexStruct& args) {
-  return gapi_->SetVertexStruct(args.id);
+  return gapi_->SetVertexStruct(args.vertex_struct_id);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleDraw(
     uint32 arg_count,
     const cmd::Draw& args) {
   // Pull out some values so they can't be changed by another thread after we've
   // validated them.
   uint32 primitive_type = args.primitive_type;
-  if (primitive_type >= GAPIInterface::MAX_PRIMITIVE_TYPE)
+  if (primitive_type >= command_buffer::kMaxPrimitiveType)
     return BufferSyncInterface::kParseInvalidArguments;
   return gapi_->Draw(
-      static_cast<GAPIInterface::PrimitiveType>(primitive_type),
+      static_cast<command_buffer::PrimitiveType>(primitive_type),
       args.first, args.count);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleDrawIndexed(
     uint32 arg_count,
     const cmd::DrawIndexed& args) {
   // Pull out some values so they can't be changed by another thread after we've
   // validated them.
   uint32 primitive_type = args.primitive_type;
-  if (primitive_type >= GAPIInterface::MAX_PRIMITIVE_TYPE)
+  if (primitive_type >= command_buffer::kMaxPrimitiveType)
     return BufferSyncInterface::kParseInvalidArguments;
   return gapi_->DrawIndexed(
-      static_cast<GAPIInterface::PrimitiveType>(primitive_type),
+      static_cast<command_buffer::PrimitiveType>(primitive_type),
       args.index_buffer_id,
       args.first, args.count, args.min_index, args.max_index);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleCreateEffect(
     uint32 arg_count,
     const cmd::CreateEffect& args) {
   // Pull out some values so they can't be changed by another thread after we've
   // validated them.
   uint32 size = args.size;
   void *data = GetAddressAndCheckSize(args.shared_memory.id,
                                       args.shared_memory.offset,
                                       size);
   if (!data) return BufferSyncInterface::kParseInvalidArguments;
-  return gapi_->CreateEffect(args.id, size, data);
+  return gapi_->CreateEffect(args.effect_id, size, data);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleCreateEffectImmediate(
     uint32 arg_count,
     const cmd::CreateEffectImmediate& args) {
   // Pull out some values so they can't be changed by another thread after we've
   // validated them.
   uint32 size = args.size;
   uint32 data_size = ImmediateDataSize(arg_count, args);
   if (size > data_size)
     return BufferSyncInterface::kParseInvalidArguments;
-  return gapi_->CreateEffect(args.id, size, AddressAfterStruct(args));
+  return gapi_->CreateEffect(args.effect_id, size, AddressAfterStruct(args));
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleDestroyEffect(
     uint32 arg_count,
     const cmd::DestroyEffect& args) {
-  return gapi_->DestroyEffect(args.id);
+  return gapi_->DestroyEffect(args.effect_id);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetEffect(
     uint32 arg_count,
     const cmd::SetEffect& args) {
-  return gapi_->SetEffect(args.id);
+  return gapi_->SetEffect(args.effect_id);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleGetParamCount(
     uint32 arg_count,
     const cmd::GetParamCount& args) {
   // Pull out some values so they can't be changed by another thread after we've
   // validated them.
   uint32 size = args.size;
   void *data = GetAddressAndCheckSize(args.shared_memory.id,
                                       args.shared_memory.offset,
                                       size);
   if (!data) return BufferSyncInterface::kParseInvalidArguments;
-  return gapi_->GetParamCount(args.id, size, data);
+  return gapi_->GetParamCount(args.effect_id, size, data);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleCreateParam(
     uint32 arg_count,
     const cmd::CreateParam& args) {
   return gapi_->CreateParam(args.param_id, args.effect_id, args.index);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleCreateParamByName(
     uint32 arg_count,
@@ skipping 18 matching lines @@
   uint32 data_size = ImmediateDataSize(arg_count, args);
   if (size > data_size)
     return BufferSyncInterface::kParseInvalidArguments;
   return gapi_->CreateParamByName(args.param_id, args.effect_id, size,
                                   AddressAfterStruct(args));
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleDestroyParam(
     uint32 arg_count,
     const cmd::DestroyParam& args) {
-  return gapi_->DestroyParam(args.id);
+  return gapi_->DestroyParam(args.param_id);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetParamData(
     uint32 arg_count,
     const cmd::SetParamData& args) {
   // Pull out some values so they can't be changed by another thread after we've
   // validated them.
   uint32 size = args.size;
   void *data = GetAddressAndCheckSize(args.shared_memory.id,
                                       args.shared_memory.offset,
                                       size);
   if (!data) return BufferSyncInterface::kParseInvalidArguments;
-  return gapi_->SetParamData(args.id, size, data);
+  return gapi_->SetParamData(args.param_id, size, data);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetParamDataImmediate(
     uint32 arg_count,
     const cmd::SetParamDataImmediate& args) {
   // Pull out some values so they can't be changed by another thread after we've
   // validated them.
   uint32 size = args.size;
   uint32 data_size = ImmediateDataSize(arg_count, args);
   if (size > data_size)
     return BufferSyncInterface::kParseInvalidArguments;
-  return gapi_->SetParamData(args.id, size, AddressAfterStruct(args));
+  return gapi_->SetParamData(args.param_id, size, AddressAfterStruct(args));
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleGetParamDesc(
     uint32 arg_count,
     const cmd::GetParamDesc& args) {
   // Pull out some values so they can't be changed by another thread after we've
   // validated them.
   uint32 size = args.size;
   void *data = GetAddressAndCheckSize(args.shared_memory.id,
                                       args.shared_memory.offset,
                                       size);
   if (!data) return BufferSyncInterface::kParseInvalidArguments;
-  return gapi_->GetParamDesc(args.id, size, data);
+  return gapi_->GetParamDesc(args.param_id, size, data);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleGetStreamCount(
     uint32 arg_count,
     const cmd::GetStreamCount& args) {
   // Pull out some values so they can't be changed by another thread after we've
   // validated them.
   uint32 size = args.size;
   void *data = GetAddressAndCheckSize(args.shared_memory.id,
                                       args.shared_memory.offset,
                                       size);
   if (!data) return BufferSyncInterface::kParseInvalidArguments;
-  return gapi_->GetStreamCount(args.id, size, data);
+  return gapi_->GetStreamCount(args.effect_id, size, data);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleGetStreamDesc(
     uint32 arg_count,
     const cmd::GetStreamDesc& args) {
   // Pull out some values so they can't be changed by another thread after we've
   // validated them.
   uint32 size = args.size;
   void *data = GetAddressAndCheckSize(args.shared_memory.id,
                                       args.shared_memory.offset,
                                       size);
   if (!data) return BufferSyncInterface::kParseInvalidArguments;
-  return gapi_->GetStreamDesc(args.id, args.index, size, data);
+  return gapi_->GetStreamDesc(args.effect_id, args.index, size, data);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleDestroyTexture(
     uint32 arg_count,
     const cmd::DestroyTexture& args) {
-  return gapi_->DestroyTexture(args.id);
+  return gapi_->DestroyTexture(args.texture_id);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleCreateTexture2d(
     uint32 arg_count,
     const cmd::CreateTexture2d& args) {
-  namespace cmd = create_texture_2d_cmd;
-  unsigned int width_height = args.fixme1;
-  unsigned int levels_format_flags = args.fixme2;
-  unsigned int width = cmd::Width::Get(width_height);
-  unsigned int height = cmd::Height::Get(width_height);
-  unsigned int levels = cmd::Levels::Get(levels_format_flags);
-  unsigned int unused = cmd::Unused::Get(levels_format_flags);
-  unsigned int format = cmd::Format::Get(levels_format_flags);
-  unsigned int flags = cmd::Flags::Get(levels_format_flags);
+  unsigned int width_height = args.width_height;
+  unsigned int levels_format_flags = args.levels_format_flags;
+  unsigned int width = cmd::CreateTexture2d::Width::Get(width_height);
+  unsigned int height = cmd::CreateTexture2d::Height::Get(width_height);
+  unsigned int levels = cmd::CreateTexture2d::Levels::Get(levels_format_flags);
+  unsigned int unused = cmd::CreateTexture2d::Unused::Get(levels_format_flags);
+  unsigned int format = cmd::CreateTexture2d::Format::Get(levels_format_flags);
+  unsigned int flags = cmd::CreateTexture2d::Flags::Get(levels_format_flags);
   unsigned int max_levels =
       1 + base::bits::Log2Ceiling(std::max(width, height));
   if ((width == 0) || (height == 0) || (levels > max_levels) ||
-      (unused != 0) || (format >= texture::NUM_FORMATS) || (levels == 0))
+      (unused != 0) || (format >= texture::kNumFormats) || (levels == 0))
     return BufferSyncInterface::kParseInvalidArguments;
   bool enable_render_surfaces = !!flags;
   return gapi_->CreateTexture2D(args.texture_id, width, height, levels,
                                 static_cast<texture::Format>(format), flags,
                                 enable_render_surfaces);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleCreateTexture3d(
     uint32 arg_count,
     const cmd::CreateTexture3d& args) {
-  namespace cmd = create_texture_3d_cmd;
-  unsigned int width_height = args.fixme1;
-  unsigned int depth_unused = args.fixme2;
-  unsigned int levels_format_flags = args.fixme3;
-  unsigned int width = cmd::Width::Get(width_height);
-  unsigned int height = cmd::Height::Get(width_height);
-  unsigned int depth = cmd::Depth::Get(depth_unused);
-  unsigned int unused1 = cmd::Unused1::Get(depth_unused);
-  unsigned int levels = cmd::Levels::Get(levels_format_flags);
-  unsigned int unused2 = cmd::Unused2::Get(levels_format_flags);
-  unsigned int format = cmd::Format::Get(levels_format_flags);
-  unsigned int flags = cmd::Flags::Get(levels_format_flags);
+  unsigned int width_height = args.width_height;
+  unsigned int depth_unused = args.depth_unused;
+  unsigned int levels_format_flags = args.levels_format_flags;
+  unsigned int width = cmd::CreateTexture3d::Width::Get(width_height);
+  unsigned int height = cmd::CreateTexture3d::Height::Get(width_height);
+  unsigned int depth = cmd::CreateTexture3d::Depth::Get(depth_unused);
+  unsigned int unused1 = cmd::CreateTexture3d::Unused1::Get(depth_unused);
+  unsigned int levels = cmd::CreateTexture3d::Levels::Get(levels_format_flags);
+  unsigned int unused2 =
+      cmd::CreateTexture3d::Unused2::Get(levels_format_flags);
+  unsigned int format = cmd::CreateTexture3d::Format::Get(levels_format_flags);
+  unsigned int flags = cmd::CreateTexture3d::Flags::Get(levels_format_flags);
   unsigned int max_levels =
       1 + base::bits::Log2Ceiling(std::max(depth, std::max(width, height)));
   if ((width == 0) || (height == 0) || (depth == 0) ||
       (levels > max_levels) || (unused1 != 0) || (unused2 != 0) ||
-      (format >= texture::NUM_FORMATS) || (levels == 0))
+      (format >= texture::kNumFormats) || (levels == 0))
     return BufferSyncInterface::kParseInvalidArguments;
   bool enable_render_surfaces = !!flags;
   return gapi_->CreateTexture3D(args.texture_id, width, height, depth, levels,
                                 static_cast<texture::Format>(format), flags,
                                 enable_render_surfaces);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleCreateTextureCube(
     uint32 arg_count,
     const cmd::CreateTextureCube& args) {
-  namespace cmd = create_texture_cube_cmd;
   unsigned int side_unused = args.edge_length;
-  unsigned int levels_format_flags = args.fixme2;
-  unsigned int side = cmd::Side::Get(side_unused);
-  unsigned int unused1 = cmd::Unused1::Get(side_unused);
-  unsigned int levels = cmd::Levels::Get(levels_format_flags);
-  unsigned int unused2 = cmd::Unused2::Get(levels_format_flags);
-  unsigned int format = cmd::Format::Get(levels_format_flags);
-  unsigned int flags = cmd::Flags::Get(levels_format_flags);
+  unsigned int levels_format_flags = args.levels_format_flags;
+  unsigned int side = cmd::CreateTextureCube::Side::Get(side_unused);
+  unsigned int unused1 = cmd::CreateTextureCube::Unused1::Get(side_unused);
+  unsigned int levels =
+      cmd::CreateTextureCube::Levels::Get(levels_format_flags);
+  unsigned int unused2 =
+      cmd::CreateTextureCube::Unused2::Get(levels_format_flags);
+  unsigned int format =
+      cmd::CreateTextureCube::Format::Get(levels_format_flags);
+  unsigned int flags = cmd::CreateTextureCube::Flags::Get(levels_format_flags);
   unsigned int max_levels = 1 + base::bits::Log2Ceiling(side);
   if ((side == 0) || (levels > max_levels) || (unused1 != 0) ||
-      (unused2 != 0) || (format >= texture::NUM_FORMATS) || (levels == 0))
+      (unused2 != 0) || (format >= texture::kNumFormats) || (levels == 0))
     return BufferSyncInterface::kParseInvalidArguments;
   bool enable_render_surfaces = !!flags;
   return gapi_->CreateTextureCube(args.texture_id, side, levels,
                                   static_cast<texture::Format>(format),
                                   flags, enable_render_surfaces);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetTextureData(
     uint32 arg_count,
     const cmd::SetTextureData& args) {
-  namespace cmd = set_texture_data_cmd;
-  unsigned int x_y = args.fixme1;
-  unsigned int width_height = args.fixme2;
-  unsigned int z_depth = args.fixme3;
-  unsigned int level_face = args.fixme4;
+  unsigned int x_y = args.x_y;
+  unsigned int width_height = args.width_height;
+  unsigned int z_depth = args.z_depth;
+  unsigned int level_face = args.level_face;
   unsigned int size = args.size;
-  unsigned int x = cmd::X::Get(x_y);
-  unsigned int y = cmd::Y::Get(x_y);
-  unsigned int width = cmd::Width::Get(width_height);
-  unsigned int height = cmd::Height::Get(width_height);
-  unsigned int z = cmd::Z::Get(z_depth);
-  unsigned int depth = cmd::Depth::Get(z_depth);
-  unsigned int level = cmd::Level::Get(level_face);
-  unsigned int face = cmd::Face::Get(level_face);
-  unsigned int unused = cmd::Unused::Get(level_face);
+  unsigned int x = cmd::SetTextureData::X::Get(x_y);
+  unsigned int y = cmd::SetTextureData::Y::Get(x_y);
+  unsigned int width = cmd::SetTextureData::Width::Get(width_height);
+  unsigned int height = cmd::SetTextureData::Height::Get(width_height);
+  unsigned int z = cmd::SetTextureData::Z::Get(z_depth);
+  unsigned int depth = cmd::SetTextureData::Depth::Get(z_depth);
+  unsigned int level = cmd::SetTextureData::Level::Get(level_face);
+  unsigned int face = cmd::SetTextureData::Face::Get(level_face);
+  unsigned int unused = cmd::SetTextureData::Unused::Get(level_face);
   const void *data = GetAddressAndCheckSize(args.shared_memory.id,
                                             args.shared_memory.offset, size);
   if (face >= 6 || unused != 0 || !data)
     return BufferSyncInterface::kParseInvalidArguments;
   return gapi_->SetTextureData(
       args.texture_id, x, y, z, width, height, depth, level,
       static_cast<texture::Face>(face), args.row_pitch,
       args.slice_pitch, size, data);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetTextureDataImmediate(
     uint32 arg_count,
     const cmd::SetTextureDataImmediate& args) {
-  namespace cmd = set_texture_data_immediate_cmd;
-  unsigned int x_y = args.fixme1;
-  unsigned int width_height = args.fixme2;
-  unsigned int z_depth = args.fixme3;
-  unsigned int level_face = args.fixme4;
+  unsigned int x_y = args.x_y;
+  unsigned int width_height = args.width_height;
+  unsigned int z_depth = args.z_depth;
+  unsigned int level_face = args.level_face;
   unsigned int size = args.size;
-  unsigned int x = cmd::X::Get(x_y);
-  unsigned int y = cmd::Y::Get(x_y);
-  unsigned int width = cmd::Width::Get(width_height);
-  unsigned int height = cmd::Height::Get(width_height);
-  unsigned int z = cmd::Z::Get(z_depth);
-  unsigned int depth = cmd::Depth::Get(z_depth);
-  unsigned int level = cmd::Level::Get(level_face);
-  unsigned int face = cmd::Face::Get(level_face);
-  unsigned int unused = cmd::Unused::Get(level_face);
+  unsigned int x = cmd::SetTextureDataImmediate::X::Get(x_y);
+  unsigned int y = cmd::SetTextureDataImmediate::Y::Get(x_y);
+  unsigned int width = cmd::SetTextureDataImmediate::Width::Get(width_height);
+  unsigned int height = cmd::SetTextureDataImmediate::Height::Get(width_height);
+  unsigned int z = cmd::SetTextureDataImmediate::Z::Get(z_depth);
+  unsigned int depth = cmd::SetTextureDataImmediate::Depth::Get(z_depth);
+  unsigned int level = cmd::SetTextureDataImmediate::Level::Get(level_face);
+  unsigned int face = cmd::SetTextureDataImmediate::Face::Get(level_face);
+  unsigned int unused = cmd::SetTextureDataImmediate::Unused::Get(level_face);
   uint32 data_size = ImmediateDataSize(arg_count, args);
   if (face >= 6 || unused != 0 ||
       size > data_size)
     return BufferSyncInterface::kParseInvalidArguments;
   return gapi_->SetTextureData(
       args.texture_id, x, y, z, width, height, depth, level,
       static_cast<texture::Face>(face), args.row_pitch,
       args.slice_pitch, size, AddressAfterStruct(args));
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleGetTextureData(
     uint32 arg_count,
     const cmd::GetTextureData& args) {
-  namespace cmd = get_texture_data_cmd;
-  unsigned int x_y = args.fixme1;
-  unsigned int width_height = args.fixme2;
-  unsigned int z_depth = args.fixme3;
-  unsigned int level_face = args.fixme4;
+  unsigned int x_y = args.x_y;
+  unsigned int width_height = args.width_height;
+  unsigned int z_depth = args.z_depth;
+  unsigned int level_face = args.level_face;
   unsigned int size = args.size;
-  unsigned int x = cmd::X::Get(x_y);
-  unsigned int y = cmd::Y::Get(x_y);
-  unsigned int width = cmd::Width::Get(width_height);
-  unsigned int height = cmd::Height::Get(width_height);
-  unsigned int z = cmd::Z::Get(z_depth);
-  unsigned int depth = cmd::Depth::Get(z_depth);
-  unsigned int level = cmd::Level::Get(level_face);
-  unsigned int face = cmd::Face::Get(level_face);
-  unsigned int unused = cmd::Unused::Get(level_face);
+  unsigned int x = cmd::GetTextureData::X::Get(x_y);
+  unsigned int y = cmd::GetTextureData::Y::Get(x_y);
+  unsigned int width = cmd::GetTextureData::Width::Get(width_height);
+  unsigned int height = cmd::GetTextureData::Height::Get(width_height);
+  unsigned int z = cmd::GetTextureData::Z::Get(z_depth);
+  unsigned int depth = cmd::GetTextureData::Depth::Get(z_depth);
+  unsigned int level = cmd::GetTextureData::Level::Get(level_face);
+  unsigned int face = cmd::GetTextureData::Face::Get(level_face);
+  unsigned int unused = cmd::GetTextureData::Unused::Get(level_face);
   void *data = GetAddressAndCheckSize(args.shared_memory.id,
                                       args.shared_memory.offset, size);
   if (face >= 6 || unused != 0 || !data)
     return BufferSyncInterface::kParseInvalidArguments;
   return gapi_->GetTextureData(
       args.texture_id, x, y, z, width, height, depth, level,
       static_cast<texture::Face>(face), args.row_pitch,
       args.slice_pitch, size, data);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleCreateSampler(
     uint32 arg_count,
     const cmd::CreateSampler& args) {
-  return gapi_->CreateSampler(args.id);
+  return gapi_->CreateSampler(args.sampler_id);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleDestroySampler(
     uint32 arg_count,
     const cmd::DestroySampler& args) {
-  return gapi_->DestroySampler(args.id);
+  return gapi_->DestroySampler(args.sampler_id);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetSamplerStates(
     uint32 arg_count,
     const cmd::SetSamplerStates& args) {
-  namespace cmd = set_sampler_states;
-  Uint32 arg = args.fixme1;
-  if (cmd::Unused::Get(arg) != 0)
+  Uint32 arg = args.sampler_states;
+  if (cmd::SetSamplerStates::Unused::Get(arg) != 0)
     return BufferSyncInterface::kParseInvalidArguments;
-  unsigned int address_u_value = cmd::AddressingU::Get(arg);
-  unsigned int address_v_value = cmd::AddressingV::Get(arg);
-  unsigned int address_w_value = cmd::AddressingW::Get(arg);
-  unsigned int mag_filter_value = cmd::MagFilter::Get(arg);
-  unsigned int min_filter_value = cmd::MinFilter::Get(arg);
-  unsigned int mip_filter_value = cmd::MipFilter::Get(arg);
-  unsigned int max_anisotropy = cmd::MaxAnisotropy::Get(arg);
-  if (address_u_value >= sampler::NUM_ADDRESSING_MODE ||
-      address_v_value >= sampler::NUM_ADDRESSING_MODE ||
-      address_w_value >= sampler::NUM_ADDRESSING_MODE ||
-      mag_filter_value >= sampler::NUM_FILTERING_MODE ||
-      min_filter_value >= sampler::NUM_FILTERING_MODE ||
-      mip_filter_value >= sampler::NUM_FILTERING_MODE ||
-      mag_filter_value == sampler::NONE ||
-      min_filter_value == sampler::NONE ||
+  unsigned int address_u_value = cmd::SetSamplerStates::AddressingU::Get(arg);
+  unsigned int address_v_value = cmd::SetSamplerStates::AddressingV::Get(arg);
+  unsigned int address_w_value = cmd::SetSamplerStates::AddressingW::Get(arg);
+  unsigned int mag_filter_value = cmd::SetSamplerStates::MagFilter::Get(arg);
+  unsigned int min_filter_value = cmd::SetSamplerStates::MinFilter::Get(arg);
+  unsigned int mip_filter_value = cmd::SetSamplerStates::MipFilter::Get(arg);
+  unsigned int max_anisotropy = cmd::SetSamplerStates::MaxAnisotropy::Get(arg);
+  if (address_u_value >= sampler::kNumAddressingMode ||
+      address_v_value >= sampler::kNumAddressingMode ||
+      address_w_value >= sampler::kNumAddressingMode ||
+      mag_filter_value >= sampler::kNumFilteringMode ||
+      min_filter_value >= sampler::kNumFilteringMode ||
+      mip_filter_value >= sampler::kNumFilteringMode ||
+      mag_filter_value == sampler::kNone ||
+      min_filter_value == sampler::kNone ||
       max_anisotropy == 0) {
     return BufferSyncInterface::kParseInvalidArguments;
   }
   gapi_->SetSamplerStates(
-      args.id,
+      args.sampler_id,
       static_cast<sampler::AddressingMode>(address_u_value),
       static_cast<sampler::AddressingMode>(address_v_value),
       static_cast<sampler::AddressingMode>(address_w_value),
       static_cast<sampler::FilteringMode>(mag_filter_value),
       static_cast<sampler::FilteringMode>(min_filter_value),
       static_cast<sampler::FilteringMode>(mip_filter_value),
       max_anisotropy);
   return BufferSyncInterface::kParseNoError;
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetSamplerBorderColor(
     uint32 arg_count,
     const cmd::SetSamplerBorderColor& args) {
   RGBA rgba;
   rgba.red = args.red;
   rgba.green = args.green;
   rgba.blue = args.blue;
   rgba.alpha = args.alpha;
-  return gapi_->SetSamplerBorderColor(args.id, rgba);
+  return gapi_->SetSamplerBorderColor(args.sampler_id, rgba);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetSamplerTexture(
     uint32 arg_count,
     const cmd::SetSamplerTexture& args) {
-  return gapi_->SetSamplerTexture(args.id, args.texture_id);
+  return gapi_->SetSamplerTexture(args.sampler_id, args.texture_id);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetScissor(
     uint32 arg_count,
     const cmd::SetScissor& args) {
-  namespace cmd = set_scissor;
-  Uint32 x_y_enable = args.fixme0;
-  if (cmd::Unused::Get(x_y_enable) != 0)
+  Uint32 x_y_enable = args.x_y_enable;
+  if (cmd::SetScissor::Unused::Get(x_y_enable) != 0)
     return BufferSyncInterface::kParseInvalidArguments;
-  unsigned int x = cmd::X::Get(x_y_enable);
-  unsigned int y = cmd::Y::Get(x_y_enable);
-  bool enable = cmd::Enable::Get(x_y_enable) != 0;
-  Uint32 width_height = args.fixme1;
-  unsigned int width = cmd::Width::Get(width_height);
-  unsigned int height = cmd::Height::Get(width_height);
+  unsigned int x = cmd::SetScissor::X::Get(x_y_enable);
+  unsigned int y = cmd::SetScissor::Y::Get(x_y_enable);
+  bool enable = cmd::SetScissor::Enable::Get(x_y_enable) != 0;
+  Uint32 width_height = args.width_height;
+  unsigned int width = cmd::SetScissor::Width::Get(width_height);
+  unsigned int height = cmd::SetScissor::Height::Get(width_height);
   gapi_->SetScissor(enable, x, y, width, height);
   return BufferSyncInterface::kParseNoError;
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetPolygonOffset(
     uint32 arg_count,
     const cmd::SetPolygonOffset& args) {
   gapi_->SetPolygonOffset(args.slope_factor, args.units);
   return BufferSyncInterface::kParseNoError;
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetPointLineRaster(
     uint32 arg_count,
     const cmd::SetPointLineRaster& args) {
-  namespace cmd = set_point_line_raster;
-  Uint32 enables = args.fixme0;
-  if (cmd::Unused::Get(enables) != 0)
+  Uint32 enables = args.enables;
+  if (cmd::SetPointLineRaster::Unused::Get(enables) != 0)
     return BufferSyncInterface::kParseInvalidArguments;
-  bool line_smooth = !!cmd::LineSmoothEnable::Get(enables);
-  bool point_sprite = !!cmd::PointSpriteEnable::Get(enables);
+  bool line_smooth = !!cmd::SetPointLineRaster::LineSmoothEnable::Get(enables);
+  bool point_sprite =
+      !!cmd::SetPointLineRaster::PointSpriteEnable::Get(enables);
   gapi_->SetPointLineRaster(line_smooth, point_sprite, args.point_size);
   return BufferSyncInterface::kParseNoError;
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetPolygonRaster(
     uint32 arg_count,
     const cmd::SetPolygonRaster& args) {
-  namespace cmd = set_polygon_raster;
-  Uint32 fill_cull = args.fixme0;
-  unsigned int fill_value = cmd::FillMode::Get(fill_cull);
-  unsigned int cull_value = cmd::CullMode::Get(fill_cull);
-  if (cmd::Unused::Get(fill_cull) != 0 ||
-      fill_value >= GAPIInterface::NUM_POLYGON_MODE ||
-      cull_value >= GAPIInterface::NUM_FACE_CULL_MODE)
+  Uint32 fill_cull = args.fill_cull;
+  unsigned int fill_value = cmd::SetPolygonRaster::FillMode::Get(fill_cull);
+  unsigned int cull_value = cmd::SetPolygonRaster::CullMode::Get(fill_cull);
+  if (cmd::SetPolygonRaster::Unused::Get(fill_cull) != 0 ||
+      fill_value >= command_buffer::kNumPolygonMode ||
+      cull_value >= command_buffer::kNumFaceCullMode)
     return BufferSyncInterface::kParseInvalidArguments;
   gapi_->SetPolygonRaster(
-      static_cast<GAPIInterface::PolygonMode>(fill_value),
-      static_cast<GAPIInterface::FaceCullMode>(cull_value));
+      static_cast<command_buffer::PolygonMode>(fill_value),
+      static_cast<command_buffer::FaceCullMode>(cull_value));
   return BufferSyncInterface::kParseNoError;
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetAlphaTest(
     uint32 arg_count,
     const cmd::SetAlphaTest& args) {
-  namespace cmd = set_alpha_test;
-  Uint32 func_enable = args.fixme0;
-  if (cmd::Unused::Get(func_enable) != 0)
+  Uint32 func_enable = args.func_enable;
+  if (cmd::SetAlphaTest::Unused::Get(func_enable) != 0)
     return BufferSyncInterface::kParseInvalidArguments;
   // Check that the bitmask get cannot generate values outside of the
   // allowed range.
-  COMPILE_ASSERT(cmd::Func::kMask < GAPIInterface::NUM_COMPARISON,
+  COMPILE_ASSERT(cmd::SetAlphaTest::Func::kMask <
+                 command_buffer::kNumComparison,
                  set_alpha_test_Func_may_produce_invalid_values);
-  GAPIInterface::Comparison comp =
-      static_cast<GAPIInterface::Comparison>(cmd::Func::Get(func_enable));
-  bool enable = cmd::Enable::Get(func_enable) != 0;
+  command_buffer::Comparison comp = static_cast<command_buffer::Comparison>(
+      cmd::SetAlphaTest::Func::Get(func_enable));
+  bool enable = cmd::SetAlphaTest::Enable::Get(func_enable) != 0;
   gapi_->SetAlphaTest(enable, args.value, comp);
   return BufferSyncInterface::kParseNoError;
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetDepthTest(
     uint32 arg_count,
     const cmd::SetDepthTest& args) {
-  namespace cmd = set_depth_test;
-  Uint32 func_enable = args.fixme0;
-  if (cmd::Unused::Get(func_enable) != 0)
+  Uint32 func_enable = args.func_enable;
+  if (cmd::SetDepthTest::Unused::Get(func_enable) != 0)
     return BufferSyncInterface::kParseInvalidArguments;
   // Check that the bitmask get cannot generate values outside of the
   // allowed range.
-  COMPILE_ASSERT(cmd::Func::kMask < GAPIInterface::NUM_COMPARISON,
+  COMPILE_ASSERT(cmd::SetDepthTest::Func::kMask <
+                 command_buffer::kNumComparison,
                  set_alpha_test_Func_may_produce_invalid_values);
-  GAPIInterface::Comparison comp =
-      static_cast<GAPIInterface::Comparison>(cmd::Func::Get(func_enable));
-  bool write_enable = cmd::WriteEnable::Get(func_enable) != 0;
-  bool enable = cmd::Enable::Get(func_enable) != 0;
+  command_buffer::Comparison comp = static_cast<command_buffer::Comparison>(
+      cmd::SetDepthTest::Func::Get(func_enable));
+  bool write_enable = cmd::SetDepthTest::WriteEnable::Get(func_enable) != 0;
+  bool enable = cmd::SetDepthTest::Enable::Get(func_enable) != 0;
   gapi_->SetDepthTest(enable, write_enable, comp);
   return BufferSyncInterface::kParseNoError;
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetStencilTest(
     uint32 arg_count,
     const cmd::SetStencilTest& args) {
-  namespace cmd = set_stencil_test;
-  Uint32 arg0 = args.fixme0;
-  Uint32 arg1 = args.fixme1;
-  if (cmd::Unused0::Get(arg0) != 0 ||
-      cmd::Unused1::Get(arg1) != 0 ||
-      cmd::Unused2::Get(arg1) != 0)
+  Uint32 arg0 = args.stencil_args0;
+  Uint32 arg1 = args.stencil_args1;
+  if (cmd::SetStencilTest::Unused0::Get(arg0) != 0 ||
+      cmd::SetStencilTest::Unused1::Get(arg1) != 0 ||
+      cmd::SetStencilTest::Unused2::Get(arg1) != 0)
     return BufferSyncInterface::kParseInvalidArguments;
-  unsigned int write_mask = cmd::WriteMask::Get(arg0);
-  unsigned int compare_mask = cmd::CompareMask::Get(arg0);
-  unsigned int ref = cmd::ReferenceValue::Get(arg0);
-  bool enable = cmd::Enable::Get(arg0) != 0;
-  bool separate_ccw = cmd::SeparateCCW::Get(arg0) != 0;
+  unsigned int write_mask = cmd::SetStencilTest::WriteMask::Get(arg0);
+  unsigned int compare_mask = cmd::SetStencilTest::CompareMask::Get(arg0);
+  unsigned int ref = cmd::SetStencilTest::ReferenceValue::Get(arg0);
+  bool enable = cmd::SetStencilTest::Enable::Get(arg0) != 0;
+  bool separate_ccw = cmd::SetStencilTest::SeparateCCW::Get(arg0) != 0;
   gapi_->SetStencilTest(enable, separate_ccw, write_mask, compare_mask, ref,
                         arg1);
   return BufferSyncInterface::kParseNoError;
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetColorWrite(
     uint32 arg_count,
     const cmd::SetColorWrite& args) {
-  namespace cmd = set_color_write;
   Uint32 enables = args.flags;
-  if (cmd::Unused::Get(enables) != 0)
+  if (cmd::SetColorWrite::Unused::Get(enables) != 0)
     return BufferSyncInterface::kParseInvalidArguments;
-  bool red = cmd::RedMask::Get(enables) != 0;
-  bool green = cmd::GreenMask::Get(enables) != 0;
-  bool blue = cmd::BlueMask::Get(enables) != 0;
-  bool alpha = cmd::AlphaMask::Get(enables) != 0;
-  bool dither = cmd::DitherEnable::Get(enables) != 0;
+  bool red = cmd::SetColorWrite::RedMask::Get(enables) != 0;
+  bool green = cmd::SetColorWrite::GreenMask::Get(enables) != 0;
+  bool blue = cmd::SetColorWrite::BlueMask::Get(enables) != 0;
+  bool alpha = cmd::SetColorWrite::AlphaMask::Get(enables) != 0;
+  bool dither = cmd::SetColorWrite::DitherEnable::Get(enables) != 0;
   gapi_->SetColorWrite(red, green, blue, alpha, dither);
   return BufferSyncInterface::kParseNoError;
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetBlending(
     uint32 arg_count,
     const cmd::SetBlending& args) {
-  namespace cmd = set_blending;
-  Uint32 arg = args.fixme0;
-  bool enable = cmd::Enable::Get(arg) != 0;
-  bool separate_alpha = cmd::SeparateAlpha::Get(arg) != 0;
-  unsigned int color_eq = cmd::ColorEq::Get(arg);
-  unsigned int color_src = cmd::ColorSrcFunc::Get(arg);
-  unsigned int color_dst = cmd::ColorDstFunc::Get(arg);
-  unsigned int alpha_eq = cmd::AlphaEq::Get(arg);
-  unsigned int alpha_src = cmd::AlphaSrcFunc::Get(arg);
-  unsigned int alpha_dst = cmd::AlphaDstFunc::Get(arg);
-  if (cmd::Unused0::Get(arg) != 0 ||
-      cmd::Unused1::Get(arg) != 0 ||
-      color_eq >= GAPIInterface::NUM_BLEND_EQ ||
-      color_src >= GAPIInterface::NUM_BLEND_FUNC ||
-      color_dst >= GAPIInterface::NUM_BLEND_FUNC ||
-      alpha_eq >= GAPIInterface::NUM_BLEND_EQ ||
-      alpha_src >= GAPIInterface::NUM_BLEND_FUNC ||
-      alpha_dst >= GAPIInterface::NUM_BLEND_FUNC)
+  Uint32 arg = args.blend_settings;
+  bool enable = cmd::SetBlending::Enable::Get(arg) != 0;
+  bool separate_alpha = cmd::SetBlending::SeparateAlpha::Get(arg) != 0;
+  unsigned int color_eq = cmd::SetBlending::ColorEq::Get(arg);
+  unsigned int color_src = cmd::SetBlending::ColorSrcFunc::Get(arg);
+  unsigned int color_dst = cmd::SetBlending::ColorDstFunc::Get(arg);
+  unsigned int alpha_eq = cmd::SetBlending::AlphaEq::Get(arg);
+  unsigned int alpha_src = cmd::SetBlending::AlphaSrcFunc::Get(arg);
+  unsigned int alpha_dst = cmd::SetBlending::AlphaDstFunc::Get(arg);
+  if (cmd::SetBlending::Unused0::Get(arg) != 0 ||
+      cmd::SetBlending::Unused1::Get(arg) != 0 ||
+      color_eq >= command_buffer::kNumBlendEq ||
+      color_src >= command_buffer::kNumBlendFunc ||
+      color_dst >= command_buffer::kNumBlendFunc ||
+      alpha_eq >= command_buffer::kNumBlendEq ||
+      alpha_src >= command_buffer::kNumBlendFunc ||
+      alpha_dst >= command_buffer::kNumBlendFunc)
     return BufferSyncInterface::kParseInvalidArguments;
   gapi_->SetBlending(enable,
                      separate_alpha,
-                     static_cast<GAPIInterface::BlendEq>(color_eq),
-                     static_cast<GAPIInterface::BlendFunc>(color_src),
-                     static_cast<GAPIInterface::BlendFunc>(color_dst),
-                     static_cast<GAPIInterface::BlendEq>(alpha_eq),
-                     static_cast<GAPIInterface::BlendFunc>(alpha_src),
-                     static_cast<GAPIInterface::BlendFunc>(alpha_dst));
+                     static_cast<command_buffer::BlendEq>(color_eq),
+                     static_cast<command_buffer::BlendFunc>(color_src),
+                     static_cast<command_buffer::BlendFunc>(color_dst),
+                     static_cast<command_buffer::BlendEq>(alpha_eq),
+                     static_cast<command_buffer::BlendFunc>(alpha_src),
+                     static_cast<command_buffer::BlendFunc>(alpha_dst));
   return BufferSyncInterface::kParseNoError;
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetBlendingColor(
     uint32 arg_count,
     const cmd::SetBlendingColor& args) {
   RGBA rgba;
   rgba.red = args.red;
   rgba.green = args.green;
   rgba.blue = args.blue;
   rgba.alpha = args.alpha;
   gapi_->SetBlendingColor(rgba);
   return BufferSyncInterface::kParseNoError;
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleCreateRenderSurface(
     uint32 arg_count,
     const cmd::CreateRenderSurface& args) {
-  namespace cmd = create_render_surface_cmd;
-  unsigned int width_height = args.fixme1;
-  unsigned int width = cmd::Width::Get(width_height);
-  unsigned int height = cmd::Height::Get(width_height);
-  unsigned int levels_side = args.fixme2;
-  unsigned int mip_level = cmd::Levels::Get(levels_side);
-  unsigned int side = cmd::Side::Get(levels_side);
-  return gapi_->CreateRenderSurface(args.id, width, height, mip_level,
+  unsigned int width_height = args.width_height;
+  unsigned int width = cmd::CreateRenderSurface::Width::Get(width_height);
+  unsigned int height = cmd::CreateRenderSurface::Height::Get(width_height);
+  unsigned int levels_side = args.levels_side;
+  unsigned int mip_level = cmd::CreateRenderSurface::Levels::Get(levels_side);
+  unsigned int side = cmd::CreateRenderSurface::Side::Get(levels_side);
+  return gapi_->CreateRenderSurface(args.render_surface_id,
+                                    width, height, mip_level,
                                     side, args.texture_id);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleDestroyRenderSurface(
     uint32 arg_count,
     const cmd::DestroyRenderSurface& args) {
-  return gapi_->DestroyRenderSurface(args.id);
+  return gapi_->DestroyRenderSurface(args.render_surface_id);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleCreateDepthSurface(
     uint32 arg_count,
     const cmd::CreateDepthSurface& args) {
-  namespace cmd = create_depth_surface_cmd;
-  unsigned int width_height = args.fixme1;
-  unsigned int width = cmd::Width::Get(width_height);
-  unsigned int height = cmd::Height::Get(width_height);
-  return gapi_->CreateDepthSurface(args.id, width, height);
+  unsigned int width_height = args.width_height;
+  unsigned int width = cmd::CreateDepthSurface::Width::Get(width_height);
+  unsigned int height = cmd::CreateDepthSurface::Height::Get(width_height);
+  return gapi_->CreateDepthSurface(args.depth_surface_id, width, height);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleDestroyDepthSurface(
     uint32 arg_count,
     const cmd::DestroyDepthSurface& args) {
-  return gapi_->DestroyDepthSurface(args.id);
+  return gapi_->DestroyDepthSurface(args.depth_surface_id);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetRenderSurface(
     uint32 arg_count,
     const cmd::SetRenderSurface& args) {
   return gapi_->SetRenderSurface(args.render_surface_id, args.depth_surface_id);
 }
 
 BufferSyncInterface::ParseError GAPIDecoder::HandleSetBackSurfaces(
     uint32 arg_count,
     const cmd::SetBackSurfaces& args) {
   gapi_->SetBackSurfaces();
   return  BufferSyncInterface::kParseNoError;
 }
 
 }  // namespace command_buffer
 }  // namespace o3d