Make base_unittests_apk actually work

Make base_unittests_apk actually work

This changes the "test" template to create a shared_library (instead of
an executable) on Android. After this, we can actually run
base_unittests with
`build/android/test_runner.py gtest -s base_unnittests`
as normal (though may need to CHROMIUM_OUT_DIR and BUILDTYPE as
appropriate).

This requires adding the following targets:

//testing/android:native_test_native_code
//testing/android:native_test_util
//testing/android:native_test_jni_headers
//tools/android/md5sum:md5sum
//tools/android/md5sum:md5sum_bin
//tools/android/md5sum:md5sum_prepare_dist
//tools/android/md5sum:md5sum_copy_host

Also, makes it so that native executables are stripped (just like shared
libraries). Adds a simple create_native_execuatable_dist template that
sets up a dist directory for the executable (see
build/android/gyp/native_app_dependencies.gyp).

BUG=359249
TBR=rlarocque

Committed: https://crrev.com/e65ead8fb1a1976682356cf824445476584e70f0
Cr-Commit-Position: refs/heads/master@{#294032}

[cjhopman, 2014-09-08 18:09:44]

cjhopman@chromium.org changed reviewers:
+ brettw@chromium.org

[cjhopman, 2014-09-08 18:09:45]

brettw: *

https://codereview.chromium.org/557463002/diff/1/build/config/BUILDCONFIG.gn
File build/config/BUILDCONFIG.gn (right):

https://codereview.chromium.org/557463002/diff/1/build/config/BUILDCONFIG.gn#...
build/config/BUILDCONFIG.gn:500: # The sources assignment filter will have
already been applied when the
I would prefer that we didn't allow variables set out here to leak into
target/template invocations (and instead required you to explicitly set them in
the scope of the invocation), but if we're going to allow it we might as well
take advantage of it.

brettw: wdyt?

[brettw, 2014-09-08 20:04:22]

lgtm

https://codereview.chromium.org/557463002/diff/1/build/config/BUILDCONFIG.gn
File build/config/BUILDCONFIG.gn (right):

https://codereview.chromium.org/557463002/diff/1/build/config/BUILDCONFIG.gn#...
build/config/BUILDCONFIG.gn:404: set_defaults("test") {
I think you will need to update the default configs for tests in android to be
the shared library configs.

https://codereview.chromium.org/557463002/diff/1/build/config/BUILDCONFIG.gn#...
build/config/BUILDCONFIG.gn:500: # The sources assignment filter will have
already been applied when the
I'd rather keep this the old way, maybe we can restrict the usage in the tool (I
agree with that this is confusing).

https://codereview.chromium.org/557463002/diff/1/testing/android/BUILD.gn
File testing/android/BUILD.gn (right):

https://codereview.chromium.org/557463002/diff/1/testing/android/BUILD.gn#new...
testing/android/BUILD.gn:17: ":native_test_jni_headers",
I put these ":" ones first rather than last.

[cjhopman, 2014-09-09 16:55:32]

https://codereview.chromium.org/557463002/diff/1/build/config/BUILDCONFIG.gn
File build/config/BUILDCONFIG.gn (right):

https://codereview.chromium.org/557463002/diff/1/build/config/BUILDCONFIG.gn#...
build/config/BUILDCONFIG.gn:404: set_defaults("test") {
On 2014/09/08 20:04:22, brettw wrote:
> I think you will need to update the default configs for tests in android to be
> the shared library configs.

Done.

https://codereview.chromium.org/557463002/diff/1/build/config/BUILDCONFIG.gn#...
build/config/BUILDCONFIG.gn:500: # The sources assignment filter will have
already been applied when the
On 2014/09/08 20:04:22, brettw wrote:
> I'd rather keep this the old way, maybe we can restrict the usage in the tool
(I
> agree with that this is confusing).

Done.

https://codereview.chromium.org/557463002/diff/1/testing/android/BUILD.gn
File testing/android/BUILD.gn (right):

https://codereview.chromium.org/557463002/diff/1/testing/android/BUILD.gn#new...
testing/android/BUILD.gn:17: ":native_test_jni_headers",
On 2014/09/08 20:04:22, brettw wrote:
> I put these ":" ones first rather than last.

Done.

[cjhopman, 2014-09-09 16:56:01]

cjhopman@chromium.org changed reviewers:
+ rlarocque@chromium.org

[cjhopman, 2014-09-09 16:56:02]

rlarocque: sync/

[cjhopman, 2014-09-09 21:02:14]

The CQ bit was checked by cjhopman@chromium.org

[cjhopman, 2014-09-09 21:02:21]

The CQ bit was unchecked by cjhopman@chromium.org

[cjhopman, 2014-09-09 21:02:38]

The CQ bit was checked by cjhopman@chromium.org

[commit-bot: I haz the power, 2014-09-09 21:22:07]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/cjhopman@chromium.org/557463002/60001

[commit-bot: I haz the power, 2014-09-09 23:23:56]

Committed patchset #4 (id:60001) as 37ee9f0e2219611793c5a4dcf846760177473b52

[Dirk Pranke, 2014-09-09 23:32:50]

On 2014/09/09 23:23:56, I haz the power (commit-bot) wrote:
> Committed patchset #4 (id:60001) as 37ee9f0e2219611793c5a4dcf846760177473b52

yay!

[commit-bot: I haz the power, 2014-09-10 03:56:26]

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