add picture-image variant

add picture-image variant

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/7628967820994c383f2f01d1c64440488d5d8308

[reed1, 2015-09-09 18:11:37]

reed@google.com changed reviewers:
+ bsalomon@google.com, robertphillips@google.com

[reed1, 2015-09-09 18:11:37]

alt version to exercise generator images (using picture in this case). This is
meant to confirm that when I change generator-images, we still handle the subset
case the same way.

[reed1, 2015-09-09 18:11:41]

The CQ bit was checked by reed@google.com to run a CQ dry run

[commit-bot: I haz the power, 2015-09-09 18:11:50]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1329283002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1329283002/1

[commit-bot: I haz the power, 2015-09-09 18:17:03]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-09-09 18:17:03]

Dry run: This issue passed the CQ dry run.

[robertphillips, 2015-09-09 18:21:05]

lgtm + question

https://codereview.chromium.org/1329283002/diff/1/gm/verylargebitmap.cpp
File gm/verylargebitmap.cpp (right):

https://codereview.chromium.org/1329283002/diff/1/gm/verylargebitmap.cpp#newc...
gm/verylargebitmap.cpp:122: DEF_GM( return new
VeryLargeBitmapGM(make_raster_image, "bitmap"); )
Wouldn't we normally name this "verylarge_picture_image" ?

[reed1, 2015-09-09 18:22:15]

On 2015/09/09 18:21:05, robertphillips wrote:
> lgtm + question
> 
> https://codereview.chromium.org/1329283002/diff/1/gm/verylargebitmap.cpp
> File gm/verylargebitmap.cpp (right):
> 
>
https://codereview.chromium.org/1329283002/diff/1/gm/verylargebitmap.cpp#newc...
> gm/verylargebitmap.cpp:122: DEF_GM( return new
> VeryLargeBitmapGM(make_raster_image, "bitmap"); )
> Wouldn't we normally name this "verylarge_picture_image" ?

Sure, I can rename the new one.

As soon as Joe/Stephan lets us "rename" existing digests, I'll rename the bitmap
one too.

[reed1, 2015-09-09 18:23:22]

The CQ bit was checked by reed@google.com

[reed1, 2015-09-09 18:23:22]

The patchset sent to the CQ was uploaded after l-g-t-m from
robertphillips@google.com
Link to the patchset: https://codereview.chromium.org/1329283002/#ps20001
(title: "rename new gm")

[commit-bot: I haz the power, 2015-09-09 18:23:33]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1329283002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1329283002/20001

[commit-bot: I haz the power, 2015-09-09 18:29:12]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/7628967820994c383f2f01d1c64440488d5d8308