[Android] Add a dummy uiautomator tests target for chromium testshell.

[Android] Add a dummy uiautomator tests target for chromium testshell.

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

[cjhopman, 2013-04-17 20:28:35]

https://codereview.chromium.org/13977004/diff/1/chrome/chrome_tests.gypi
File chrome/chrome_tests.gypi (right):

https://codereview.chromium.org/13977004/diff/1/chrome/chrome_tests.gypi#newc...
chrome/chrome_tests.gypi:3233: 'input_jars_paths': [
'<(android_sdk)/uiautomator.jar' ],
Targets shouldn't add jars directly to input_jars_paths. Instead, create a
target for the uiautomator jar that includes build/java_prebuilt.gypi and then
depend on that.

I.e. add a target like this:
{
  'target_name': 'uiautomator_java',
  'variables': { 'jar_path': '<(android_sdk)/uiautomator.jar' },
  'includes': [ '../../build/java_prebuilt.gypi' ],
}
to third_party/android_tools/android_tools.gypi

and depend on that here.

[frankf, 2013-04-19 18:06:37]

PTAL

https://codereview.chromium.org/13977004/diff/1/chrome/chrome_tests.gypi
File chrome/chrome_tests.gypi (right):

https://codereview.chromium.org/13977004/diff/1/chrome/chrome_tests.gypi#newc...
chrome/chrome_tests.gypi:3233: 'input_jars_paths': [
'<(android_sdk)/uiautomator.jar' ],
Done.

On 2013/04/17 20:28:35, cjhopman wrote:
> Targets shouldn't add jars directly to input_jars_paths. Instead, create a
> target for the uiautomator jar that includes build/java_prebuilt.gypi and then
> depend on that.
> 
> I.e. add a target like this:
> {
>   'target_name': 'uiautomator_java',
>   'variables': { 'jar_path': '<(android_sdk)/uiautomator.jar' },
>   'includes': [ '../../build/java_prebuilt.gypi' ],
> }
> to third_party/android_tools/android_tools.gypi
> 
> and depend on that here.

[frankf, 2013-04-19 18:38:02]

+nilesh for OWNERS

[nilesh, 2013-04-19 19:00:42]

LGTM for chrome/android/

[frankf, 2013-04-22 17:47:59]

chris, can you take another look?

[cjhopman, 2013-04-22 17:52:46]

lgtm

[commit-bot: I haz the power, 2013-04-22 17:54:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/frankf@chromium.org/13977004/8001

[commit-bot: I haz the power, 2013-04-22 21:07:29]

Change committed as 195592