Add a test runner for android_webview.

Add a test runner for android_webview.

This change adds the necessary build rules for an android_webview
test APK and test runner APK and a trivial test case.
This change also adds a static Java library target to the WebContentsDelegateAndroid component.

BUG=

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

[mkosiba (inactive), 2012-08-23 14:24:07]

Joi,
    Could you please take a look at DEPS changes?

Torne,
    Could you please take a look at the gyp stuff?

Yaron,
    I need an OWNER approval for the content/android stuff, could you please
take a look at that?

[Jói, 2012-08-23 14:27:46]

DEPS LGTM.

https://chromiumcodereview.appspot.com/10855171/diff/6001/android_webview/lib...
File android_webview/lib/DEPS (right):

https://chromiumcodereview.appspot.com/10855171/diff/6001/android_webview/lib...
android_webview/lib/DEPS:21:
"!chrome/browser/api/infobars/confirm_infobar_delegate.h",
This dependency would be OK as a permanent one, up to you whether to use ! or +
for it.

[mkosiba (inactive), 2012-08-23 14:40:22]

After re-running checkdeps it seems I'll need to add a DEPS file under
chrome/android/java to allow for the web_contents_delegate_android component.
Will update the change in a sec.

https://chromiumcodereview.appspot.com/10855171/diff/6001/android_webview/lib...
File android_webview/lib/DEPS (right):

https://chromiumcodereview.appspot.com/10855171/diff/6001/android_webview/lib...
android_webview/lib/DEPS:21:
"!chrome/browser/api/infobars/confirm_infobar_delegate.h",
On 2012/08/23 14:27:46, Jói wrote:
> This dependency would be OK as a permanent one, up to you whether to use ! or
+
> for it.
Actually looks like we don't need it anymore - it was included form
android_webview/lib/main/webview_stubs.cc but the stub was removed while the
include and DEPS entry stayed.

[mkosiba (inactive), 2012-08-23 15:03:14]

New change is out. This only adds a DEPS file under chrome/android/java and
removes the infobar dep from android_webview/lib.

[Torne, 2012-08-23 15:11:46]

GYPs lgtm

https://chromiumcodereview.appspot.com/10855171/diff/6001/android_webview/lib...
File android_webview/lib/DEPS (right):

https://chromiumcodereview.appspot.com/10855171/diff/6001/android_webview/lib...
android_webview/lib/DEPS:21:
"!chrome/browser/api/infobars/confirm_infobar_delegate.h",
On 2012/08/23 14:27:46, Jói wrote:
> This dependency would be OK as a permanent one, up to you whether to use ! or
+
> for it.

We won't have the same infobar functionality as Chrome here (we have only
callbacks at the webview API level, no UI for these things), so it's probably
fine that it's temporary.

[Yaron, 2012-08-23 15:52:54]

content/public/android lgtm

https://chromiumcodereview.appspot.com/10855171/diff/6002/content/public/andr...
File
content/public/android/javatests/src/org/chromium/content/browser/test/CallbackHelper.java
(right):

https://chromiumcodereview.appspot.com/10855171/diff/6002/content/public/andr...
content/public/android/javatests/src/org/chromium/content/browser/test/CallbackHelper.java:30:
public synchronized int getCallCount() {
Please use a private object and synchronize on that instead of using a
synchronized method.

[Jói, 2012-08-23 16:06:08]

DEPS still LGTM

[Yaron, 2012-08-23 17:06:01]

http://codereview.chromium.org/10855171/diff/13003/content/public/android/jav...
File
content/public/android/javatests/src/org/chromium/content/browser/test/CallbackHelper.java
(right):

http://codereview.chromium.org/10855171/diff/13003/content/public/android/jav...
content/public/android/javatests/src/org/chromium/content/browser/test/CallbackHelper.java:87:
public synchronized void notifyCalled() {
This shouldn't be synchronized either. It should also use mLock

[mkosiba (inactive), 2012-08-23 19:05:52]

http://codereview.chromium.org/10855171/diff/13003/content/public/android/jav...
File
content/public/android/javatests/src/org/chromium/content/browser/test/CallbackHelper.java
(right):

http://codereview.chromium.org/10855171/diff/13003/content/public/android/jav...
content/public/android/javatests/src/org/chromium/content/browser/test/CallbackHelper.java:87:
public synchronized void notifyCalled() {
On 2012/08/23 17:06:01, Yaron wrote:
> This shouldn't be synchronized either. It should also use mLock

Done.