Revert of Reland Switch Android gtests to use generated wrapper scripts

Revert of Reland Switch Android gtests to use generated wrapper scripts (patchset #2 id:20001 of https://codereview.chromium.org/1881233003/ )

Reason for revert:
Broke https://build.chromium.org/p/chromium.fyi/builders/Android%20Builder%20%28dbg%29/builds/67100/steps/compile/logs/stdio

Original issue's description:
> Reland Switch Android gtests to use generated wrapper scripts
> 
> TBR=sergeyberezin
> BUG=599919
> 
> Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299906

TBR=bpastene@chromium.org,kjellander@chromium.org,mikecase@chromium.org,sergeyberezin@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=599919
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299909

[agrieve, 2016-04-13 20:27:35]

Created Revert of Reland Switch Android gtests to use generated wrapper scripts

[commit-bot: I haz the power, 2016-04-13 20:27:42]

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

[commit-bot: I haz the power, 2016-04-13 20:28:08]

Description was changed from

==========
Revert of Reland Switch Android gtests to use generated wrapper scripts
(patchset #2 id:20001 of https://codereview.chromium.org/1881233003/ )

Reason for revert:
Broke
https://build.chromium.org/p/chromium.fyi/builders/Android%20Builder%20%28dbg...

Original issue's description:
> Reland Switch Android gtests to use generated wrapper scripts
> 
> TBR=sergeyberezin
> BUG=599919
> 
> Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299906

TBR=bpastene@chromium.org,kjellander@chromium.org,mikecase@chromium.org,serge...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=599919
==========

to

==========
Revert of Reland Switch Android gtests to use generated wrapper scripts
(patchset #2 id:20001 of https://codereview.chromium.org/1881233003/ )

Reason for revert:
Broke
https://build.chromium.org/p/chromium.fyi/builders/Android%20Builder%20%28dbg...

Original issue's description:
> Reland Switch Android gtests to use generated wrapper scripts
> 
> TBR=sergeyberezin
> BUG=599919
> 
> Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299906

TBR=bpastene@chromium.org,kjellander@chromium.org,mikecase@chromium.org,serge...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=599919

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

[commit-bot: I haz the power, 2016-04-13 20:28:09]

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

[agrieve, 2016-04-13 20:30:20]

On 2016/04/13 20:28:09, commit-bot: I haz the power wrote:
> Committed patchset #1 (id:1) as
> http://src.chromium.org/viewvc/chrome?view=rev&revision=299909

This ones trickier. I suspect what happened is that there's a name collision for
unit_tests, so GN didn't create an convience ninja target for it. Once we're
fully on GN, it will make sense to use fully-qualified build labels in the
recipes, but for now I might look at special-casing unit_tests.

[agrieve, 2016-04-13 20:31:11]

On 2016/04/13 20:30:20, agrieve wrote:
> On 2016/04/13 20:28:09, commit-bot: I haz the power wrote:
> > Committed patchset #1 (id:1) as
> > http://src.chromium.org/viewvc/chrome?view=rev&revision=299909
> 
> This ones trickier. I suspect what happened is that there's a name collision
for
> unit_tests, so GN didn't create an convience ninja target for it. Once we're
> fully on GN, it will make sense to use fully-qualified build labels in the
> recipes, but for now I might look at special-casing unit_tests.

Here's the command-line for future reference:
/mnt/data/b/build/scripts/slave/.recipe_deps/depot_tools/ninja -C
/mnt/data/b/build/slave/Android_Builder__dbg_/build/src/out/Debug
android_webview_test_apk android_webview_unittests android_webview_unittests_apk
base_unittests base_unittests_apk blink_heap_unittests blink_heap_unittests_apk
blink_platform_unittests blink_platform_unittests_apk breakpad_unittests_deps
cc_unittests cc_unittests_apk chrome_junit_tests chrome_public_apk
chrome_public_test_apk chrome_sync_shell_test_apk chromedriver_webview_shell_apk
components_unittests components_unittests_apk content_browsertests
content_browsertests_apk content_junit_tests content_shell_test_apk
content_unittests content_unittests_apk events_unittests events_unittests_apk
gfx_unittests gfx_unittests_apk gl_tests gl_tests_apk gpu_unittests
gpu_unittests_apk ipc_tests ipc_tests_apk junit_unit_tests media_unittests
media_unittests_apk net_unittests net_unittests_apk remoting_test_apk
remoting_unittests remoting_unittests_apk sandbox_linux_unittests_deps
sql_unittests sql_unittests_apk sync_unit_tests sync_unit_tests_apk
ui_android_unittests ui_android_unittests_apk ui_base_unittests
ui_base_unittests_apk ui_touch_selection_unittests
ui_touch_selection_unittests_apk unit_tests unit_tests_apk -j50
ninja: Entering directory
`/mnt/data/b/build/slave/Android_Builder__dbg_/build/src/out/Debug'
ninja: error: unknown target 'unit_tests', did you mean 'unit_tests_js'?

[mikecase (-- gone --), 2016-04-13 21:49:22]

lgtm

[agrieve, 2016-04-18 18:57:00]

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

The reason for reverting is: will fix #3.