Set timeout before killing a process to 20 times the 30-second nag

Set timeout before killing a process to 30 times the 30-second nag
timer, i.e. ~900 seconds.

This should at least help make 'fetch chromium' slightly more reliable
over somewhat slow connections.

BUG=230691, 247173
R=rogerta@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=210190

[Jói, 2013-06-27 21:32:18]

Please see discussion of the new timeout in the bug I've linked to.

Cheers,
Jói

[Jói, 2013-06-27 21:37:19]

Changed to 30 times the nag limit, based on latest updates in the bug.

Cheers,
Jói

[Jói, 2013-07-01 13:17:14]

ping

[Jói, 2013-07-02 19:42:50]

szager -> maruel for review

[Jói, 2013-07-04 10:56:30]

ping

On Tue, Jul 2, 2013 at 7:42 PM,  <joi@chromium.org> wrote:
> szager -> maruel for review
>
> https://codereview.chromium.org/18150002/

[Jói, 2013-07-04 10:59:24]

maruel@ -> rogerta@ for review as maruel@ appears to be on vacation.

Cheers,
Jói

[Roger Tawa OOO till Jul 10th, 2013-07-04 14:10:26]

lgtm

In CL description, you say 30 times 30 seconds, or ~600 seconds.  Did you mean
900 seconds, or did you really want 600 seconds?

[Roger Tawa OOO till Jul 10th, 2013-07-04 14:11:56]

On 2013/07/04 14:10:26, Roger Tawa wrote:
> lgtm
> 
> In CL description, you say 30 times 30 seconds, or ~600 seconds.  Did you mean
> 900 seconds, or did you really want 600 seconds?

Oops, just noticed your comment above that the original 20 was changed to 30.  I
guess the CL description is out of date.

[Jói, 2013-07-04 14:28:59]

Thanks for catching that, updated the change description to say 900.

Cheers,
Jói


On Thu, Jul 4, 2013 at 2:11 PM,  <rogerta@chromium.org> wrote:
> On 2013/07/04 14:10:26, Roger Tawa wrote:
>>
>> lgtm
>
>
>> In CL description, you say 30 times 30 seconds, or ~600 seconds.  Did you
>> mean
>> 900 seconds, or did you really want 600 seconds?
>
>
> Oops, just noticed your comment above that the original 20 was changed to
> 30.  I
> guess the CL description is out of date.
>
> https://codereview.chromium.org/18150002/

[Jói, 2013-07-04 14:42:41]

Committed patchset #2 manually as r210190.

[iannucci, 2013-07-04 20:29:12]

On 2013/07/04 14:42:41, Jói wrote:
> Committed patchset #2 manually as r210190.

Skipped the pre commit tests, eh? *wags finger*

Tests fixed in https://codereview.chromium.org/18603010/ :)

[Jói, 2013-07-04 20:33:04]

> Skipped the pre commit tests, eh? *wags finger*

Sheepishly sorry :(

> Tests fixed in https://codereview.chromium.org/18603010/ :)

See further apologies on the review thread :)

Cheers,
Jói


On Thu, Jul 4, 2013 at 8:29 PM,  <iannucci@chromium.org> wrote:
> On 2013/07/04 14:42:41, Jói wrote:
>>
>> Committed patchset #2 manually as r210190.
>
>
> Skipped the pre commit tests, eh? *wags finger*
>
> Tests fixed in https://codereview.chromium.org/18603010/ :)
>
> https://codereview.chromium.org/18150002/

[Jói, 2013-07-04 20:33:22]

Also, thanks for cleaning up my mess.

On Thu, Jul 4, 2013 at 8:32 PM, Jói Sigurðsson <joi@chromium.org> wrote:
>> Skipped the pre commit tests, eh? *wags finger*
>
> Sheepishly sorry :(
>
>> Tests fixed in https://codereview.chromium.org/18603010/ :)
>
> See further apologies on the review thread :)
>
> Cheers,
> Jói
>
>
> On Thu, Jul 4, 2013 at 8:29 PM,  <iannucci@chromium.org> wrote:
>> On 2013/07/04 14:42:41, Jói wrote:
>>>
>>> Committed patchset #2 manually as r210190.
>>
>>
>> Skipped the pre commit tests, eh? *wags finger*
>>
>> Tests fixed in https://codereview.chromium.org/18603010/ :)
>>
>> https://codereview.chromium.org/18150002/