cc: Only make num_threads - 1 SkPicture clones.

cc/resources/picture.cc

Index: cc/resources/picture.cc
diff --git a/cc/resources/picture.cc b/cc/resources/picture.cc
index 2034ecb9bea064da73f3d494a2884d8a6447a9ce..02d7650e00fcf41897f5f1ee33137b70296869e3 100644
--- a/cc/resources/picture.cc
+++ b/cc/resources/picture.cc
@@ -92,11 +92,7 @@ scoped_refptr<Picture> Picture::Create(
   picture->Record(client, tile_grid_info);
   if (gather_pixel_refs)
     picture->GatherPixelRefs(tile_grid_info);
-  if (num_raster_threads > 0)
-    picture->CloneForDrawing(num_raster_threads);
-
-  // This picture may be rasterized on a different thread.
-  picture->raster_thread_checker_.DetachFromThread();
+  picture->CloneForDrawing(num_raster_threads);
 
   return picture;
 }
@@ -192,12 +188,11 @@ Picture::~Picture() {
     TRACE_DISABLED_BY_DEFAULT("cc.debug"), "cc::Picture", this);
 }
 
-scoped_refptr<Picture> Picture::GetCloneForDrawingOnThread(
-    unsigned thread_index) const {
+Picture* Picture::GetCloneForDrawingOnThread(unsigned thread_index) {
   // SkPicture is not thread-safe to rasterize with, this returns a clone
   // to rasterize with on a specific thread.
-  CHECK_GT(clones_.size(), thread_index);
-  return clones_[thread_index];
+  CHECK_GE(clones_.size(), thread_index);
+  return thread_index == clones_.size() ? this : clones_[thread_index].get();
 }
 
 void Picture::CloneForDrawing(int num_threads) {
@@ -206,19 +201,24 @@ void Picture::CloneForDrawing(int num_threads) {
   DCHECK(picture_);
   DCHECK(clones_.empty());
 
-  scoped_ptr<SkPicture[]> clones(new SkPicture[num_threads]);
-  picture_->clone(&clones[0], num_threads);
+  // We can re-use this picture for one raster worker thread.
+  raster_thread_checker_.DetachFromThread();
+
+  if (num_threads > 1) {
+    scoped_ptr<SkPicture[]> clones(new SkPicture[num_threads - 1]);
+    picture_->clone(&clones[0], num_threads - 1);
 
-  for (int i = 0; i < num_threads; i++) {
-    scoped_refptr<Picture> clone = make_scoped_refptr(
-        new Picture(skia::AdoptRef(new SkPicture(clones[i])),
-                    layer_rect_,
-                    opaque_rect_,
-                    pixel_refs_));
-    clones_.push_back(clone);
+    for (int i = 0; i < num_threads - 1; i++) {
+      scoped_refptr<Picture> clone = make_scoped_refptr(
+          new Picture(skia::AdoptRef(new SkPicture(clones[i])),
+                      layer_rect_,
+                      opaque_rect_,
+                      pixel_refs_));
+      clones_.push_back(clone);
 
-    clone->EmitTraceSnapshotAlias(this);
-    clone->raster_thread_checker_.DetachFromThread();
+      clone->EmitTraceSnapshotAlias(this);

[enne (OOO), 2014/02/11 22:30:45]

Do you need to emit the trace snapshot alias for the 0th picture too?

[reveman, 2014/02/11 23:40:42]

Not needed as discussed.

+      clone->raster_thread_checker_.DetachFromThread();
+    }
   }
 }