Convert Mojo tests to use swarming.

Convert Mojo tests to use swarming.

BUG=604847
Committed: https://crrev.com/98ecd4a58d5b625759b2121be8254cd1abec8dc7
Cr-Commit-Position: refs/heads/master@{#390573}

[Sam McNally, 2016-04-28 04:50:28]

Patchset #1 (id:1) has been deleted

[Sam McNally, 2016-04-28 04:50:41]

Patchset #1 (id:20001) has been deleted

[Sam McNally, 2016-04-28 04:54:43]

Patchset #1 (id:40001) has been deleted

[Sam McNally, 2016-04-28 04:54:46]

Patchset #1 (id:60001) has been deleted

[Sam McNally, 2016-04-28 04:54:48]

Patchset #1 (id:80001) has been deleted

[Sam McNally, 2016-04-28 04:54:50]

Patchset #1 (id:100001) has been deleted

[Sam McNally, 2016-04-28 04:54:52]

Patchset #1 (id:120001) has been deleted

[Sam McNally, 2016-04-28 04:54:54]

Patchset #1 (id:140001) has been deleted

[Sam McNally, 2016-04-28 04:54:56]

Patchset #1 (id:160001) has been deleted

[Sam McNally, 2016-04-28 04:54:57]

Patchset #1 (id:180001) has been deleted

[Sam McNally, 2016-04-28 04:55:00]

Patchset #1 (id:200001) has been deleted

[Sam McNally, 2016-04-28 04:55:02]

Patchset #1 (id:220001) has been deleted

[Sam McNally, 2016-04-28 04:55:06]

Patchset #1 (id:240001) has been deleted

[Sam McNally, 2016-04-28 05:55:43]

Patchset #1 (id:260001) has been deleted

[Sam McNally, 2016-04-28 08:13:33]

sammc@chromium.org changed reviewers:
+ jam@chromium.org

[jam, 2016-04-28 15:59:30]

lgtm with minor comments to simplify build files

https://codereview.chromium.org/1918143004/diff/280001/BUILD.gn
File BUILD.gn (right):

https://codereview.chromium.org/1918143004/diff/280001/BUILD.gn#newcode625
BUILD.gn:625: "//mojo/tools/mopy/__init__.py",
just make this //mojo/tools/mopy/

you're listing all the files. even if you weren't, it's fine to list a few more
than you need to make the data deps easier to read

https://codereview.chromium.org/1918143004/diff/280001/mojo/mojo_js_integrati...
File mojo/mojo_js_integration_tests.isolate (right):

https://codereview.chromium.org/1918143004/diff/280001/mojo/mojo_js_integrati...
mojo/mojo_js_integration_tests.isolate:21:
'../mojo/edk/js/tests/connection_tests.js',
mojo/edk/js/tests/ is fine

https://codereview.chromium.org/1918143004/diff/280001/mojo/mojo_js_integrati...
mojo/mojo_js_integration_tests.isolate:24: '../mojo/public/js/bindings.js',
just mojo/public/js

https://codereview.chromium.org/1918143004/diff/280001/mojo/public/js/BUILD.gn
File mojo/public/js/BUILD.gn (right):

https://codereview.chromium.org/1918143004/diff/280001/mojo/public/js/BUILD.g...
mojo/public/js/BUILD.gn:40:
"//mojo/public/interfaces/bindings/tests/data/validation/associated_conformance_mthd0_good.data",
just
//mojo/public/interfaces/bindings/tests/data/validation

https://codereview.chromium.org/1918143004/diff/280001/services/catalog/BUILD.gn
File services/catalog/BUILD.gn (right):

https://codereview.chromium.org/1918143004/diff/280001/services/catalog/BUILD...
services/catalog/BUILD.gn:56: "//services/catalog/data/capabilities",
just //services/catalog/data/

[Sam McNally, 2016-04-29 00:21:24]

https://codereview.chromium.org/1918143004/diff/280001/BUILD.gn
File BUILD.gn (right):

https://codereview.chromium.org/1918143004/diff/280001/BUILD.gn#newcode625
BUILD.gn:625: "//mojo/tools/mopy/__init__.py",
On 2016/04/28 15:59:29, jam wrote:
> just make this //mojo/tools/mopy/
> 
> you're listing all the files. even if you weren't, it's fine to list a few
more
> than you need to make the data deps easier to read

Done. I wasn't aware that GN allowed directories as data.

https://codereview.chromium.org/1918143004/diff/280001/mojo/mojo_js_integrati...
File mojo/mojo_js_integration_tests.isolate (right):

https://codereview.chromium.org/1918143004/diff/280001/mojo/mojo_js_integrati...
mojo/mojo_js_integration_tests.isolate:21:
'../mojo/edk/js/tests/connection_tests.js',
On 2016/04/28 15:59:29, jam wrote:
> mojo/edk/js/tests/ is fine

Done.

https://codereview.chromium.org/1918143004/diff/280001/mojo/mojo_js_integrati...
mojo/mojo_js_integration_tests.isolate:24: '../mojo/public/js/bindings.js',
On 2016/04/28 15:59:29, jam wrote:
> just mojo/public/js

Done.

https://codereview.chromium.org/1918143004/diff/280001/mojo/public/js/BUILD.gn
File mojo/public/js/BUILD.gn (right):

https://codereview.chromium.org/1918143004/diff/280001/mojo/public/js/BUILD.g...
mojo/public/js/BUILD.gn:40:
"//mojo/public/interfaces/bindings/tests/data/validation/associated_conformance_mthd0_good.data",
On 2016/04/28 15:59:29, jam wrote:
> just
> //mojo/public/interfaces/bindings/tests/data/validation

Done.

https://codereview.chromium.org/1918143004/diff/280001/services/catalog/BUILD.gn
File services/catalog/BUILD.gn (right):

https://codereview.chromium.org/1918143004/diff/280001/services/catalog/BUILD...
services/catalog/BUILD.gn:56: "//services/catalog/data/capabilities",
On 2016/04/28 15:59:29, jam wrote:
> just //services/catalog/data/

Done.

[Sam McNally, 2016-04-29 01:38:17]

The CQ bit was checked by sammc@chromium.org

[Sam McNally, 2016-04-29 01:38:18]

The patchset sent to the CQ was uploaded after l-g-t-m from jam@chromium.org
Link to the patchset: https://codereview.chromium.org/1918143004/#ps320001
(title: "rebase")

[commit-bot: I haz the power, 2016-04-29 01:38:32]

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

[commit-bot: I haz the power, 2016-04-29 01:59:31]

Committed patchset #3 (id:320001)

[commit-bot: I haz the power, 2016-04-30 17:23:48]

Patchset 3 (id:??) landed as
https://crrev.com/98ecd4a58d5b625759b2121be8254cd1abec8dc7
Cr-Commit-Position: refs/heads/master@{#390573}

[commit-bot: I haz the power, 2016-04-30 17:59:04]

Description was changed from

==========
Convert Mojo tests to use swarming.

BUG=604847
==========

to

==========
Convert Mojo tests to use swarming.

BUG=604847

Committed: https://crrev.com/98ecd4a58d5b625759b2121be8254cd1abec8dc7
Cr-Commit-Position: refs/heads/master@{#390573}
==========