cc: Shrink size of display item

cc/trees/layer_tree_host_pixeltest_tiles.cc

 // Copyright 2015 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.
 
 #include "cc/layers/content_layer_client.h"
 #include "cc/layers/picture_layer.h"
 #include "cc/output/copy_output_request.h"
 #include "cc/playback/display_item_list.h"
 #include "cc/playback/display_item_list_settings.h"
 #include "cc/playback/drawing_display_item.h"
@@ skipping 115 matching lines @@
     paint.setStyle(SkPaint::kFill_Style);
 
     paint.setColor(SK_ColorBLUE);
     canvas->drawRect(gfx::RectToSkRect(blue_rect), paint);
     paint.setColor(SK_ColorYELLOW);
     canvas->drawRect(gfx::RectToSkRect(yellow_rect), paint);
 
     skia::RefPtr<SkPicture> picture =
         skia::AdoptRef(recorder.endRecordingAsPicture());
 
-    auto* item = display_list->CreateAndAppendItem<DrawingDisplayItem>(
-        PaintableRegion());
-    item->SetNew(std::move(picture));
-
+    display_list->CreateAndAppendItem<DrawingDisplayItem>(PaintableRegion(),
+                                                          std::move(picture));

[vmpstr, 2015/12/04 00:43:09]

For all these 

skia::RefPtr<SkPicture> picture = stuff;
...(..., std::move(picture));

i wonder if it would just be cleaner to

...(..., stuff); directly? Or do you think this is cleaner?

[danakj, 2015/12/04 00:53:54]

When you get a pointer from skia APIs the first thing you do with it is put it
in a skia::RefPtr.

[vmpstr, 2015/12/04 01:24:28]

"stuff" would include skia::AdoptRef()

[danakj, 2015/12/04 01:24:54]

Ooh ok that'd be fine. :)

     display_list->Finalize();
     return display_list;
   }
 
   bool FillsBoundsCompletely() const override { return true; }
   size_t GetApproximateUnsharedMemoryUsage() const override { return 0; }
 
   void set_blue_top(bool b) { blue_top_ = b; }
 
  private:
@@ skipping 87 matching lines @@
        FullRaster_SingleThread_GpuRaster) {
   RunRasterPixelTest(
       false, FULL_GPU, picture_layer_,
       base::FilePath(FILE_PATH_LITERAL("blue_yellow_flipped.png")));
 }
 
 }  // namespace
 }  // namespace cc
 
 #endif  // !defined(OS_ANDROID)