New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 593510 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Feature
Team-Security-UX



Sign in to add a comment

Merge ChooserContext with PermissionContext

Project Member Reported by raymes@chromium.org, Mar 9 2016

Issue description

We should consider merging ChooserContext and PermissionContext so that we have the same codepaths for permission logic (to minimise duplication, avoid policy violations, etc.)

jyasskin: were you still interested in looking at this?
 
Cc: tsergeant@chromium.org
Components: -Permissions Internals>Permissions
Yes, I'm still interested in looking at this. I'm working on the Permissions API update first, at https://github.com/w3c/permissions/pull/66, so that we have a better idea what the PermissionContext API ought to be, before I start refactoring it.

Comment 4 by raymes@chromium.org, Mar 14 2016

Great, thanks!
Components: Internals>Permissions>Model

Comment 6 by raymes@chromium.org, Nov 29 2016

Components: -Internals>Permissions
Owner: ----

Comment 8 by est...@chromium.org, Nov 10 2017

Labels: Hotlist-EnamelAndFriendsFixIt

Comment 9 by est...@chromium.org, Feb 18 2018

Labels: -Hotlist-EnamelAndFriendsFixIt
Labels: -Type-Bug Type-Feature
Raymes, would you still encourage us to make this change?
Yes I think it's still something worth investigating but not urgent. The main reason would be to ensure that we don't duplicate checks between choosers and regular permissions and that both are subject to the same constraints.

Sign in to add a comment