Change drawPicture calls to playback to avoid culling

cc/paint/paint_canvas.h

Index: cc/paint/paint_canvas.h
diff --git a/cc/paint/paint_canvas.h b/cc/paint/paint_canvas.h
index cae112e57e9951ea7f468c5c79f9a02e24e85f2c..f753af48dc18f55a05e21d12cfb03b02eda23886 100644
--- a/cc/paint/paint_canvas.h
+++ b/cc/paint/paint_canvas.h
@@ -196,6 +196,10 @@ class CC_PAINT_EXPORT PaintCanvas {
   virtual void AnnotateNamedDestination(const SkPoint& point, SkData* data) = 0;
   virtual void AnnotateLinkToDestination(const SkRect& rect, SkData* data) = 0;
 
+  // TODO(enne): ideally this should live on PaintRecord, but that's not

[danakj, 2017/03/20 14:53:54]

Or, more like this shouldn't exist really and PLS DONT USE IT and here's a bug
where we can delete this, and such?

[enne (OOO), 2017/03/20 17:41:26]

The cleanup here is just moving one function from PaintCanvas to PaintRecord, so
it's not really something that needs a "don't use this because it'll make it
harder to remove later" sort of thing.  I think a TODO suffices, to be honest. 
Let me know if you disagree and I'll file a bug.

It's also not clear to me if that cleanup is the right thing to do? We kind of
want 3 things
(1) PaintRecord -> SkCanvas (raster)
(2) PaintRecord -> PaintCanvas (playback, flatten ops)
(3) PaintRecord -> PaintCanvas (drawPicture, if paint canvas is recording, then
just record an indirection instead of flattening)

I think #1 should live on PaintRecord.  #3 should live on PaintCanvas.  This
case is #2, and I could see it going either way.

+  // possible when PaintRecord is a typedef.
+  virtual void PlaybackPaintRecord(PaintRecord* record) = 0;

[danakj, 2017/03/20 14:53:54]

Can it be an sk_sp<PaintRecord> like the drawPicture, that'll be less churn at
callers.

[enne (OOO), 2017/03/20 17:41:26]

Done done done.

+
  protected:
   friend class PaintSurface;
   friend class PaintRecorder;