remote_run_factory: keep timeout defaults in sync with annotator_factory

remote_run_factory: keep timeout defaults in sync with annotator_factory

Otherwise unrelated builds e.g. on chromium.fyi start to time out,
and we wouldn't be able to seamlessly migrate builders.

BUG=584079, 459840, 593999
Committed: https://chromium.googlesource.com/chromium/tools/build/+/a44ca7067bf9c87e1877b1c667c5560d2dd7aa23

[Paweł Hajdan Jr., 2016-06-27 13:27:07]

phajdan.jr@chromium.org changed reviewers:
+ emso@chromium.org, sergiyb@chromium.org, tandrii@chromium.org

[Sergiy Byelozyorov, 2016-06-27 13:30:30]

lgtm

why remove max_time?

[Paweł Hajdan Jr., 2016-06-27 13:32:09]

On 2016/06/27 at 13:30:30, sergiyb wrote:
> lgtm
> 
> why remove max_time?

That's the default in annotator_factory, and diverging from that was breaking
things, e.g.
https://build.chromium.org/p/chromium.fyi/builders/ClangToTAndroidASan%20test...
.

[Paweł Hajdan Jr., 2016-06-27 13:32:12]

The CQ bit was checked by phajdan.jr@chromium.org

[commit-bot: I haz the power, 2016-06-27 13:32:20]

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 13:36:15]

Description was changed from

==========
remote_run_factory: keep timeout defaults in sync with annotator_factory

Otherwise unrelated builds e.g. on chromium.fyi start to time out,
and we wouldn't be able to seamlessly migrate builders.

BUG=584079, 459840, 593999
==========

to

==========
remote_run_factory: keep timeout defaults in sync with annotator_factory

Otherwise unrelated builds e.g. on chromium.fyi start to time out,
and we wouldn't be able to seamlessly migrate builders.

BUG=584079, 459840, 593999

Committed:
https://chromium.googlesource.com/chromium/tools/build/+/a44ca7067bf9c87e1877...
==========

[commit-bot: I haz the power, 2016-06-27 13:36:17]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/tools/build/+/a44ca7067bf9c87e1877...