JNI Bindings on Chrome for Android: unfork.

JNI Bindings on Chrome for Android: unfork.

A few more changes as required by:
http://codereview.chromium.org/9466024/
Also, rather than just running the python-based tests
(jni_generator_py_tests), we'll now run the full suite
(jni_generator_tests) instead.

BUG=none
TEST=jni_generator_tests


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=125604

[joth, 2012-03-05 17:21:32]

lgtm

[bulach, 2012-03-06 10:28:10]

+mark, for base.gyp

[Mark Mentovai, 2012-03-06 22:02:29]

http://codereview.chromium.org/9599010/diff/1/base/android/jni_android.h
File base/android/jni_android.h (right):

http://codereview.chromium.org/9599010/diff/1/base/android/jni_android.h#newc...
base/android/jni_android.h:15: #define JNI_EXPORT __attribute__
((visibility("default")))
Unused?

[bulach, 2012-03-07 11:04:23]

thanks mark! comment addressed, another look please?

http://codereview.chromium.org/9599010/diff/1/base/android/jni_android.h
File base/android/jni_android.h (right):

http://codereview.chromium.org/9599010/diff/1/base/android/jni_android.h#newc...
base/android/jni_android.h:15: #define JNI_EXPORT __attribute__
((visibility("default")))
On 2012/03/06 22:02:30, Mark Mentovai wrote:
> Unused?

ops. this will be upstreamed soon, removed for now.

[Mark Mentovai, 2012-03-07 17:11:11]

LGTM

[commit-bot: I haz the power, 2012-03-08 08:54:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bulach@chromium.org/9599010/4002

[commit-bot: I haz the power, 2012-03-08 12:39:16]

Change committed as 125604