Revert of GN: Make breakpad_unittests & sandbox_linux_unittests use test()

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@chromium.org
# 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}

[agrieve, 2016-04-04 15:51:58]

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

[commit-bot: I haz the power, 2016-04-04 15:52:22]

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

[commit-bot: I haz the power, 2016-04-04 16:01:46]

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

[commit-bot: I haz the power, 2016-04-04 16:31:51]

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

[commit-bot: I haz the power, 2016-04-04 17:01:50]

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

[commit-bot: I haz the power, 2016-04-04 17:31:55]

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

[Dirk Pranke, 2016-04-04 17:36:53]

lgtm

[commit-bot: I haz the power, 2016-04-04 17:57:06]

Description was changed from

==========
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
==========

to

==========
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
==========

[commit-bot: I haz the power, 2016-04-04 17:57:07]

Committed patchset #1 (id:1)

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

Description was changed from

==========
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
==========

to

==========
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}
==========

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

Patchset 1 (id:??) landed as
https://crrev.com/e30b022bb758a10e9ecea81d5433bd1e046ccac4
Cr-Commit-Position: refs/heads/master@{#384949}

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

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

The reason for reverting is: attempt 3 here we come!.