[android_webview] Add test shell for webview.

[android_webview] Add test shell for webview.

This adds a simple test shell (most code stolen from chrome testshell)
for android_webview testing.
The only currently supported rendering mode is s/w rendering.

BUG=None
Android_webview-only change, ran through trybots.
NOTRY=true

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=189246

[mkosiba (inactive), 2013-03-12 14:56:20]

PTAL

[mkosiba (inactive), 2013-03-14 15:23:25]

Ping?

[mkosiba (inactive), 2013-03-19 11:21:29]

ping ping ping?

[benm (inactive), 2013-03-19 12:55:01]

https://chromiumcodereview.appspot.com/12317015/diff/15001/android_webview/an...
File android_webview/android_webview_tests.gypi (right):

https://chromiumcodereview.appspot.com/12317015/diff/15001/android_webview/an...
android_webview/android_webview_tests.gypi:33: 'is_test_apk': 0, # We want
resources from android_webview_test_java.
We could just remove this, I think it defaults to 0?

https://chromiumcodereview.appspot.com/12317015/diff/15001/android_webview/an...
android_webview/android_webview_tests.gypi:62: 'native_libs_paths':
['<(SHARED_LIB_DIR)/libwebviewchromium.so'],
Why doesn't android_webview_test_apk need this variable?

https://chromiumcodereview.appspot.com/12317015/diff/15001/android_webview/an...
android_webview/android_webview_tests.gypi:64: 'asset_location':
'<(ant_build_out)/android_webview_apk/assets',
could we refactor the android_webview_test_apk to use asset_location instead of
additional_input_paths?

[mkosiba (inactive), 2013-03-19 17:11:26]

https://codereview.chromium.org/12317015/diff/15001/android_webview/android_w...
File android_webview/android_webview_tests.gypi (right):

https://codereview.chromium.org/12317015/diff/15001/android_webview/android_w...
android_webview/android_webview_tests.gypi:33: 'is_test_apk': 0, # We want
resources from android_webview_test_java.
On 2013/03/19 12:55:01, benm wrote:
> We could just remove this, I think it defaults to 0?

I wanted some place to stick the comment in. It seemed cleaner to leave it set
explicitly to 0 and explain why.

https://codereview.chromium.org/12317015/diff/15001/android_webview/android_w...
android_webview/android_webview_tests.gypi:62: 'native_libs_paths':
['<(SHARED_LIB_DIR)/libwebviewchromium.so'],
On 2013/03/19 12:55:01, benm wrote:
> Why doesn't android_webview_test_apk need this variable?

because the test apk only contains test code and at runtime "merges" with the
android_webview_apk which contains the .so.

https://codereview.chromium.org/12317015/diff/15001/android_webview/android_w...
android_webview/android_webview_tests.gypi:64: 'asset_location':
'<(ant_build_out)/android_webview_apk/assets',
On 2013/03/19 12:55:01, benm wrote:
> could we refactor the android_webview_test_apk to use asset_location instead
of
> additional_input_paths?

not sure, nobody uses it like that in the tree. Would it be ok to land that as a
separate patch so that if someone needs to revert it they don't end up reverting
this change as well?

[benm (inactive), 2013-03-19 19:03:57]

lgtm

thanks! Sorry for the delay in looking over this!

[commit-bot: I haz the power, 2013-03-20 10:43:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mkosiba@chromium.org/12317015/27001

[commit-bot: I haz the power, 2013-03-20 12:06:54]

Change committed as 189246