Make more unit tests run on Vulkan

Make more unit tests run on Vulkan
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1885623002

Committed: https://skia.googlesource.com/skia/+/68d9134bec16e91c4a6cde071bcaa579bc0801a7

[bsalomon, 2016-04-12 16:40:23]

Description was changed from

==========
Make more unit tests run on Vulkan
==========

to

==========
Make more unit tests run on Vulkan
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[bsalomon, 2016-04-12 16:42:34]

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

[bsalomon, 2016-04-12 16:42:34]

These are the unit tests that ran on all GL rendering contexts but pass/don't
crash when run on Vulkan.

[bsalomon, 2016-04-12 16:42:40]

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

[commit-bot: I haz the power, 2016-04-12 16:42:50]

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

[jvanverth1, 2016-04-12 16:49:40]

lgtm + comment

https://codereview.chromium.org/1885623002/diff/1/tests/Test.h
File tests/Test.h (right):

https://codereview.chromium.org/1885623002/diff/1/tests/Test.h#newcode174
tests/Test.h:174: DEF_GPUTEST_FOR_CONTEXTS(name,
sk_gpu_test::GrContextFactory::IsRenderingContext,   \
Why is this sk_gpu_test while the others are skiatest?

[commit-bot: I haz the power, 2016-04-12 16:52:20]

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

[commit-bot: I haz the power, 2016-04-12 16:52:21]

Dry run: This issue passed the CQ dry run.

[bsalomon, 2016-04-12 16:58:34]

https://codereview.chromium.org/1885623002/diff/1/tests/Test.h
File tests/Test.h (right):

https://codereview.chromium.org/1885623002/diff/1/tests/Test.h#newcode174
tests/Test.h:174: DEF_GPUTEST_FOR_CONTEXTS(name,
sk_gpu_test::GrContextFactory::IsRenderingContext,   \
On 2016/04/12 16:49:40, jvanverth1 wrote:
> Why is this sk_gpu_test while the others are skiatest?

The other predicates are declared above in this function. We already happened to
have a static function in GrContextFactory that was the predicate I needed.
GrContextFactory is in the sk_gpu_test namespace because it is in the skgputest
target.

[bsalomon, 2016-04-12 16:58:38]

The CQ bit was checked by bsalomon@google.com

[commit-bot: I haz the power, 2016-04-12 16:58:42]

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

[commit-bot: I haz the power, 2016-04-12 17:00:00]

Description was changed from

==========
Make more unit tests run on Vulkan
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Make more unit tests run on Vulkan
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/68d9134bec16e91c4a6cde071bcaa579bc0801a7
==========

[commit-bot: I haz the power, 2016-04-12 17:00:01]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/68d9134bec16e91c4a6cde071bcaa579bc0801a7