Implement RenderPass StructTraits

cc/ipc/quads_struct_traits.cc

Index: cc/ipc/quads_struct_traits.cc
diff --git a/cc/ipc/quads_struct_traits.cc b/cc/ipc/quads_struct_traits.cc
index 26e1d3f5b9548ab40569213a01bbdc9aff39a71c..28b636539a47d1db12592da8b90d23d78f96e022 100644
--- a/cc/ipc/quads_struct_traits.cc
+++ b/cc/ipc/quads_struct_traits.cc
@@ -146,6 +146,7 @@ bool StructTraits<cc::mojom::RenderPassQuadState, cc::DrawQuad>::Read(
   cc::RenderPassDrawQuad* quad = static_cast<cc::RenderPassDrawQuad*>(out);
   quad->resources.ids[cc::RenderPassDrawQuad::kMaskResourceIdIndex] =
       data.mask_resource_id();
+  quad->resources.count = data.mask_resource_id() ? 1 : 0;
   return data.ReadRenderPassId(&quad->render_pass_id) &&
          data.ReadMaskUvScale(&quad->mask_uv_scale) &&
          data.ReadMaskTextureSize(&quad->mask_texture_size) &&
@@ -179,6 +180,7 @@ bool StructTraits<cc::mojom::TextureQuadState, cc::DrawQuad>::Read(
   cc::TextureDrawQuad* quad = static_cast<cc::TextureDrawQuad*>(out);
   quad->resources.ids[cc::TextureDrawQuad::kResourceIdIndex] =
       data.resource_id();
+  quad->resources.count = 1;
   quad->premultiplied_alpha = data.premultiplied_alpha();
   if (!data.ReadUvTopLeft(&quad->uv_top_left) ||
       !data.ReadUvBottomRight(&quad->uv_bottom_right)) {
@@ -208,6 +210,7 @@ bool StructTraits<cc::mojom::TileQuadState, cc::DrawQuad>::Read(
   quad->swizzle_contents = data.swizzle_contents();
   quad->nearest_neighbor = data.nearest_neighbor();
   quad->resources.ids[cc::TileDrawQuad::kResourceIdIndex] = data.resource_id();
+  quad->resources.count = 1;
   return true;
 }
 
@@ -263,6 +266,8 @@ bool StructTraits<cc::mojom::YUVVideoQuadState, cc::DrawQuad>::Read(
       data.v_plane_resource_id();
   quad->resources.ids[cc::YUVVideoDrawQuad::kAPlaneResourceIdIndex] =
       data.a_plane_resource_id();
+  quad->resources.count = data.a_plane_resource_id() ? 4 : 3;

[danakj, 2016/06/22 20:51:35]

This depends on the A resource coming last in the array, since you insert them
based on the A index, not just in order that they are written. Maybe you can
static assert this here so we know it's the case in the future?

[Fady Samuel, 2016/06/22 21:41:19]

Done.

+
   if (!data.ReadColorSpace(&quad->color_space))
     return false;
   quad->resource_offset = data.resource_offset();