🐎 Remove --shard-timeout from webrtc recipes now that they are in wrapper scripts

Remove --shard-timeout from webrtc recipes now that they are in wrapper scripts

BUG=599919
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=300083

[agrieve, 2016-04-13 00:22:37]

Description was changed from

==========
Remove --shard-timeout from webrtc recipes now that they are in wrapper scripts

BUG=599919
==========

to

==========
Remove --shard-timeout from webrtc recipes now that they are in wrapper scripts

BUG=599919
==========

[agrieve, 2016-04-13 00:22:37]

agrieve@chromium.org changed reviewers:
+ kjellander@chromium.org

[agrieve, 2016-04-13 01:27:05]

On 2016/04/13 00:22:37, agrieve wrote:
> mailto:agrieve@chromium.org changed reviewers:
> + mailto:kjellander@chromium.org

the wrapper CL was reverted, so we'll have to wait on this until it sticks for a
bit longer.

[kjellander_chromium, 2016-04-13 05:27:07]

lgtm.
OK we'll wait for the other CL to reland then.

[kjellander_chromium, 2016-04-18 08:30:02]

On 2016/04/13 05:27:07, kjellander (chromium) wrote:
> lgtm.
> OK we'll wait for the other CL to reland then.

Can this land now or do we need to wait for more?

[agrieve, 2016-04-18 16:50:14]

On 2016/04/18 08:30:02, kjellander (chromium) wrote:
> On 2016/04/13 05:27:07, kjellander (chromium) wrote:
> > lgtm.
> > OK we'll wait for the other CL to reland then.
> 
> Can this land now or do we need to wait for more?

We still need to wait. Upstream still not using wrapper scripts for gtests :(

[kjellander_chromium, 2016-04-19 20:44:53]

On 2016/04/18 16:50:14, agrieve wrote:
> On 2016/04/18 08:30:02, kjellander (chromium) wrote:
> > On 2016/04/13 05:27:07, kjellander (chromium) wrote:
> > > lgtm.
> > > OK we'll wait for the other CL to reland then.
> > 
> > Can this land now or do we need to wait for more?
> 
> We still need to wait. Upstream still not using wrapper scripts for gtests :(

It seems https://codereview.chromium.org/1897823006/ is sticking? Then you'll
land this, right?
Or do you want to wait a few more hours?

[agrieve, 2016-04-21 16:45:30]

The CQ bit was checked by agrieve@chromium.org

[commit-bot: I haz the power, 2016-04-21 16:45:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1882293003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1882293003/1

[agrieve, 2016-04-21 16:45:56]

On 2016/04/19 20:44:53, kjellander (chromium) wrote:
> On 2016/04/18 16:50:14, agrieve wrote:
> > On 2016/04/18 08:30:02, kjellander (chromium) wrote:
> > > On 2016/04/13 05:27:07, kjellander (chromium) wrote:
> > > > lgtm.
> > > > OK we'll wait for the other CL to reland then.
> > > 
> > > Can this land now or do we need to wait for more?
> > 
> > We still need to wait. Upstream still not using wrapper scripts for gtests
:(
> 
> It seems https://codereview.chromium.org/1897823006/ is sticking? Then you'll
> land this, right?
> Or do you want to wait a few more hours?

I think it's finally stuck! Submitting this now.

[commit-bot: I haz the power, 2016-04-21 16:49:15]

Description was changed from

==========
Remove --shard-timeout from webrtc recipes now that they are in wrapper scripts

BUG=599919
==========

to

==========
Remove --shard-timeout from webrtc recipes now that they are in wrapper scripts

BUG=599919

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=300083
==========

[commit-bot: I haz the power, 2016-04-21 16:49:16]

Committed patchset #1 (id:1) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=300083

[kjellander_chromium, 2016-04-22 12:11:06]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1913103002/ by kjellander@chromium.org.

The reason for reverting is: It seems the run_* scripts in WebRTC actually don't
include the timeouts yet, I will investigate before relanding this..