Rename paint data structures

cc/test/skia_common.cc

Index: cc/test/skia_common.cc
diff --git a/cc/test/skia_common.cc b/cc/test/skia_common.cc
index ec399087f54e8ee19f2722150eec95d0a12bdc7d..320bbdd1154414e879953ed015638f5474f9d359 100644
--- a/cc/test/skia_common.cc
+++ b/cc/test/skia_common.cc
@@ -6,8 +6,8 @@
 
 #include <stddef.h>
 
+#include "cc/paint/paint_canvas.h"
 #include "cc/playback/display_item_list.h"
-#include "third_party/skia/include/core/SkCanvas.h"
 #include "third_party/skia/include/core/SkImageGenerator.h"
 #include "third_party/skia/include/core/SkPixmap.h"
 #include "ui/gfx/geometry/rect.h"
@@ -47,7 +47,7 @@ void DrawDisplayList(unsigned char* buffer,
       SkImageInfo::MakeN32Premul(layer_rect.width(), layer_rect.height());
   SkBitmap bitmap;
   bitmap.installPixels(info, buffer, info.minRowBytes());
-  SkCanvas canvas(bitmap);
+  PaintCanvas canvas(bitmap);

[danakj, 2017/01/20 23:34:14]

this is raster, shouldn't this be SkCanvas?

[enne (OOO), 2017/01/24 01:51:28]

So, this depends on what you think the future should look like in terms of the
RasterSource/DisplayItemList/DisplayItem interface.  Should they have:

(a) Raster(PaintCanvas*), and if you want to use an SkCanvas to raster directly,
then you wrap it in a PaintCanvas.
(b) Raster(PaintCanvas*) and Raster(SkCanvas*) with extremely similar
functionality.

I think as part of the command buffer canvas step, we're going to want to
"raster" into the PaintCanvas interface.  In a far off future, I'd love to just
transport display lists and paint records (and even then only transport pieces
that have been changed), but I think we should ship the command buffer canvas
intermediate step in the short term.

[danakj, 2017/01/24 15:32:07]

OK ya I think you're right.

   canvas.clipRect(gfx::RectToSkRect(layer_rect));
   list->Raster(&canvas, NULL, layer_rect, 1.0f);
 }