diff options
| author | jamescook@chromium.org <jamescook@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-07-31 12:44:06 +0000 |
|---|---|---|
| committer | jamescook@chromium.org <jamescook@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-07-31 12:44:06 +0000 |
| commit | adf5a10eeb5dc230f27d3b1344f2111bdfcd97d6 (patch) | |
| tree | 8ba5fbfcb3ddaa54355fa3b03cbdcadcbc6d875d /extensions/DEPS | |
| parent | 248ee9be3db637800a1bf7b13817df43f904248b (diff) | |
| download | chromium_src-adf5a10eeb5dc230f27d3b1344f2111bdfcd97d6.zip chromium_src-adf5a10eeb5dc230f27d3b1344f2111bdfcd97d6.tar.gz chromium_src-adf5a10eeb5dc230f27d3b1344f2111bdfcd97d6.tar.bz2 | |
Move extension notifications to extensions/browser/notification_types.h
As part of the extensions module refactor we temporarily allowed the use
of chrome_notification_types.h in src/extensions. However, this allowed
code moving into src/extensions to listen to notifications from Chrome,
which was a layering violation.
This CL moves all the extensions notifications into their own header and
cleans up DEPS to disallow the exception.
BUG=392622
TEST=compiles, unit_tests, browser_tests, extensions_unittests
TBR=miket@chromium.org for apps/
Review URL: https://codereview.chromium.org/425303002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286770 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'extensions/DEPS')
| -rw-r--r-- | extensions/DEPS | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/extensions/DEPS b/extensions/DEPS index f0606ba..fede72f 100644 --- a/extensions/DEPS +++ b/extensions/DEPS @@ -12,13 +12,8 @@ include_rules = [ "+testing", "+ui", - # Temporarily allowed includes as part of the app shell/extensions refactor. - # # NOTE: Please do not add includes without talking to the app shell team; # see OWNERS for this directory. - # - # TODO(jamescook): Remove this. http://crbug.com/392622 - "!chrome/browser/chrome_notification_types.h", ] specific_include_rules = { |