GN: add skia_enable_gpu (default true)

GN: add skia_enable_gpu (default true)

This lets us replace the NoGPU bot.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2356853002

NOTREECHECKS=true

Committed: https://skia.googlesource.com/skia/+/06c35c0953b4884c18d1f2ce32c4c5045f86fa81

[mtklein_C, 2016-09-20 17:57:11]

Description was changed from

==========
GN: add skia_enable_gpu (default true)

This lets us replace the NoGPU bot.

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

to

==========
GN: add skia_enable_gpu (default true)

This lets us replace the NoGPU bot.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2356853002
==========

[mtklein_C, 2016-09-20 18:44:55]

mtklein@chromium.org changed reviewers:
+ brianosman@google.com

[Brian Osman, 2016-09-20 18:51:09]

lgtm

[mtklein_C, 2016-09-20 18:51:56]

Description was changed from

==========
GN: add skia_enable_gpu (default true)

This lets us replace the NoGPU bot.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2356853002
==========

to

==========
GN: add skia_enable_gpu (default true)

This lets us replace the NoGPU bot.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2356853002

NOTREECHECKS=true
==========

[mtklein_C, 2016-09-20 18:51:59]

The CQ bit was checked by mtklein@chromium.org

[commit-bot: I haz the power, 2016-09-20 18:52:11]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-20 19:28:15]

Description was changed from

==========
GN: add skia_enable_gpu (default true)

This lets us replace the NoGPU bot.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2356853002

NOTREECHECKS=true
==========

to

==========
GN: add skia_enable_gpu (default true)

This lets us replace the NoGPU bot.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2356853002

NOTREECHECKS=true

Committed:
https://skia.googlesource.com/skia/+/06c35c0953b4884c18d1f2ce32c4c5045f86fa81
==========

[commit-bot: I haz the power, 2016-09-20 19:28:16]

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

[fmalita_google_do_not_use, 2016-09-20 21:40:56]

On 2016/09/20 19:28:16, commit-bot: I haz the power wrote:
> Committed patchset #4 (id:60001) as
> https://skia.googlesource.com/skia/+/06c35c0953b4884c18d1f2ce32c4c5045f86fa81

Maybe related?

https://luci-milo.appspot.com/swarming/task/31624bbca8fa2e10/steps/dm/0/stdout

WARNING: can not create GPU context for config 'vk'. GM tests will be skipped.
Skipping config vk: Don't understand 'vk'.
+ >/data/local/tmp/rc 
+ echo 1
step returned non-zero exit code: 1

[mtklein, 2016-09-20 21:49:30]

On 2016/09/20 21:40:56, fmalita_google_do_not_use wrote:
> On 2016/09/20 19:28:16, commit-bot: I haz the power wrote:
> > Committed patchset #4 (id:60001) as
> >
https://skia.googlesource.com/skia/+/06c35c0953b4884c18d1f2ce32c4c5045f86fa81
> 
> Maybe related?
> 
> https://luci-milo.appspot.com/swarming/task/31624bbca8fa2e10/steps/dm/0/stdout
> 
> WARNING: can not create GPU context for config 'vk'. GM tests will be skipped.
> Skipping config vk: Don't understand 'vk'.
> + >/data/local/tmp/rc 
> + echo 1
> step returned non-zero exit code: 1

Yes, definitely.  It'll be fixed by https://codereview.chromium.org/2352923003/.
 If that doesn't land soon I'll revert and reland them together.