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

Issue 726699 link

Starred by 1 user
Status: Assigned
Owner:
█ raymes@chromium.org
OOO until 4th Feb
Cc:
jochen@chromium.org
hu...@vewd.com
most...@vewd.com
benwells@chromium.org
█ brettw@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task
Team-Security-UX

Blocking:
issue 726700



Sign in to add a comment

//components/content_settings depends on //extensions, possibly violating the layering rule

Reported by most...@opera.com, May 26 2017 
//components/content_settings depends on //extensions which IIUC is specific to the chrome layer.  This appears to violate the layering rule, ie that components are only supposed to depend upon lower layers.

Reference:
https://chromium.googlesource.com/chromium/src/+/master/components/README

Comment 1 by jochen@chromium.org, May 26 2017

Cc: benwells@chromium.org
Components: UI>Browser>SiteSettings

Comment 2 by most...@opera.com, May 26 2017

Blocking: 726700

Comment 3 by jialiul@chromium.org, May 26 2017

Cc: brettw@chromium.org

Comment 4 by benwells@chromium.org, May 28 2017

Owner: raymes@chromium.org

Comment 5 by raymes@chromium.org, May 31 2017

Status: Assigned (was: Untriaged)

Comment 6 by most...@vewd.com, Sep 28 2017

Cc: -hu...@opera.com most...@vewd.com hu...@vewd.com

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

Labels: Hotlist-EnamelAndFriendsFixIt

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

Labels: -Hotlist-EnamelAndFriendsFixIt
►

Sign in to add a comment