Add a blink_heap_unittests_apk so we can run the tests on Android.

Add a blink_heap_unittests_apk so we can run the tests on Android.

R=yfriedman@chromium.org, maruel@chromium.org, abarth@chromium.org
BUG=334018
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=165897

[Dirk Pranke, 2014-01-27 21:18:46]

yfriedman - does this change look correct?

maruel - do we actually need isolates for android unit tests?

abarth - can I get an OWNERS rubber stamp for public/* ?

ager - FYI :).

[M-A Ruel, 2014-01-27 21:27:53]

> maruel - do we actually need isolates for android unit tests?

Yes, it's how it knows which files must be pushed on the device. Android testing
is only using isolated testing, not swarming.

https://codereview.chromium.org/148453004/diff/10001/Source/heap/blink_heap_t...
File Source/heap/blink_heap_tests.gyp (right):

https://codereview.chromium.org/148453004/diff/10001/Source/heap/blink_heap_t...
Source/heap/blink_heap_tests.gyp:83: ['OS=="android" and
android_webview_build==0 and gtest_target_type == "shared_library"', {
either "foo==bar" or "foo == bar" but not both.

I'm not sure who started this trend of putting spaces in the conditions.

[Dirk Pranke, 2014-01-27 22:09:39]

https://codereview.chromium.org/148453004/diff/10001/Source/heap/blink_heap_t...
File Source/heap/blink_heap_tests.gyp (right):

https://codereview.chromium.org/148453004/diff/10001/Source/heap/blink_heap_t...
Source/heap/blink_heap_tests.gyp:83: ['OS=="android" and
android_webview_build==0 and gtest_target_type == "shared_library"', {
On 2014/01/27 21:27:53, M-A Ruel wrote:
> either "foo==bar" or "foo == bar" but not both.
> 
> I'm not sure who started this trend of putting spaces in the conditions.

Fixed (to the proper, no-spaces, style :)

[Yaron, 2014-01-27 22:25:33]

lgtm

https://codereview.chromium.org/148453004/diff/30001/Source/heap/blink_heap_t...
File Source/heap/blink_heap_tests.gyp (right):

https://codereview.chromium.org/148453004/diff/30001/Source/heap/blink_heap_t...
Source/heap/blink_heap_tests.gyp:94: 'input_shlib_path':
'<(SHARED_LIB_DIR)/<(SHARED_LIB_PREFIX)webkit_unit_tests<(SHARED_LIB_SUFFIX)',
This is the wrong value but it's actually unused now. It's the deprecated way
and was replaced with test_suite_name. You can remove this line.

[Dirk Pranke, 2014-01-27 22:46:19]

https://codereview.chromium.org/148453004/diff/30001/Source/heap/blink_heap_t...
File Source/heap/blink_heap_tests.gyp (right):

https://codereview.chromium.org/148453004/diff/30001/Source/heap/blink_heap_t...
Source/heap/blink_heap_tests.gyp:94: 'input_shlib_path':
'<(SHARED_LIB_DIR)/<(SHARED_LIB_PREFIX)webkit_unit_tests<(SHARED_LIB_SUFFIX)',
On 2014/01/27 22:25:34, Yaron wrote:
> This is the wrong value but it's actually unused now. It's the deprecated way
> and was replaced with test_suite_name. You can remove this line.

Fixed, thanks!

[abarth-chromium, 2014-01-27 23:13:01]

lgtm

[commit-bot: I haz the power, 2014-01-28 00:01:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dpranke@chromium.org/148453004/50001

[commit-bot: I haz the power, 2014-01-28 01:21:18]

Change committed as 165897