More cleanup in the Vulkan viewer

More cleanup in the Vulkan viewer

* add animation support for GMs
* don't load all SkPictures at start, only as needed
* don't update constantly, only for animation or stats
* hide/show stats
* scale slides up/down

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1873733003

Committed: https://skia.googlesource.com/skia/+/c265a92741a8deee8adb62bb20c756b46672a11c

[jvanverth1, 2016-04-08 18:22:14]

Description was changed from

==========
More cleanup in the Vulkan viewer

BUG=skia:
==========

to

==========
More cleanup in the Vulkan viewer

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[jvanverth1, 2016-04-08 18:25:14]

Description was changed from

==========
More cleanup in the Vulkan viewer

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
More cleanup in the Vulkan viewer

* add animation support for GMs
* don't load all SkPictures at start, only as needed
* don't update constantly, only for animation or stats
* hide/show stats
* scale slides up/down

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[jvanverth1, 2016-04-08 18:25:31]

jvanverth@google.com changed reviewers:
+ djsollen@google.com, egdaniel@google.com

[egdaniel, 2016-04-08 19:01:50]

lgtm plus comment

https://codereview.chromium.org/1873733003/diff/1/tools/vulkan/viewer/VulkanV...
File tools/vulkan/viewer/VulkanViewer.cpp (right):

https://codereview.chromium.org/1873733003/diff/1/tools/vulkan/viewer/VulkanV...
tools/vulkan/viewer/VulkanViewer.cpp:188: //m.postTranslate(fOffset.fX,
fOffset.fY);
maybe add a comment about why these are all commented out

[jvanverth1, 2016-04-08 19:39:12]

https://codereview.chromium.org/1873733003/diff/1/tools/vulkan/viewer/VulkanV...
File tools/vulkan/viewer/VulkanViewer.cpp (right):

https://codereview.chromium.org/1873733003/diff/1/tools/vulkan/viewer/VulkanV...
tools/vulkan/viewer/VulkanViewer.cpp:188: //m.postTranslate(fOffset.fX,
fOffset.fY);
On 2016/04/08 19:01:50, egdaniel wrote:
> maybe add a comment about why these are all commented out

Done.

[jvanverth1, 2016-04-08 19:39:18]

The CQ bit was checked by jvanverth@google.com

[jvanverth1, 2016-04-08 19:39:19]

The patchset sent to the CQ was uploaded after l-g-t-m from egdaniel@google.com
Link to the patchset: https://codereview.chromium.org/1873733003/#ps20001
(title: "Address comments")

[commit-bot: I haz the power, 2016-04-08 19:39:29]

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

[commit-bot: I haz the power, 2016-04-08 19:51:48]

Description was changed from

==========
More cleanup in the Vulkan viewer

* add animation support for GMs
* don't load all SkPictures at start, only as needed
* don't update constantly, only for animation or stats
* hide/show stats
* scale slides up/down

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
More cleanup in the Vulkan viewer

* add animation support for GMs
* don't load all SkPictures at start, only as needed
* don't update constantly, only for animation or stats
* hide/show stats
* scale slides up/down

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/c265a92741a8deee8adb62bb20c756b46672a11c
==========

[commit-bot: I haz the power, 2016-04-08 19:51:53]

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