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

Unified Diff: android_webview/browser/aw_permission_manager.cc

Issue 2791623004: [sensors][permission] Add new permission types in permission module. (Closed)
Patch Set: Use 1 single permissionName SENSORS, add ContentSettings Created 3 years, 5 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
« no previous file with comments | « no previous file | chrome/browser/BUILD.gn » ('j') | chrome/browser/generic_sensor/OWNERS » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 e6eafa0552eab261fa04272956fcd4b1d1e6d3de..deda4e2e685a4ae595a659b29e6c25628453d096 100644
--- a/android_webview/browser/aw_permission_manager.cc
+++ b/android_webview/browser/aw_permission_manager.cc
@@ -324,6 +324,7 @@ int AwPermissionManager::RequestPermissions(
case PermissionType::PUSH_MESSAGING:
case PermissionType::DURABLE_STORAGE:
case PermissionType::BACKGROUND_SYNC:
+ case PermissionType::SENSORS:
case PermissionType::FLASH:
NOTIMPLEMENTED() << "RequestPermissions is not implemented for "
<< static_cast<int>(permissions[i]);
@@ -462,6 +463,7 @@ void AwPermissionManager::CancelPermissionRequest(int request_id) {
case PermissionType::AUDIO_CAPTURE:
case PermissionType::VIDEO_CAPTURE:
case PermissionType::BACKGROUND_SYNC:
+ case PermissionType::SENSORS:
case PermissionType::FLASH:
NOTIMPLEMENTED() << "CancelPermission not implemented for "
<< static_cast<int>(permission);
« no previous file with comments | « no previous file | chrome/browser/BUILD.gn » ('j') | chrome/browser/generic_sensor/OWNERS » ('J')

Powered by Google App Engine
This is Rietveld 408576698