Enable basic drawing with SkRecord-based pictures.

Enable basic drawing with SkRecord-based pictures.

I've tagged all the functions in SkPicture.cpp is // fRecord TODO or // fRecord
OK, depending on whether or not they're totally broken when used from an
SkRecord-based picture.  Obviously next steps are to eliminate all the TODOs,
then clean up the notes.

I converted SkPicture over to smart pointers too.  It's particularly helpful
that the smart pointers initialize to NULL by default.

For now I've got all the SkRecord-based code jammed in at the bottom of the file.  I figure it'll help me keep things straight for a bit, then we can rearrange later.

BUG=skia:
R=robertphillips@google.com

Committed: https://skia.googlesource.com/skia/+/744fb73

[robertphillips, 2014-06-23 18:53:19]

lgtm + nits

https://codereview.chromium.org/333823007/diff/20001/src/core/SkPicture.cpp
File src/core/SkPicture.cpp (right):

https://codereview.chromium.org/333823007/diff/20001/src/core/SkPicture.cpp#n...
src/core/SkPicture.cpp:299: void SkPicture::draw(SkCanvas* canvas,
SkDrawPictureCallback* callback) const {
NULL != canvas ?

https://codereview.chromium.org/333823007/diff/20001/src/core/SkPicture.cpp#n...
src/core/SkPicture.cpp:539: SkPicture::SkPicture(int width, int height,
SkRecord* record)
on own lines ?

[mtklein, 2014-06-23 18:55:45]

https://codereview.chromium.org/333823007/diff/20001/src/core/SkPicture.cpp
File src/core/SkPicture.cpp (right):

https://codereview.chromium.org/333823007/diff/20001/src/core/SkPicture.cpp#n...
src/core/SkPicture.cpp:299: void SkPicture::draw(SkCanvas* canvas,
SkDrawPictureCallback* callback) const {
On 2014/06/23 18:53:19, robertphillips wrote:
> NULL != canvas ?

Done.

https://codereview.chromium.org/333823007/diff/20001/src/core/SkPicture.cpp#n...
src/core/SkPicture.cpp:539: SkPicture::SkPicture(int width, int height,
SkRecord* record)
On 2014/06/23 18:53:19, robertphillips wrote:
> on own lines ?

Done.

[mtklein_C, 2014-06-23 19:13:31]

Committed patchset #4 manually as r744fb73.