Commit bf1c79c8 authored by Raymes Khoury's avatar Raymes Khoury Committed by Commit Bot

Remove set no-parent from per-file PermissionContext OWNERS

In https://codereview.chromium.org/2969393002 per-file OWNERS were added
for PermissionContextBase subclasses, as they would benefit from
approval by permissions owners.

In https://codereview.chromium.org/2791623004/#msg148 jochen raised the
concern that set no-parent would interfere with large-scale
refactorings. Remove set no-parent declarations for now until we figure
out what the best way forward is.

Change-Id: I13b0fdec6b259e5db3bebf9dda425497b885af6f
Tbr: jochen@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/598662
Commit-Queue: Raymes Khoury <raymes@chromium.org>
Reviewed-by: default avatarTimothy Loh <timloh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#491668}
parent 44e59041
iclelland@chromium.org iclelland@chromium.org
jkarlin@chromium.org jkarlin@chromium.org
per-file *permission_context*=set noparent
per-file *permission_context*=file://chrome/browser/permissions/PERMISSIONS_OWNERS per-file *permission_context*=file://chrome/browser/permissions/PERMISSIONS_OWNERS
# COMPONENT: Blink>BackgroundSync # COMPONENT: Blink>BackgroundSync
file://services/device/generic_sensor/OWNERS file://services/device/generic_sensor/OWNERS
per-file *permission_context*=set noparent
per-file *permission_context*=file://chrome/browser/permissions/PERMISSIONS_OWNERS per-file *permission_context*=file://chrome/browser/permissions/PERMISSIONS_OWNERS
# COMPONENT: Blink>Sensor # COMPONENT: Blink>Sensor
......
...@@ -2,7 +2,6 @@ benwells@chromium.org ...@@ -2,7 +2,6 @@ benwells@chromium.org
raymes@chromium.org raymes@chromium.org
timvolodine@chromium.org timvolodine@chromium.org
per-file *permission_context*=set noparent
per-file *permission_context*=file://chrome/browser/permissions/PERMISSIONS_OWNERS per-file *permission_context*=file://chrome/browser/permissions/PERMISSIONS_OWNERS
# COMPONENT: Internals>Permissions # COMPONENT: Internals>Permissions
...@@ -13,7 +13,6 @@ per-file media_engagement*=mlamouri@chromium.org ...@@ -13,7 +13,6 @@ per-file media_engagement*=mlamouri@chromium.org
per-file *.mojom=set noparent per-file *.mojom=set noparent
per-file *.mojom=file://ipc/SECURITY_OWNERS per-file *.mojom=file://ipc/SECURITY_OWNERS
per-file *permission_context*=set noparent
per-file *permission_context*=file://chrome/browser/permissions/PERMISSIONS_OWNERS per-file *permission_context*=file://chrome/browser/permissions/PERMISSIONS_OWNERS
# COMPONENT: Blink>Media # COMPONENT: Blink>Media
...@@ -15,7 +15,6 @@ per-file media_permission*=raymes@chromium.org ...@@ -15,7 +15,6 @@ per-file media_permission*=raymes@chromium.org
# For WebRTC desktop capturer related changes only # For WebRTC desktop capturer related changes only
per-file native_desktop_media_list*=zijiehe@chromium.org per-file native_desktop_media_list*=zijiehe@chromium.org
per-file *permission_context*=set noparent
per-file *permission_context*=file://chrome/browser/permissions/PERMISSIONS_OWNERS per-file *permission_context*=file://chrome/browser/permissions/PERMISSIONS_OWNERS
# COMPONENT: Blink>WebRTC # COMPONENT: Blink>WebRTC
...@@ -8,7 +8,6 @@ yoshiki@chromium.org ...@@ -8,7 +8,6 @@ yoshiki@chromium.org
# Mac files # Mac files
per-file *_mac*=rsesek@chromium.org per-file *_mac*=rsesek@chromium.org
per-file *permission_context*=set noparent
per-file *permission_context*=file://chrome/browser/permissions/PERMISSIONS_OWNERS per-file *permission_context*=file://chrome/browser/permissions/PERMISSIONS_OWNERS
# TEAM: platform-capabilities@chromium.org # TEAM: platform-capabilities@chromium.org
......
...@@ -3,5 +3,4 @@ bauerb@chromium.org ...@@ -3,5 +3,4 @@ bauerb@chromium.org
per-file flash*=tommycli@chromium.org per-file flash*=tommycli@chromium.org
per-file plugin_power_saver*=tommycli@chromium.org per-file plugin_power_saver*=tommycli@chromium.org
per-file *permission_context*=set noparent
per-file *permission_context*=file://chrome/browser/permissions/PERMISSIONS_OWNERS per-file *permission_context*=file://chrome/browser/permissions/PERMISSIONS_OWNERS
...@@ -4,7 +4,6 @@ michaeln@chromium.org ...@@ -4,7 +4,6 @@ michaeln@chromium.org
per-file storage_info_fetcher.*=finnur@chromium.org per-file storage_info_fetcher.*=finnur@chromium.org
per-file *permission_context*=set noparent
per-file *permission_context*=file://chrome/browser/permissions/PERMISSIONS_OWNERS per-file *permission_context*=file://chrome/browser/permissions/PERMISSIONS_OWNERS
# TEAM: storage-dev@chromium.org # TEAM: storage-dev@chromium.org
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment