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..e04c8041b57fbcf9778b75c1b371e995aeb07761 100644
--- a/cc/resources/resource_pool.h
+++ b/cc/resources/resource_pool.h
@@ -18,6 +18,10 @@ class ScopedResource;
 
 class CC_EXPORT ResourcePool {
  public:
+  static scoped_ptr<ResourcePool> Create(ResourceProvider* resource_provider) {
+    return make_scoped_ptr(new ResourcePool(resource_provider));
+  }
+
   static scoped_ptr<ResourcePool> Create(ResourceProvider* resource_provider,
                                          GLenum target) {
     return make_scoped_ptr(new ResourcePool(resource_provider, target));
@@ -52,6 +56,7 @@ class CC_EXPORT ResourcePool {
   size_t busy_resource_count() const { return busy_resources_.size(); }
 
  protected:
+  ResourcePool(ResourceProvider* resource_provider);

[reveman, 2015/07/28 08:58:25]

nit: explicit ResourcePool(ResourceProv...

   ResourcePool(ResourceProvider* resource_provider, GLenum target);

[reveman, 2015/07/28 08:58:25]

Can we remove this ctor with "target" and and always use the image texture
target?

[jchen10, 2015/07/29 04:33:47]

I'm afraid we can't. The image texture is only used by the zero-copy and
one-copy tile task worker pools. Other pools(including the one-copy) need to use
GL_TEXTURE_2D target. The functions that can be used for GL_TEXTURE_2D are not
guaranteed by other texture targets, e.g. "gl*Tex*Image*()" with
GL_TEXTURE_EXTERNAL_OES(See
https://www.khronos.org/registry/gles/extensions/OES/OES_EGL_image_external.txt).
Actually I have ever tried it, but met some wired gfx error.

[reveman, 2015/07/30 02:41:43]

Ok. No need to do anything in this patch but I think we should change this so
the user of this API will always have to ask for a specific target when
acquiring a resource.

 
   bool ResourceUsageTooHigh();