Reland 2 of GN: Make breakpad_unittests & sandbox_linux_unittests use test()

Reland 2 of GN: Make breakpad_unittests & sandbox_linux_unittests use test()

This simplifies build rules for native tests, and allows us to get rid
of ${target}_deps targets (once recipes are updated).

This change fixes the generated wrapper scripts, which didn't work.

TBR=jbudorick
BUG=589318
Committed: https://crrev.com/62ab0028a0165eecd60568783531d1ad3d031a26
Cr-Commit-Position: refs/heads/master@{#385084}

[agrieve, 2016-04-04 19:28:53]

Created Reland of GN: Make breakpad_unittests & sandbox_linux_unittests use
test()

[agrieve, 2016-04-04 19:31:14]

Description was changed from

==========
Reland of GN: Make breakpad_unittests & sandbox_linux_unittests use test()
(patchset #1 id:1 of https://codereview.chromium.org/1858683002/ )

Reason for revert:
attempt 3 here we come!

Original issue's description:
> Revert of GN: Make breakpad_unittests & sandbox_linux_unittests use test()
(patchset #2 id:200001 of https://codereview.chromium.org/1854733003/ )
> 
> Reason for revert:
> Still breaking bots:
> 
> crbug.com/600342
> 
> Original issue's description:
> > Reland of GN: Make breakpad_unittests & sandbox_linux_unittests use test()
> >
> > Reason for revert:
> > Now doesn't use obj/ directory
> >
> > This simplifies build rules for native tests, and allows us to get rid
> > of ${target}_deps targets (once recipes are updated).
> >
> > This change fixes the generated wrapper scripts, which didn't work.
> >
> > TBR=jorgelo
> > BUG=589318
> >
> > Committed: https://crrev.com/66605a610bce190e818ebfc73913a126c025ffb3
> > Cr-Commit-Position: refs/heads/master@{#384658}
> 
>
TBR=mark@chromium.org,dpranke@chromium.org,jorgelo@chromium.org,jbudorick@chr...
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=589318
> 
> Committed: https://crrev.com/e30b022bb758a10e9ecea81d5433bd1e046ccac4
> Cr-Commit-Position: refs/heads/master@{#384949}

TBR=mark@chromium.org,dpranke@chromium.org,jorgelo@chromium.org,jbudorick@chr...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=589318
==========

to

==========
Reland 2 of GN: Make breakpad_unittests & sandbox_linux_unittests use test()

This simplifies build rules for native tests, and allows us to get rid
of ${target}_deps targets (once recipes are updated).

This change fixes the generated wrapper scripts, which didn't work.

BUG=589318
==========

[agrieve, 2016-04-04 19:31:31]

yfriedman for tools/android

[agrieve, 2016-04-04 19:31:55]

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

[agrieve, 2016-04-04 19:32:14]

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

[commit-bot: I haz the power, 2016-04-04 19:32:33]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1854233002/170001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1854233002/170001

[Dirk Pranke, 2016-04-04 19:36:56]

lgtm

[Yaron, 2016-04-04 19:39:04]

lgtm

[commit-bot: I haz the power, 2016-04-04 21:47:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-04 21:47:27]

Dry run: This issue passed the CQ dry run.

[agrieve, 2016-04-05 01:56:03]

Description was changed from

==========
Reland 2 of GN: Make breakpad_unittests & sandbox_linux_unittests use test()

This simplifies build rules for native tests, and allows us to get rid
of ${target}_deps targets (once recipes are updated).

This change fixes the generated wrapper scripts, which didn't work.

BUG=589318
==========

to

==========
Reland 2 of GN: Make breakpad_unittests & sandbox_linux_unittests use test()

This simplifies build rules for native tests, and allows us to get rid
of ${target}_deps targets (once recipes are updated).

This change fixes the generated wrapper scripts, which didn't work.

TBR=jbudorick
BUG=589318
==========

[agrieve, 2016-04-05 01:56:37]

On 2016/04/04 21:47:27, commit-bot: I haz the power wrote:
> Dry run: This issue passed the CQ dry run.

TBR-ing for reland since i'll be around for the next hour or so.

[agrieve, 2016-04-05 01:56:45]

The CQ bit was checked by agrieve@chromium.org

[commit-bot: I haz the power, 2016-04-05 01:57:08]

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

[commit-bot: I haz the power, 2016-04-05 02:03:53]

Description was changed from

==========
Reland 2 of GN: Make breakpad_unittests & sandbox_linux_unittests use test()

This simplifies build rules for native tests, and allows us to get rid
of ${target}_deps targets (once recipes are updated).

This change fixes the generated wrapper scripts, which didn't work.

TBR=jbudorick
BUG=589318
==========

to

==========
Reland 2 of GN: Make breakpad_unittests & sandbox_linux_unittests use test()

This simplifies build rules for native tests, and allows us to get rid
of ${target}_deps targets (once recipes are updated).

This change fixes the generated wrapper scripts, which didn't work.

TBR=jbudorick
BUG=589318
==========

[commit-bot: I haz the power, 2016-04-05 02:03:54]

Committed patchset #2 (id:170001)

[commit-bot: I haz the power, 2016-04-05 02:04:53]

Description was changed from

==========
Reland 2 of GN: Make breakpad_unittests & sandbox_linux_unittests use test()

This simplifies build rules for native tests, and allows us to get rid
of ${target}_deps targets (once recipes are updated).

This change fixes the generated wrapper scripts, which didn't work.

TBR=jbudorick
BUG=589318
==========

to

==========
Reland 2 of GN: Make breakpad_unittests & sandbox_linux_unittests use test()

This simplifies build rules for native tests, and allows us to get rid
of ${target}_deps targets (once recipes are updated).

This change fixes the generated wrapper scripts, which didn't work.

TBR=jbudorick
BUG=589318

Committed: https://crrev.com/62ab0028a0165eecd60568783531d1ad3d031a26
Cr-Commit-Position: refs/heads/master@{#385084}
==========

[commit-bot: I haz the power, 2016-04-05 02:04:54]

Patchset 2 (id:??) landed as
https://crrev.com/62ab0028a0165eecd60568783531d1ad3d031a26
Cr-Commit-Position: refs/heads/master@{#385084}