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

Issue 842886 link

Starred by 3 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 696650



Sign in to add a comment

Feature: Provide Read/Write versions of contentSettings extension permission

Project Member Reported by tsepez@chromium.org, May 14 2018

Issue description

Currently, for an extension to query content settings, it requires the same permission as allows it to modify content settings. This grants too much power to an extension that simply wants to change its behaviour based on a content setting.
 

Comment 1 by tsepez@chromium.org, May 21 2018

Blocking: 696650

Comment 2 by tsepez@chromium.org, May 21 2018

Cc: thestig@chromium.org
Owner: rdevlin....@chromium.org
Status: Assigned (was: Untriaged)
Devlin, could you comment on this or re-assign as appropriate?  If it is a quick fix, we might want to hold off on landing the blocking bug until this gets resolved.  Thanks.
Cc: rdevlin....@chromium.org raymes@chromium.org
Owner: ----
Status: Available (was: Assigned)
From a high level, I'm not opposed to this, and can see the potential benefit (though I doubt it will be widely used).  However, it won't be a trivial change, and it's probably not something that I (or anyone on my team) will have the bandwidth to handle in the near future.

Also +raymes@ for content settings expertise, in case he has opinions.

Comment 4 by raymes@chromium.org, May 22 2018

I think separating permissions would be a reasonable step. 

I've expressed this before but I think that we should actually deprecate this API and have a more thought-out API. The API is very complicated with subtle semantics in how patterns interact with other patterns and so on. It also doesn't work well for sensitive permissions.

Sign in to add a comment