Handle media access permission request

android_webview/java/src/org/chromium/android_webview/permission/AwPermissionRequest.java

Index: android_webview/java/src/org/chromium/android_webview/permission/AwPermissionRequest.java
diff --git a/android_webview/java/src/org/chromium/android_webview/permission/AwPermissionRequest.java b/android_webview/java/src/org/chromium/android_webview/permission/AwPermissionRequest.java
new file mode 100644
index 0000000000000000000000000000000000000000..98acde44adfdffe5966711d1818389e6ebe2203a
--- /dev/null
+++ b/android_webview/java/src/org/chromium/android_webview/permission/AwPermissionRequest.java
@@ -0,0 +1,64 @@
+// Copyright 2014 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.
+
+package org.chromium.android_webview.permission;
+
+import android.net.Uri;
+
+import org.chromium.base.CalledByNative;
+import org.chromium.base.JNINamespace;
+
+/**
+ * This class wraps permission request in Chromium side, and can only be created
+ * by native side.
+ */
+@JNINamespace("android_webview")
+public class AwPermissionRequest {
+    private Uri mOrigin;
+    private long mResources;
+
+    // AwPermissionRequest native instance.
+    private long mNativeAwPermissionRequest;
+
+    @CalledByNative
+    public static AwPermissionRequest create(long nativeAwPermissionRequest) {
+        if (nativeAwPermissionRequest == 0) return null;
+        return new AwPermissionRequest(nativeAwPermissionRequest);
+    }
+
+    private AwPermissionRequest(long nativeAwPermissionRequest) {
+        mNativeAwPermissionRequest = nativeAwPermissionRequest;
+        mOrigin = Uri.parse(nativeGetOrigin(mNativeAwPermissionRequest));

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

Is it better to pass these values from native into the factory function, to
avoid hopping into and out of native here?

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

Thanks

+        mResources = nativeGetResources(mNativeAwPermissionRequest);
+    }
+
+    public Uri getOrigin() {
+        return mOrigin;
+    }
+
+    public long getResources() {
+        return mResources;
+    }
+
+    public void grant() {
+        if (mNativeAwPermissionRequest != 0)
+            nativeOnAccept(mNativeAwPermissionRequest, true);
+        destory();
+    }
+
+    public void deny() {
+        if (mNativeAwPermissionRequest != 0)
+            nativeOnAccept(mNativeAwPermissionRequest, false);
+        destory();
+    }
+
+    @CalledByNative
+    private void destory() {

[mkosiba (inactive), 2014/04/17 10:34:06]

probably a good idea to have a finalizer for this. It could just throw an
exception or log an error to logcat and call deny.

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

Right, thanks

On 2014/04/17 10:34:06, mkosiba wrote:

+        mNativeAwPermissionRequest = 0;
+    }
+
+    private native void nativeOnAccept(long nativeAwPermissionRequest, boolean allowed);
+    private native String nativeGetOrigin(long nativeAwPermissionRequest);
+    private native long nativeGetResources(long nativeAwPermissionRequest);
+}