cc: Fix the format of GpuMemoryBuffer for SurfaceTexture

cc/resources/resource_pool.h

Index: cc/resources/resource_pool.h
diff --git a/cc/resources/resource_pool.h b/cc/resources/resource_pool.h
index 53f4135e4d084482e8606d8333c75c97c88a9590..93afe54eda3817d709f46244debbdd06977715e3 100644
--- a/cc/resources/resource_pool.h
+++ b/cc/resources/resource_pool.h
@@ -19,8 +19,8 @@ class ScopedResource;
 class CC_EXPORT ResourcePool {
  public:
   static scoped_ptr<ResourcePool> Create(ResourceProvider* resource_provider,
-                                         GLenum target) {
-    return make_scoped_ptr(new ResourcePool(resource_provider, target));
+                                         std::vector<uint> targets) {

[reveman, 2015/07/27 17:07:15]

nit: here and everywhere else in this patch "std::vector<unsigned>". that or
uint32 is preferred. we use unsigned for texture target in general as that what
GL.h defines GLenum to as I think that's best.

+    return make_scoped_ptr(new ResourcePool(resource_provider, targets));
   }
 
   virtual ~ResourcePool();
@@ -52,7 +52,7 @@ class CC_EXPORT ResourcePool {
   size_t busy_resource_count() const { return busy_resources_.size(); }
 
  protected:
-  ResourcePool(ResourceProvider* resource_provider, GLenum target);
+  ResourcePool(ResourceProvider* resource_provider, std::vector<uint> targets);

[reveman, 2015/07/27 17:07:15]

Instead of having to pass the target vector to the ResourcePool, can we keep
this in the ResourceProvider and just have a helper function there to get the
correct target for a specific resource format?

 
   bool ResourceUsageTooHigh();
 
@@ -60,7 +60,7 @@ class CC_EXPORT ResourcePool {
   void DidFinishUsingResource(ScopedResource* resource, uint64_t content_id);
 
   ResourceProvider* resource_provider_;
-  const GLenum target_;
+  const std::vector<uint> targets_;
   size_t max_memory_usage_bytes_;
   size_t max_unused_memory_usage_bytes_;
   size_t max_resource_count_;