cc: Changes to use GL API for GpuMemoryBuffers

cc/resources/resource_provider.h

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CC_RESOURCES_RESOURCE_PROVIDER_H_
 #define CC_RESOURCES_RESOURCE_PROVIDER_H_
 
 #include <deque>
 #include <set>
 #include <string>
@@ skipping 274 matching lines @@
 
   // Update pixels from acquired pixel buffer.
   void SetPixelsFromBuffer(ResourceId id);
 
   // Asynchronously update pixels from acquired pixel buffer.
   void BeginSetPixels(ResourceId id);
   void ForceSetPixelsToComplete(ResourceId id);
   bool DidSetPixelsComplete(ResourceId id);
   void AbortSetPixels(ResourceId id);
 
+  // Methods for GLImage zero-copy API.

[reveman, 2013/05/20 23:16:01]

I think these functions could need a bit more of a description. Can you split
them up like the pixel buffer functions above and add a line or to describing
what they do? Btw, the GLImage name is an implementation detail, please avoid
using it here.

[kaanb, 2013/05/21 00:52:29]

Done.

+  void AcquireImage(ResourceId id);
+  void ReleaseImage(ResourceId id);
+  uint8_t* MapImage(ResourceId id);
+  void UnmapImage(ResourceId id);
+  void BindImage(ResourceId id);
+  int GetImageStride(ResourceId id);
+
   // For tests only! This prevents detecting uninitialized reads.
   // Use SetPixels or LockForWrite to allocate implicitly.
   void AllocateForTesting(ResourceId id);
 
   // Sets the current read fence. If a resource is locked for read
   // and has read fences enabled, the resource will not allow writes
   // until this fence has passed.
   void SetReadLockFence(scoped_refptr<Fence> fence) {
     current_read_lock_fence_ = fence;
   }
@@ skipping 35 matching lines @@
     bool marked_for_deletion;
     bool pending_set_pixels;
     bool set_pixels_completion_forced;
     bool allocated;
     bool enable_read_lock_fences;
     scoped_refptr<Fence> read_lock_fence;
     gfx::Size size;
     GLenum format;
     // TODO(skyostil): Use a separate sampler object for filter state.
     GLenum filter;
+    unsigned image_id;
     ResourceType type;
   };
   typedef base::hash_map<ResourceId, Resource> ResourceMap;
   struct Child {
     Child();
     ~Child();
 
     ResourceIdMap child_to_parent_map;
     ResourceIdMap parent_to_child_map;
   };
@@ skipping 45 matching lines @@
   base::ThreadChecker thread_checker_;
 
   scoped_refptr<Fence> current_read_lock_fence_;
 
   DISALLOW_COPY_AND_ASSIGN(ResourceProvider);
 };
 
 }  // namespace cc
 
 #endif  // CC_RESOURCES_RESOURCE_PROVIDER_H_