Upstream Tab's audio/video capturing APIs to ChromeWebContentsDelegateAndroid

chrome/browser/android/chrome_web_contents_delegate_android.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/android/chrome_web_contents_delegate_android.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "base/command_line.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/file_select_helper.h"
 #include "chrome/browser/media/media_capture_devices_dispatcher.h"
+#include "chrome/browser/media/media_stream_capture_indicator.h"
 #include "chrome/browser/media/protected_media_identifier_permission_context.h"
 #include "chrome/browser/media/protected_media_identifier_permission_context_factory.h"
 #include "chrome/browser/prerender/prerender_manager.h"
 #include "chrome/browser/prerender/prerender_manager_factory.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/blocked_content/popup_blocker_tab_helper.h"
 #include "chrome/browser/ui/browser_navigator.h"
 #include "chrome/browser/ui/find_bar/find_notification_details.h"
 #include "chrome/browser/ui/find_bar/find_tab_helper.h"
 #include "chrome/browser/ui/tab_helpers.h"
@@ skipping 317 matching lines @@
   }
 
   if (was_blocked)
     *was_blocked = !handled;
   if (!handled)
     delete new_contents;
 }
 
 }  // namespace android
 }  // namespace chrome
+
+jboolean IsCapturingAudio(JNIEnv* env,

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

is there a way to add nested namespaces? Maybe we need to change jni_generator,
but can leave this as a TODO.

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

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

+                          jclass clazz,
+                          jobject java_web_contents) {
+  content::WebContents* web_contents =
+      content::WebContents::FromJavaWebContents(java_web_contents);
+  scoped_refptr<MediaStreamCaptureIndicator> indicator =
+      MediaCaptureDevicesDispatcher::GetInstance()->
+          GetMediaStreamCaptureIndicator();
+  return indicator->IsCapturingAudio(web_contents);
+}
+
+jboolean IsCapturingVideo(JNIEnv* env,
+                          jclass clazz,jobject java_web_contents) {

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

nit: match this with the style above

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

Done.

+  content::WebContents* web_contents =
+      content::WebContents::FromJavaWebContents(java_web_contents);
+  scoped_refptr<MediaStreamCaptureIndicator> indicator =
+      MediaCaptureDevicesDispatcher::GetInstance()->
+          GetMediaStreamCaptureIndicator();
+  return indicator->IsCapturingVideo(web_contents);
+}
+