cc: return resources via a ReturnedResource struct rather than TransferableResource

cc/resources/transferable_resource.h

Index: cc/resources/transferable_resource.h
diff --git a/cc/resources/transferable_resource.h b/cc/resources/transferable_resource.h
index 5c979433c90c1f13cba2949b3f963ef76bf87c8c..872f1682b6ffb1fa4aefa3e5bcad4a5b405bfc69 100644
--- a/cc/resources/transferable_resource.h
+++ b/cc/resources/transferable_resource.h
@@ -14,10 +14,14 @@
 
 namespace cc {
 
+struct ReturnedResource;
+
 struct CC_EXPORT TransferableResource {
   TransferableResource();
   ~TransferableResource();
 
+  ReturnedResource Return() const;
+
   unsigned id;
   unsigned sync_point;
   uint32 format;
@@ -28,6 +32,19 @@ struct CC_EXPORT TransferableResource {
 
 typedef std::vector<TransferableResource> TransferableResourceArray;
 
+struct CC_EXPORT ReturnedResource {

[danakj, 2013/08/16 23:33:35]

Can you move this to its own file please?

[piman, 2013/08/17 01:54:31]

Done.

+  ReturnedResource() : id(0), sync_point(0), filter(0), count(0) {}
+  unsigned id;
+  unsigned sync_point;
+  uint32 filter;
+  int count;
+};
+
+typedef std::vector<ReturnedResource> ReturnedResourceArray;
+
+CC_EXPORT void ReturnResources(const TransferableResourceArray& input,

[danakj, 2013/08/16 23:33:35]

I'm kinda meh about a free function like this, as it's hard to tell where the
function comes from when you look at the call sites.

WDYT of making ReturnedResourceArray a subclass of std::vector<RR>, and making
this a (static?) method of the class?

[piman, 2013/08/17 01:54:31]

As discussed, made it a static in TransferableResource

+                               ReturnedResourceArray* output);
+
 }  // namespace cc
 
 #endif  // CC_RESOURCES_TRANSFERABLE_RESOURCE_H_