Roll GN: gn format is --in-place by default

Roll GN: gn format is --in-place by default

As usual, might as well roll, and this new behavior is a small convenience.

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

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

[mtklein_C, 2016-08-10 13:42:24]

Description was changed from

==========
Roll GN: gn format is --in-place by default

As usual, might as well roll, and this new behavior is a small convenience.

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

to

==========
Roll GN: gn format is --in-place by default

As usual, might as well roll, and this new behavior is a small convenience.

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

[mtklein_C, 2016-08-10 13:42:48]

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

[mtklein_C, 2016-08-10 13:42:48]

The CQ bit was checked by mtklein@chromium.org

[commit-bot: I haz the power, 2016-08-10 13:42:51]

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

[jcgregorio, 2016-08-10 13:43:26]

lgtm

[commit-bot: I haz the power, 2016-08-10 13:46:00]

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

[commit-bot: I haz the power, 2016-08-10 13:46:01]

Try jobs failed on following builders:
  Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Debug-GN-Trybot on master.client.skia
(JOB_FAILED,
http://build.chromium.org/p/client.skia/builders/Test-Ubuntu-GCC-GCE-CPU-AVX2...)
  Build-Ubuntu-Clang-x86_64-Debug-GN-Trybot on master.client.skia.compile
(JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-Clang-x...)

[mtklein_C, 2016-08-10 13:58:36]

Seems like the pool option on linking tools has switched from optional to
seems-optional-really-required.  I've added a pool that replicates the old
default unlimited link pool.

(The link pool is a throttling mechanism meant to keep bots from running out of
memory when linking too many large targets in Chrome at once.  We don't have
that problem, so unlimited is just fine.)

This gave me a good opportunity to re-test the presubmit.  Still works.

[mtklein_C, 2016-08-10 13:58:43]

The CQ bit was checked by mtklein@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-10 13:58:55]

Dry run: 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-08-10 14:27:06]

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

[commit-bot: I haz the power, 2016-08-10 14:27:06]

Dry run: This issue passed the CQ dry run.

[mtklein, 2016-08-10 14:29:57]

The CQ bit was checked by mtklein@google.com

[mtklein, 2016-08-10 14:29:58]

The patchset sent to the CQ was uploaded after l-g-t-m from
jcgregorio@google.com
Link to the patchset: https://codereview.chromium.org/2231833002/#ps20001
(title: "add a link pool")

[commit-bot: I haz the power, 2016-08-10 14:30:07]

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-08-10 14:31:01]

Description was changed from

==========
Roll GN: gn format is --in-place by default

As usual, might as well roll, and this new behavior is a small convenience.

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

to

==========
Roll GN: gn format is --in-place by default

As usual, might as well roll, and this new behavior is a small convenience.

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

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

[commit-bot: I haz the power, 2016-08-10 14:31:01]

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