Move base Java utils to base/test/android/javatests.

Move base Java utils to base/test/android/javatests.

gyp targets should not depend on tests from other packages, but
instead be able to depend on their test utils.

The target including the base utilities is called
base_java_test_support and contains files from
base/test/android/javatests.

base_java_test_support was also the name of the wrongly named target for
building base/android/javatests, but since only contained util-classes,
no new target has been created for tests of base/android/java.

BUG=151561, 150773
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=158422

[nyquist, 2012-09-24 18:34:59]

PTAL

[Yaron, 2012-09-24 18:44:28]

lgtm

Please also add a base/test/android/OWNERS file with ppl from 
base/android/OWNERS

https://codereview.chromium.org/10974012/diff/1/android_webview/javatests/src...
File
android_webview/javatests/src/org/chromium/android_webview/test/WebViewSynchronousFindApisTest.java
(right):

https://codereview.chromium.org/10974012/diff/1/android_webview/javatests/src...
android_webview/javatests/src/org/chromium/android_webview/test/WebViewSynchronousFindApisTest.java:9:
import org.chromium.base.util.Feature;
This is not clear that it's test-only. As discussed please make this
org.chromium.base.test.util.Feature (and to all others)

[klundberg, 2012-09-24 18:52:30]

lgtm

https://codereview.chromium.org/10974012/diff/1/android_webview/javatests/src...
File
android_webview/javatests/src/org/chromium/android_webview/test/WebViewSynchronousFindApisTest.java
(right):

https://codereview.chromium.org/10974012/diff/1/android_webview/javatests/src...
android_webview/javatests/src/org/chromium/android_webview/test/WebViewSynchronousFindApisTest.java:9:
import org.chromium.base.util.Feature;
On 2012/09/24 18:44:28, Yaron wrote:
> This is not clear that it's test-only. As discussed please make this
> org.chromium.base.test.util.Feature (and to all others) 

I think this is a good idea.

I'll update the guidelines to reflect this.

[nyquist, 2012-09-24 19:09:04]

mark: could you please take a look at base/?

[Mark Mentovai, 2012-09-24 19:16:02]

Just to appease the commit queue? LGTM for purposes of moving things between
OWNed directories in base. I haven’t looked at any code.

[joth, 2012-09-24 19:51:35]

lgtm

[commit-bot: I haz the power, 2012-09-24 19:52:09]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nyquist@chromium.org/10974012/5001

[commit-bot: I haz the power, 2012-09-24 22:14:26]

Commit queue rejected this change because the description was changed
between the time the change entered the commit queue and the time it
was ready to commit. You can safely check the commit box again.

[commit-bot: I haz the power, 2012-09-24 22:17:11]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nyquist@chromium.org/10974012/5001

[commit-bot: I haz the power, 2012-09-24 22:22:31]

Change committed as 158422