Support serialization in SkRecord-backed SkPictures.

Support serialization in SkRecord-backed SkPictures.

Update DM to test SkRecord through SkPictureRecorder API.

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/7373456679b809d4e7649a438f0fe5b59257e4e5

[robertphillips, 2014-06-24 18:34:46]

lgtm + nits/suggestions

https://codereview.chromium.org/345553003/diff/20001/dm/DMReplayTask.cpp
File dm/DMReplayTask.cpp (right):

https://codereview.chromium.org/345553003/diff/20001/dm/DMReplayTask.cpp#newc...
dm/DMReplayTask.cpp:49: switch (fMode) {
if (!*) {
   return true;
}
break;

?

https://codereview.chromium.org/345553003/diff/20001/dm/DMSKPTask.cpp
File dm/DMSKPTask.cpp (right):

https://codereview.chromium.org/345553003/diff/20001/dm/DMSKPTask.cpp#newcode15
dm/DMSKPTask.cpp:15: public:
const SkBitmap& reference ?

https://codereview.chromium.org/345553003/diff/20001/dm/DMSerializeTask.cpp
File dm/DMSerializeTask.cpp (right):

https://codereview.chromium.org/345553003/diff/20001/dm/DMSerializeTask.cpp#n...
dm/DMSerializeTask.cpp:49: switch (fMode) {
formatting ?

[mtklein, 2014-06-24 19:17:03]

https://codereview.chromium.org/345553003/diff/20001/dm/DMReplayTask.cpp
File dm/DMReplayTask.cpp (right):

https://codereview.chromium.org/345553003/diff/20001/dm/DMReplayTask.cpp#newc...
dm/DMReplayTask.cpp:49: switch (fMode) {
On 2014/06/24 18:34:46, robertphillips wrote:
> if (!*) {
>    return true;
> }
> break;
> 
> ?

Got rid of the switch entirely.

https://codereview.chromium.org/345553003/diff/20001/dm/DMSKPTask.cpp
File dm/DMSKPTask.cpp (right):

https://codereview.chromium.org/345553003/diff/20001/dm/DMSKPTask.cpp#newcode15
dm/DMSKPTask.cpp:15: public:
On 2014/06/24 18:34:46, robertphillips wrote:
> const SkBitmap& reference ?

This one's tricky.  I've always found it more helpful to see that we're
certainly going to copy the SkBitmap as part of the constructor API.  We'll copy
it exactly once either way, but this way the creator of SkrComparisonTask knows
this too, so they don't have to feel at all unsure about whether or not they
have to keep the SkBitmap alive for SkrComparisonTask.

The rest of DM works with SkBitmaps by value, so I'm going to leave it for
consistency.  I could be convinced to convert it so the parameters pass by
const& if it's really important to you.

https://codereview.chromium.org/345553003/diff/20001/dm/DMSerializeTask.cpp
File dm/DMSerializeTask.cpp (right):

https://codereview.chromium.org/345553003/diff/20001/dm/DMSerializeTask.cpp#n...
dm/DMSerializeTask.cpp:49: switch (fMode) {
On 2014/06/24 18:34:46, robertphillips wrote:
> formatting ?

Ditto.

[mtklein, 2014-06-24 19:17:10]

The CQ bit was checked by mtklein@google.com

[commit-bot: I haz the power, 2014-06-24 19:17:51]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/mtklein@chromium.org/345553003/30001

[commit-bot: I haz the power, 2014-06-24 19:28:38]

Change committed as 7373456679b809d4e7649a438f0fe5b59257e4e5