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

Unified Diff: android_webview/browser/aw_permission_manager.cc

Issue 2458453002: [sensors] Add Permission guard to the generic sensor apis.
Patch Set: Created 4 years, 2 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/browser/aw_permission_manager.cc
diff --git a/android_webview/browser/aw_permission_manager.cc b/android_webview/browser/aw_permission_manager.cc
index 12a6cdd1a840377462fddf9adcff9e1515145945..1e402c6ba6d40f36fc1d662c2b0fb74ec1621bb1 100644
--- a/android_webview/browser/aw_permission_manager.cc
+++ b/android_webview/browser/aw_permission_manager.cc
@@ -311,6 +311,12 @@ int AwPermissionManager::RequestPermissions(
base::Bind(&OnRequestResponse, weak_ptr_factory_.GetWeakPtr(),
request_id, permissions[i]));
break;
+ case PermissionType::SENSORS:
+ delegate->RequestSensorPermission(
+ pending_request->requesting_origin,
+ base::Bind(&OnRequestResponse, weak_ptr_factory_.GetWeakPtr(),
+ request_id, permissions[i]));
+ break;
case PermissionType::AUDIO_CAPTURE:
case PermissionType::VIDEO_CAPTURE:
case PermissionType::NOTIFICATIONS:
@@ -449,6 +455,10 @@ void AwPermissionManager::CancelPermissionRequest(int request_id) {
if (delegate)
delegate->CancelMIDISysexPermissionRequests(requesting_origin);
break;
+ case PermissionType::SENSORS:
+ if (delegate)
+ delegate->CancelSensorPermissionRequests(requesting_origin);
+ break;
case PermissionType::NOTIFICATIONS:
case PermissionType::PUSH_MESSAGING:
case PermissionType::DURABLE_STORAGE:

Powered by Google App Engine
This is Rietveld 408576698