cc: Remove LayerTextureUpdater::Texture::updateRect() callback.

cc/CCResourceProvider.h

Index: cc/CCResourceProvider.h
diff --git a/cc/CCResourceProvider.h b/cc/CCResourceProvider.h
index 1e08882bec6f5900e342e602d9f99cc5b03ec97c..37bf03f06c6be9ec9f93d74d4e6c50c915f63e9f 100644
--- a/cc/CCResourceProvider.h
+++ b/cc/CCResourceProvider.h
@@ -95,9 +95,12 @@ public:
     // Deletes all resources owned by a given pool.
     void deleteOwnedResources(int pool);
 
-    // Upload data from image, copying sourceRect (in image) into destRect (in the resource).
+    // Upload data from image, copying sourceRect (in image) to destOffset (in the resource).
     void upload(ResourceId, const uint8_t* image, const IntRect& imageRect, const IntRect& sourceRect, const IntSize& destOffset);
 
+    // Playback contents from picture, painting sourceRect (in picture) to destOffset (in the resource).
+    void acceleratedUpdate(ResourceId, SkPicture*, const IntRect& pictureRect, const IntRect& sourceRect, const IntSize& destOffset);

[piman, 2012/10/08 18:33:42]

This feels a bit out of place. I'd rather have this sit in a separate
function/class, outside of CCResourceProvider if possible. I would like to keep
CCResourceProvider as simple as possible, and not know anything about painting,
or the magic shared contexts for example.

[reveman, 2012/10/08 19:45:02]

Ok, maybe it belongs in CCTextureUpdateController. Whether we can use
accelerated painting or not depends on the resource type so we'll need some
minimal CCResourceProvider interface to handle accelerated painting properly.

+
     // Flush all context operations, kicking uploads and ensuring ordering with
     // respect to other contexts.
     void flush();