Turn ContextInfos returned by GrContextFactory into structs.

Turn ContextInfos returned by GrContextFactory into structs.
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1966013002

Committed: https://skia.googlesource.com/skia/+/8b7451aaf6b1c71e9d343a4df107893db277b6aa

[bsalomon, 2016-05-10 21:41:00]

Description was changed from

==========
Turn ContextInfos returned by GrContextFactory into structs.
==========

to

==========
Turn ContextInfos returned by GrContextFactory into structs.
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[bsalomon, 2016-05-10 21:42:50]

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

[bsalomon, 2016-05-10 21:42:51]

The only interesting part of this change is in GrContextFactory.h. The rest is
just replacing fGrContext with grContext() and fGLContext with glContext().

This is to prepare for having a VkTestContext class for Vulkan.

[bsalomon, 2016-05-10 21:42:56]

The CQ bit was checked by bsalomon@google.com

[commit-bot: I haz the power, 2016-05-10 21:43:02]

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

[commit-bot: I haz the power, 2016-05-10 21:43:03]

Note for Reviewers:
The CQ is waiting for an approval.
If you believe that the CL is not ready yet, or if you would like to L-G-T-M
with comments then please uncheck the CQ checkbox.

Waiting for LGTM from valid reviewer(s) till 2016-05-11 03:43 UTC

[commit-bot: I haz the power, 2016-05-10 21:45:52]

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

[commit-bot: I haz the power, 2016-05-10 21:45:54]

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...)

[egdaniel, 2016-05-11 00:37:56]

looks like you may have missed a tool or two the used the Context

https://codereview.chromium.org/1966013002/diff/20001/dm/DMSrcSink.cpp
File dm/DMSrcSink.cpp (right):

https://codereview.chromium.org/1966013002/diff/20001/dm/DMSrcSink.cpp#newcod...
dm/DMSrcSink.cpp:1090: factory.getContextInfo(fContextType,
fContextOptions).grContext()->caps()->maxTextureSize();
Maybe split this into two calls to fix it all?

[bsalomon, 2016-05-11 02:14:38]

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

[commit-bot: I haz the power, 2016-05-11 02:14:46]

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

[commit-bot: I haz the power, 2016-05-11 02:34:52]

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

[commit-bot: I haz the power, 2016-05-11 02:34:53]

Dry run: This issue passed the CQ dry run.

[bsalomon, 2016-05-11 13:00:17]

https://codereview.chromium.org/1966013002/diff/20001/dm/DMSrcSink.cpp
File dm/DMSrcSink.cpp (right):

https://codereview.chromium.org/1966013002/diff/20001/dm/DMSrcSink.cpp#newcod...
dm/DMSrcSink.cpp:1090: factory.getContextInfo(fContextType,
fContextOptions).grContext()->caps()->maxTextureSize();
On 2016/05/11 00:37:55, egdaniel wrote:
> Maybe split this into two calls to fix it all?

Done.

[egdaniel, 2016-05-11 13:05:07]

lgtm

[bsalomon, 2016-05-11 13:09:15]

The CQ bit was checked by bsalomon@google.com

[commit-bot: I haz the power, 2016-05-11 13:09:24]

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

[commit-bot: I haz the power, 2016-05-11 13:33:08]

Description was changed from

==========
Turn ContextInfos returned by GrContextFactory into structs.
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Turn ContextInfos returned by GrContextFactory into structs.
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/8b7451aaf6b1c71e9d343a4df107893db277b6aa
==========

[commit-bot: I haz the power, 2016-05-11 13:33:09]

Committed patchset #4 (id:60001) as
https://skia.googlesource.com/skia/+/8b7451aaf6b1c71e9d343a4df107893db277b6aa