Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1631)

Unified Diff: android_webview/java/src/org/chromium/android_webview/permission/AwPermissionRequest.java

Issue 239793002: Handle media access permission request (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: addressed comments, added unittests Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..173fb1914e81850c53b061d990d6b5cb89d4a1e0
--- /dev/null
+++ b/android_webview/java/src/org/chromium/android_webview/permission/AwPermissionRequest.java
@@ -0,0 +1,76 @@
+// 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 android.util.Log;
+
+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 static String TAG = "AwPermissionRequest";
+
+ private Uri mOrigin;
+ private long mResources;
+
+ // AwPermissionRequest native instance.
+ private long mNativeAwPermissionRequest;
+
+ @CalledByNative
+ private static AwPermissionRequest create(long nativeAwPermissionRequest, String url,
+ long resources) {
+ if (nativeAwPermissionRequest == 0) return null;
+ Uri origin = Uri.parse(url);
+ return new AwPermissionRequest(nativeAwPermissionRequest, origin, resources);
+ }
+
+ private AwPermissionRequest(long nativeAwPermissionRequest, Uri origin,
+ long resources) {
+ mNativeAwPermissionRequest = nativeAwPermissionRequest;
+ mOrigin = origin;
+ mResources = resources;
+ }
+
+ public Uri getOrigin() {
+ return mOrigin;
+ }
+
+ public long getResources() {
+ return mResources;
+ }
+
+ public void grant() {
mkosiba (inactive) 2014/04/23 18:04:40 do we want to enforce the correct thread here or w
michaelbai 2014/04/24 00:56:43 Done.
+ if (mNativeAwPermissionRequest != 0)
+ nativeOnAccept(mNativeAwPermissionRequest, true);
+ destory();
+ }
+
+ public void deny() {
+ if (mNativeAwPermissionRequest != 0)
+ nativeOnAccept(mNativeAwPermissionRequest, false);
+ destory();
+ }
+
+ @CalledByNative
+ private void destory() {
+ mNativeAwPermissionRequest = 0;
+ }
+
+ @Override
+ protected void finalize() throws Throwable {
+ if (mNativeAwPermissionRequest == 0) return;
+ Log.e(TAG, "Neither grant() nor deny() has been called, "
+ + "the permission request is denied by default.");
+ deny();
+ }
+
+ private native void nativeOnAccept(long nativeAwPermissionRequest, boolean allowed);
+}

Powered by Google App Engine
This is Rietveld 408576698