Upstream Tab's audio/video capturing APIs to ChromeWebContentsDelegateAndroid

Upstream Tab's audio/video capturing APIs to ChromeWebContentsDelegateAndroid

This is a little icky but probably better than just putting it in Tab.
It seems like MediaCaptureDevicesDispatcher could have its own JNI wrapper but
it would be quite thin. At least this way, the accesses are co-located.

BUG=443773
Committed: https://crrev.com/fd6f8f6532cd0d6b1c97ec28945ed0b9c2f974f6
Cr-Commit-Position: refs/heads/master@{#317585}

[Yaron, 2015-02-20 16:08:31]

yfriedman@chromium.org changed reviewers:
+ dtrainor@chromium.org, qinmin@chromium.org

[Yaron, 2015-02-20 16:08:31]

min: is there a better place for this within media?

[qinmin, 2015-02-20 18:34:59]

lgtm

https://codereview.chromium.org/942873002/diff/1/chrome/browser/android/chrom...
File chrome/browser/android/chrome_web_contents_delegate_android.cc (right):

https://codereview.chromium.org/942873002/diff/1/chrome/browser/android/chrom...
chrome/browser/android/chrome_web_contents_delegate_android.cc:352: jboolean
IsCapturingAudio(JNIEnv* env,
is there a way to add nested namespaces? Maybe we need to change jni_generator,
but can leave this as a TODO.

https://codereview.chromium.org/942873002/diff/1/chrome/browser/android/chrom...
chrome/browser/android/chrome_web_contents_delegate_android.cc:364: jclass
clazz,jobject java_web_contents) {
nit: match this with the style above

[David Trainor- moved to gerrit, 2015-02-20 19:42:35]

lgtm

[Yaron, 2015-02-23 14:26:24]

https://chromiumcodereview.appspot.com/942873002/diff/1/chrome/browser/androi...
File chrome/browser/android/chrome_web_contents_delegate_android.cc (right):

https://chromiumcodereview.appspot.com/942873002/diff/1/chrome/browser/androi...
chrome/browser/android/chrome_web_contents_delegate_android.cc:352: jboolean
IsCapturingAudio(JNIEnv* env,
On 2015/02/20 18:34:59, qinmin wrote:
> is there a way to add nested namespaces? Maybe we need to change
jni_generator,
> but can leave this as a TODO.

You can using JNINamespace but not selectively for a particular function and
there are other JNI members that would be affected
(CreateJavaRect/CreateJavaRectF).

I've left it as is but let me know if you want me to move all of them and I can
do in a follow up

https://chromiumcodereview.appspot.com/942873002/diff/1/chrome/browser/androi...
chrome/browser/android/chrome_web_contents_delegate_android.cc:364: jclass
clazz,jobject java_web_contents) {
On 2015/02/20 18:34:59, qinmin wrote:
> nit: match this with the style above

Done.

[Yaron, 2015-02-23 14:26:51]

New patchsets have been uploaded after l-g-t-m from
qinmin@chromium.org,dtrainor@chromium.org

[Yaron, 2015-02-23 14:27:02]

The CQ bit was checked by yfriedman@chromium.org

[commit-bot: I haz the power, 2015-02-23 14:28:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/942873002/20001

[commit-bot: I haz the power, 2015-02-23 14:50:09]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-23 14:51:01]

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