Handle media access permission request

android_webview/native/aw_contents.cc

Index: android_webview/native/aw_contents.cc
diff --git a/android_webview/native/aw_contents.cc b/android_webview/native/aw_contents.cc
index bde37b6821be0dd25df59896982b98011d88dfa8..762c736db0904f3686fb0cbb617a7295290e9250 100644
--- a/android_webview/native/aw_contents.cc
+++ b/android_webview/native/aw_contents.cc
@@ -24,6 +24,7 @@
 #include "android_webview/native/aw_picture.h"
 #include "android_webview/native/aw_web_contents_delegate.h"
 #include "android_webview/native/java_browser_view_renderer_helper.h"
+#include "android_webview/native/permission/permission_request_handler.h"
 #include "android_webview/native/state_serializer.h"
 #include "android_webview/public/browser/draw_gl.h"
 #include "base/android/jni_android.h"
@@ -204,6 +205,8 @@ AwContents::AwContents(scoped_ptr<WebContents> web_contents)
   render_view_host_ext_.reset(
       new AwRenderViewHostExt(this, web_contents_.get()));
 
+  permission_request_handler_.reset(new PermissionRequestHandler(this));
+
   AwAutofillManagerDelegate* autofill_manager_delegate =
       AwAutofillManagerDelegate::FromWebContents(web_contents_.get());
   if (autofill_manager_delegate)
@@ -577,6 +580,18 @@ void AwContents::HideGeolocationPrompt(const GURL& origin) {
   }
 }
 
+void AwContents::OnPermissionRequest(ScopedJavaLocalRef<jobject> obj) {

[benm (inactive), 2014/04/16 14:51:10]

can we pass the native type into this function, and convert to a java ref here?
Same for OnPermissionRequestCanceled

[michaelbai, 2014/04/22 20:53:51]

I were thinking passing in AwPermissionRequest requires AwContents know the
detail of AwPermissionRequest, but it seemed that using java object here isn't
good for testing.

On 2014/04/16 14:51:10, benm wrote:

+    JNIEnv* env = AttachCurrentThread();
+    ScopedJavaLocalRef<jobject> j_ref = java_ref_.get(env);
+    Java_AwContents_onPermissionRequest(env, j_ref.obj(), obj.obj());
+}
+
+void AwContents::OnPermissionRequestCanceled(ScopedJavaLocalRef<jobject> obj) {
+    JNIEnv* env = AttachCurrentThread();
+    ScopedJavaLocalRef<jobject> j_ref = java_ref_.get(env);
+    Java_AwContents_onPermissionRequestCanceled(env, j_ref.obj(), obj.obj());
+}
+
 void AwContents::FindAllAsync(JNIEnv* env, jobject obj, jstring search_string) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   GetFindHelper()->FindAllAsync(ConvertJavaStringToUTF16(env, search_string));