[iOS/GN] Add a variable to configure the depth of "bundle_pool".

[iOS/GN] Add a variable to configure the depth of "bundle_pool".

Chrome on iOS downstream developer wants to be able to control the
size of the "bundle_pool" independently of the number of concurrent
link targets.

BUG=612786
Committed: https://crrev.com/f42145ad884229d1bd0f1b9012d5afeedb180f68
Cr-Commit-Position: refs/heads/master@{#402166}

[sdefresne, 2016-06-20 12:50:09]

sdefresne@chromium.org changed reviewers:
+ dpranke@chromium.org, justincohen@chromium.org

[sdefresne, 2016-06-20 12:50:10]

Please take a look.

[justincohen, 2016-06-20 12:50:47]

lgtm, thanks!

[Dirk Pranke, 2016-06-20 16:32:32]

What do you need to change the value to? Can we just adjust the default instead?

[justincohen, 2016-06-21 07:25:52]

sdefresne@ and I discussed using 64.

[sdefresne, 2016-06-21 07:35:28]

On 2016/06/21 07:25:52, justincohen wrote:
> sdefresne@ and I discussed using 64.

The default is currently 16 which makes a gn build slower than the same build
with gyp. A value of 64 makes the gn build faster for developers that have fast
machines with SSD (higher value are not beneficial, and build time starts to
degrade if value starts going above 100). However, as you mentioned during the
original review, bots are not as powerful as developer machines and for them,
the current default is probably fine.

[Dirk Pranke, 2016-06-21 23:45:51]

okay. It'd be nice if we can figure out a way to be smarter here, but it's not
too important.

[sdefresne, 2016-06-22 07:30:29]

On 2016/06/21 23:45:51, Dirk Pranke wrote:
> okay. It'd be nice if we can figure out a way to be smarter here, but it's not
> too important.

Ack, does this mean the CL LGTY or should I make any change/close it?

[Dirk Pranke, 2016-06-22 15:14:26]

Oh, sorry, I thought I had approved this already.

lgtm.

[sdefresne, 2016-06-27 12:58:31]

The CQ bit was checked by sdefresne@chromium.org

[sdefresne, 2016-06-27 12:58:32]

The patchset sent to the CQ was uploaded after l-g-t-m from
justincohen@chromium.org, dpranke@chromium.org
Link to the patchset: https://codereview.chromium.org/2085463002/#ps60001
(title: "Rebase")

[commit-bot: I haz the power, 2016-06-27 12:58:45]

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-06-27 14:05:42]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-06-27 14:07:34]

Description was changed from

==========
[iOS/GN] Add a variable to configure the depth of "bundle_pool".

Chrome on iOS downstream developer wants to be able to control the
size of the "bundle_pool" independently of the number of concurrent
link targets.

BUG=612786
==========

to

==========
[iOS/GN] Add a variable to configure the depth of "bundle_pool".

Chrome on iOS downstream developer wants to be able to control the
size of the "bundle_pool" independently of the number of concurrent
link targets.

BUG=612786

Committed: https://crrev.com/f42145ad884229d1bd0f1b9012d5afeedb180f68
Cr-Commit-Position: refs/heads/master@{#402166}
==========

[commit-bot: I haz the power, 2016-06-27 14:07:35]

Patchset 4 (id:??) landed as
https://crrev.com/f42145ad884229d1bd0f1b9012d5afeedb180f68
Cr-Commit-Position: refs/heads/master@{#402166}