cc: Refactor ResourceProvider.

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 135 matching lines @@
                  const gfx::Vector2d& dest_offset);
 
   // Check upload status.
   size_t NumBlockingUploads();
   void MarkPendingUploadsAsNonBlocking();
   size_t EstimatedUploadsPerTick();
   void FlushUploads();
   void ReleaseCachedData();
   base::TimeTicks EstimatedUploadCompletionTime(size_t uploads_per_tick);
 
-  // Flush all context operations, kicking uploads and ensuring ordering with
-  // respect to other contexts.
-  void Flush();
-
-  // Finish all context operations, causing any pending callbacks to be
-  // scheduled.
-  void Finish();
-
   // Only flush the command buffer if supported.
   // Returns true if the shallow flush occurred, false otherwise.
   bool ShallowFlushIfSupported();
 
   // Creates accounting for a child. Returns a child ID.
   int CreateChild(const ReturnCallback& return_callback);
 
   // Destroys accounting for the child, deleting all accounted resources.
   void DestroyChild(int child);
 
@@ skipping 125 matching lines @@
 
    private:
     ResourceProvider* resource_provider_;
     ResourceProvider::ResourceId resource_id_;
     SkBitmap sk_bitmap_;
     scoped_ptr<SkCanvas> sk_canvas_;
 
     DISALLOW_COPY_AND_ASSIGN(ScopedWriteLockSoftware);
   };
 
-  // The following class is needed to modify GL resources using GPU
-  // raster. The user must ensure that they only use GPU raster on
-  // GL resources while an instance of this class is alive.
-  class CC_EXPORT ScopedGpuRaster {
-   public:
-    ScopedGpuRaster(ResourceProvider* resource_provider);
-    ~ScopedGpuRaster();
-
-   private:
-    ResourceProvider* resource_provider_;
-
-    DISALLOW_COPY_AND_ASSIGN(ScopedGpuRaster);
-  };
-
   class Fence : public base::RefCounted<Fence> {
    public:
     Fence() {}
     virtual bool HasPassed() = 0;
 
    protected:
     friend class base::RefCounted<Fence>;
     virtual ~Fence() {}
 
    private:
@@ skipping 48 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);
 
+  // Returns NULL if the output_surface_ does not have a ContextProvider.
+  gpu::gles2::GLES2Interface* ContextGL() const;
+  class GrContext* GrContext() const;
+
  private:
   class GpuRasterBuffer;
   class ImageRasterBuffer;
   class PixelRasterBuffer;
 
   struct Resource {
     enum Origin { Internal, External, Delegated };
 
     Resource();
     ~Resource();
@@ skipping 229 matching lines @@
   void UnmapImage(const Resource* resource);
 
   void BindImageForSampling(Resource* resource);
   // Binds the given GL resource to a texture target for sampling using the
   // specified filter for both minification and magnification. Returns the
   // texture target used. The resource must be locked for reading.
   GLenum BindForSampling(ResourceProvider::ResourceId resource_id,
                          GLenum unit,
                          GLenum filter);
 
-  // Returns NULL if the output_surface_ does not have a ContextProvider.
-  gpu::gles2::GLES2Interface* ContextGL() const;
-  class GrContext* GrContext() const;
-
-  void BeginGpuRaster();
-  void EndGpuRaster();
-
   OutputSurface* output_surface_;
   SharedBitmapManager* shared_bitmap_manager_;
   bool lost_output_surface_;
   int highp_threshold_min_;
   ResourceId next_id_;
   ResourceMap resources_;
   int next_child_;
   ChildMap children_;
 
   ResourceType default_resource_type_;
@@ skipping 13 matching lines @@
   scoped_ptr<IdAllocator> texture_id_allocator_;
   scoped_ptr<IdAllocator> buffer_id_allocator_;
 
   bool use_sync_query_;
 
   bool use_distance_field_text_;
 
   DISALLOW_COPY_AND_ASSIGN(ResourceProvider);
 };
 
+// The following class is needed to modify GL resources using GPU
+// raster. The user must ensure that they only use GPU raster on
+// GL resources while an instance of this class is alive.
+class CC_EXPORT ScopedGpuRaster {

[danakj, 2014/07/10 17:01:44]

move to its own file, 1 class per file

[sohanjg, 2014/07/11 16:43:32]

Done.

+ public:
+  ScopedGpuRaster(ResourceProvider* resource_provider);
+  ~ScopedGpuRaster();
+
+ private:
+  void BeginGpuRaster();
+  void EndGpuRaster();
+
+  ResourceProvider* resource_provider_;
+
+  DISALLOW_COPY_AND_ASSIGN(ScopedGpuRaster);
+};
 
 // TODO(epenner): Move these format conversions to resource_format.h
 // once that builds on mac (npapi.h currently #includes OpenGL.h).
 inline unsigned BitsPerPixel(ResourceFormat format) {
   DCHECK_LE(format, RESOURCE_FORMAT_MAX);
   static const unsigned format_bits_per_pixel[RESOURCE_FORMAT_MAX + 1] = {
     32,  // RGBA_8888
     16,  // RGBA_4444
     32,  // BGRA_8888
     8,   // LUMINANCE_8
@@ skipping 29 matching lines @@
   return format_gl_data_format[format];
 }
 
 inline GLenum GLInternalFormat(ResourceFormat format) {
   return GLDataFormat(format);
 }
 
 }  // namespace cc
 
 #endif  // CC_RESOURCES_RESOURCE_PROVIDER_H_