cc: Hide Gpu Rasterization details in Resource Provider.

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 356 matching lines @@
   void EnableReadLockFences(ResourceProvider::ResourceId id, bool enable);
 
   // Indicates if we can currently lock this resource for write.
   bool CanLockForWrite(ResourceId id);
 
   // Copy pixels from source to destination.
   void CopyResource(ResourceId source_id, ResourceId dest_id);
 
   static GLint GetActiveTextureUnit(gpu::gles2::GLES2Interface* gl);
 
+  // Hide Gpu Raster implementation details.

[reveman, 2014/07/01 14:49:23]

Please explain what these functions are used for instead of what you're trying
to do with this patch.

[sohanjg, 2014/07/01 16:25:59]

Done.

+  void PushGroupMarker(const char* name);
+  void PopGroupMarker();
+
+  void ResetGpuContext();
+  void FlushGpuContext();

[reveman, 2014/07/01 14:49:23]

I think we can come up with a better API than this. Please combine these two
pair of functions into one pair. And I think it'd be nice to expose it using a
"Scoped" class. ie. private API Begin/EndGpuRaster(), public API "class
ScopedGpuRaster".

[sohanjg, 2014/07/01 16:25:59]

Done.

+
  private:
   class GpuRasterBuffer;
   class ImageRasterBuffer;
   class PixelRasterBuffer;
 
   struct Resource {
     enum Origin { Internal, External, Delegated };
 
     Resource();
     ~Resource();
@@ skipping 315 matching lines @@
   return format_gl_data_format[format];
 }
 
 inline GLenum GLInternalFormat(ResourceFormat format) {
   return GLDataFormat(format);
 }
 
 }  // namespace cc
 
 #endif  // CC_RESOURCES_RESOURCE_PROVIDER_H_