cc: Remove DrawQuad::IterateResoruces

cc/layers/delegated_renderer_layer_impl.cc

Index: cc/layers/delegated_renderer_layer_impl.cc
diff --git a/cc/layers/delegated_renderer_layer_impl.cc b/cc/layers/delegated_renderer_layer_impl.cc
index 5b6bc6cc3fff7c9cf45bab30afddc738919dc47e..3d5e6ddbdf2af19b48c4582d86badeefa42478bc 100644
--- a/cc/layers/delegated_renderer_layer_impl.cc
+++ b/cc/layers/delegated_renderer_layer_impl.cc
@@ -134,14 +134,14 @@ void DelegatedRendererLayerImpl::SetFrameData(
   if (reserve_size)
     resources_in_frame.resize(reserve_size);
 #endif
-  DrawQuad::ResourceIteratorCallback remap_resources_to_parent_callback =
-      base::Bind(&ResourceRemapHelper,
-                 &invalid_frame,
-                 resource_map,
-                 &resources_in_frame);
   for (const auto& pass : render_pass_list) {
     for (const auto& quad : pass->quad_list)

[danakj, 2015/05/28 17:10:05]

{}

[vmpstr, 2015/05/28 18:38:45]

Done.

-      quad->IterateResources(remap_resources_to_parent_callback);
+      quad->IterateResources(

[danakj, 2015/05/28 17:10:05]

Does this look nicer/worse with just walking thru an array? It seems like it'd
be simpler to me, did you try it?

[piman, 2015/05/28 18:37:59]

Actually... I think we may be able to do something nice if we just add a begin()
and end() to DrawQuad::Resources, then I think we can do something like
for (ResourceId& resource_id : quad->resources()) {
  resource_id = ...
}

[vmpstr, 2015/05/28 18:38:45]

Yep, that does look better. In the current version there's no
IterateResources...

+          [&invalid_frame, &resource_map, &resources_in_frame](
+              ResourceId resource_id) {
+            return ResourceRemapHelper(&invalid_frame, resource_map,

[danakj, 2015/05/28 17:10:05]

I *think* I'd just put the body of this helper here.. unless that's difficult to
read. I'm new to lambdas.

[vmpstr, 2015/05/28 18:38:45]

Yeah, I could, but I prefer keeping lambdas very small... otherwise you're just
reading another function inside a function, which is kinda weird...

+                                       &resources_in_frame, resource_id);
+          });
   }
 
   if (invalid_frame) {