webrtc: drop dependency on chromium_tests; extract chromium_swarming

webrtc: drop dependency on chromium_tests; extract chromium_swarming

This will make it easier to move chromium_tests out of build repo.

BUG=584079
Committed: https://chromium.googlesource.com/chromium/tools/build/+/8ce4d7f4b246ac1b3487675d93e65f6ad6cc61d4

[Paweł Hajdan Jr., 2016-07-13 10:34:40]

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

[Paweł Hajdan Jr., 2016-07-13 10:34:45]

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

[commit-bot: I haz the power, 2016-07-13 10:34:51]

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-07-13 10:39:01]

The CQ bit was unchecked by commit-bot@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-13 10:39:02]

Dry run: This issue passed the CQ dry run.

[tandrii(chromium), 2016-07-13 11:27:03]

LGTM % comment, though ideally for another CL

https://codereview.chromium.org/2147913002/diff/1/scripts/slave/recipe_module...
File scripts/slave/recipe_modules/chromium_swarming/api.py (right):

https://codereview.chromium.org/2147913002/diff/1/scripts/slave/recipe_module...
scripts/slave/recipe_modules/chromium_swarming/api.py:56: if requester ==
'commit-bot@chromium.org':
can this instead be a api.properties.category 'cq' or cq experimental?

[Paweł Hajdan Jr., 2016-07-13 11:28:03]

https://codereview.chromium.org/2147913002/diff/1/scripts/slave/recipe_module...
File scripts/slave/recipe_modules/chromium_swarming/api.py (right):

https://codereview.chromium.org/2147913002/diff/1/scripts/slave/recipe_module...
scripts/slave/recipe_modules/chromium_swarming/api.py:56: if requester ==
'commit-bot@chromium.org':
On 2016/07/13 at 11:27:03, tandrii(chromium) wrote:
> can this instead be a api.properties.category 'cq' or cq experimental?

This is preexisting code just being moved.

I'm not sure what the change you're suggesting would mean.

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

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

[commit-bot: I haz the power, 2016-07-13 11:28:14]

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

[tandrii(chromium), 2016-07-13 11:30:57]

https://codereview.chromium.org/2147913002/diff/1/scripts/slave/recipe_module...
File scripts/slave/recipe_modules/chromium_swarming/api.py (right):

https://codereview.chromium.org/2147913002/diff/1/scripts/slave/recipe_module...
scripts/slave/recipe_modules/chromium_swarming/api.py:56: if requester ==
'commit-bot@chromium.org':
On 2016/07/13 11:28:03, Paweł Hajdan Jr. wrote:
> On 2016/07/13 at 11:27:03, tandrii(chromium) wrote:
> > can this instead be a api.properties.category 'cq' or cq experimental?
> 
> This is preexisting code just being moved.
Yeahk i know, that's why I'm asking about another Cl.
> 
> I'm not sure what the change you're suggesting would mean.

I mean don't check the requester email, check the category instead (it's either
git cl try, cq or cq experimental), afaik.

[commit-bot: I haz the power, 2016-07-13 11:32:14]

Description was changed from

==========
webrtc: drop dependency on chromium_tests; extract chromium_swarming

This will make it easier to move chromium_tests out of build repo.

BUG=584079
==========

to

==========
webrtc: drop dependency on chromium_tests; extract chromium_swarming

This will make it easier to move chromium_tests out of build repo.

BUG=584079

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

[commit-bot: I haz the power, 2016-07-13 11:32:15]

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