cc: Remove SkSurface cache from Resource

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 3a8d2e0f9ec0325ae56e64de274f6caa3ac073a9..8ab15dd555c495d7283b2e55357220b0a05589ed 100644
--- a/cc/resources/resource_provider.h
+++ b/cc/resources/resource_provider.h
@@ -335,13 +335,10 @@ class CC_EXPORT ResourceProvider {
                       ResourceProvider::ResourceId resource_id);
     ~ScopedWriteLockGr();
 
-    SkSurface* GetSkSurface(bool use_distance_field_text,
-                            bool can_use_lcd_text);
+    skia::RefPtr<SkSurface> GetSkSurface(bool use_distance_field_text,
+                                         bool can_use_lcd_text);

[reveman, 2015/01/27 13:10:14]

Does this have to return a RefPtr? I prefer if we store the reference as a
ScopedWriteLockGr member and keep having this return a raw pointer and no
ownership of the surface. If you need to pass ownership, then I think the name
of this function should probable change. ie. CreateSkSurface. 

 
    private:
-    bool SurfaceHasMatchingProperties(bool use_distance_field_text,
-                                      bool can_use_lcd_text) const;
-
     ResourceProvider* resource_provider_;
     ResourceProvider::Resource* resource_;
     base::ThreadChecker thread_checker_;
@@ -494,7 +491,6 @@ class CC_EXPORT ResourceProvider {
     SharedBitmapId shared_bitmap_id;
     SharedBitmap* shared_bitmap;
     gfx::GpuMemoryBuffer* gpu_memory_buffer;
-    skia::RefPtr<SkSurface> sk_surface;
   };
   typedef base::hash_map<ResourceId, Resource> ResourceMap;