cc: Remove redundant is_overlay arguments

cc/resources/resource_provider.h

Index: cc/resources/resource_provider.h
diff --git a/cc/resources/resource_provider.h b/cc/resources/resource_provider.h
index 4af135eab57613d1070f77558020606222b00357..cc118a87682a0b114cc9b9f96ad173718c5ad0c5 100644
--- a/cc/resources/resource_provider.h
+++ b/cc/resources/resource_provider.h
@@ -422,6 +422,8 @@ class CC_EXPORT ResourceProvider
   // Indicates if we can currently lock this resource for write.
   bool CanLockForWrite(ResourceId id);
 
+  bool IsBoundToImage(ResourceId id);
+
   void WaitSyncPointIfNeeded(ResourceId id);
 
   static GLint GetActiveTextureUnit(gpu::gles2::GLES2Interface* gl);
@@ -495,6 +497,7 @@ class CC_EXPORT ResourceProvider
     bool allocated : 1;
     bool read_lock_fences_enabled : 1;
     bool has_shared_bitmap_id : 1;
+    bool bound_to_image : 1;

[reveman, 2015/10/07 09:32:58]

I think this is a bit confusing as we already have Resource::bound_image_id. I
think "allow_overlay" or simply "overlay_candidate" would be better.

[ccameron, 2015/10/07 19:13:11]

Yeah the original name promises a bit too much. Changed to
"is_overlay_candidate" (because "overlay_candidate" name suggests the type is
not bool).

[reveman, 2015/10/07 19:24:18]

Good point.

     scoped_refptr<Fence> read_lock_fence;
     gfx::Size size;
     Origin origin;