One signature for creating unit tests that run on premade GrContexts

One signature for creating unit tests that run on premade GrContexts
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1860593002

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

[bsalomon, 2016-04-04 17:26:06]

Description was changed from

==========
One signature for creating unit tests that run on premade GrContexts
==========

to

==========
One signature for creating unit tests that run on premade GrContexts
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[bsalomon, 2016-04-04 19:04:41]

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

[bsalomon, 2016-04-04 19:04:42]

Currently the macros for writing tests that take premade GrConexts can take
either just a GrContext* or a GrContext* and a GLTestContext*.

This change makes all the macros take a ContextInfo which has both the
GrContext* and GLTestContext*. I think this will make it easier to extend unit
tests for Vulkan-based contexts. It also allows deletion of complicated var_args
and templates in the test running code.

Most of the files are *trivial* changes to the unit tests themselves. The
interesting files are:

Test.h
TestTest.cpp
GrContextFactory.h
DM.cpp

I pulled the ContextInfo struct out of GrContextFactory just so that it could be
forward declared in the non-gpu build.

[jvanverth1, 2016-04-04 20:38:00]

lgtm

We should probably talk about the structure of VulkanTestContext vs
GLTestContext. Right now I have the GrContext inside VulkanTestContext so I can
wrap the VkImages in a GrRenderTarget. That's not a requirement, though.

[jvanverth1, 2016-04-04 20:38:56]

On 2016/04/04 20:38:00, jvanverth1 wrote:
> lgtm
> 
> We should probably talk about the structure of VulkanTestContext vs
> GLTestContext. Right now I have the GrContext inside VulkanTestContext so I
can
> wrap the VkImages in a GrRenderTarget. That's not a requirement, though.

Correction: I return an SkSurface to the client, so each VkImage is wrapped in
one of those.

[bsalomon, 2016-04-05 18:07:39]

The CQ bit was checked by bsalomon@google.com

[bsalomon, 2016-04-05 18:07:40]

The patchset sent to the CQ was uploaded after l-g-t-m from jvanverth@google.com
Link to the patchset: https://codereview.chromium.org/1860593002/#ps30001
(title: "rebase")

[commit-bot: I haz the power, 2016-04-05 18:07:50]

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

[commit-bot: I haz the power, 2016-04-05 18:11:20]

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

[commit-bot: I haz the power, 2016-04-05 18:11:21]

Try jobs failed on following builders:
  Build-Win-MSVC-x86_64-Debug-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Win-MSVC-x86_6...)

[bsalomon, 2016-04-05 18:56:55]

The CQ bit was checked by bsalomon@google.com

[bsalomon, 2016-04-05 18:56:56]

The patchset sent to the CQ was uploaded after l-g-t-m from jvanverth@google.com
Link to the patchset: https://codereview.chromium.org/1860593002/#ps50001
(title: "fix windows?
")

[commit-bot: I haz the power, 2016-04-05 18:57:08]

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

[commit-bot: I haz the power, 2016-04-05 19:00:18]

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

[commit-bot: I haz the power, 2016-04-05 19:00:19]

Try jobs failed on following builders:
  Build-Win-MSVC-x86-Debug-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Win-MSVC-x86-D...)

[bsalomon, 2016-04-05 19:47:20]

The CQ bit was checked by bsalomon@google.com

[bsalomon, 2016-04-05 19:47:21]

The patchset sent to the CQ was uploaded after l-g-t-m from jvanverth@google.com
Link to the patchset: https://codereview.chromium.org/1860593002/#ps70001
(title: "another guess to fix windows")

[commit-bot: I haz the power, 2016-04-05 19:47:26]

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

[commit-bot: I haz the power, 2016-04-05 19:59:10]

Description was changed from

==========
One signature for creating unit tests that run on premade GrContexts
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
One signature for creating unit tests that run on premade GrContexts
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

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

[commit-bot: I haz the power, 2016-04-05 19:59:12]

Committed patchset #5 (id:70001) as
https://skia.googlesource.com/skia/+/f2f1c17e331fe1e0ce695969970ecebc81e12ceb