jni: Pass method parameters as JavaParamRef in content.

jni: Pass method parameters as JavaParamRef in content.

Pass all object parameters to JNI methods in JavaParamRef<> wrappers.
This matches previous changes made to do this for JNI non-method
functions.

This change omits changes to speech as there are some less-trivial cases
there which will be dealt with in a later CL.

BUG=506850
Committed: https://crrev.com/e1bfee9a009140ffef7945a083cf1c66ac15def7
Cr-Commit-Position: refs/heads/master@{#362409}

[Torne, 2015-11-30 11:47:05]

torne@chromium.org changed reviewers:
+ sievers@chromium.org

[Torne, 2015-11-30 11:47:06]

Automated changes except for the parts called out in comments:

https://codereview.chromium.org/1489453003/diff/1/content/browser/android/con...
File content/browser/android/content_view_core_impl.cc (right):

https://codereview.chromium.org/1489453003/diff/1/content/browser/android/con...
content/browser/android/content_view_core_impl.cc:1216:
ConvertJavaStringToUTF8(env, name), object, safe_annotation_clazz);
manual change: no need to construct ScopedJavaLocalRefs any more as the
parameters are already JavaRefs.

https://codereview.chromium.org/1489453003/diff/1/content/browser/android/dat...
File content/browser/android/date_time_chooser_android.cc (right):

https://codereview.chromium.org/1489453003/diff/1/content/browser/android/dat...
content/browser/android/date_time_chooser_android.cc:54: const
JavaRef<jobject>&,
I'm using JavaRef instead of JavaParamRef in this file to allow the call on line
105 to work. The ultimate goal is to use JavaRef everywhere, and JavaParamRef is
just an intermediate step to allow incremental migration (it has an implicit
conversion back to jobject, which isn't needed here), so skipping ahead here is
okay :)

[no sievers, 2015-11-30 22:54:11]

lgtm

[Torne, 2015-12-01 12:50:51]

The CQ bit was checked by torne@chromium.org

[commit-bot: I haz the power, 2015-12-01 12:51:33]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1489453003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1489453003/1

[commit-bot: I haz the power, 2015-12-01 14:05:08]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-01 14:06:06]

Description was changed from

==========
jni: Pass method parameters as JavaParamRef in content.

Pass all object parameters to JNI methods in JavaParamRef<> wrappers.
This matches previous changes made to do this for JNI non-method
functions.

This change omits changes to speech as there are some less-trivial cases
there which will be dealt with in a later CL.

BUG=506850
==========

to

==========
jni: Pass method parameters as JavaParamRef in content.

Pass all object parameters to JNI methods in JavaParamRef<> wrappers.
This matches previous changes made to do this for JNI non-method
functions.

This change omits changes to speech as there are some less-trivial cases
there which will be dealt with in a later CL.

BUG=506850

Committed: https://crrev.com/e1bfee9a009140ffef7945a083cf1c66ac15def7
Cr-Commit-Position: refs/heads/master@{#362409}
==========

[commit-bot: I haz the power, 2015-12-01 14:06:07]

Patchset 1 (id:??) landed as
https://crrev.com/e1bfee9a009140ffef7945a083cf1c66ac15def7
Cr-Commit-Position: refs/heads/master@{#362409}