diff options
| author | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-26 21:21:22 +0000 |
|---|---|---|
| committer | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-26 21:21:22 +0000 |
| commit | 4432769ac309c0c14eeed2769cbbfabef7f9e59a (patch) | |
| tree | d91e06840aa03f0c412077da168be4c0e8b7c682 /content/browser/renderer_host/DEPS | |
| parent | fef2673b5a02496db07a537143b90a81e6efd055 (diff) | |
| download | chromium_src-4432769ac309c0c14eeed2769cbbfabef7f9e59a.zip chromium_src-4432769ac309c0c14eeed2769cbbfabef7f9e59a.tar.gz chromium_src-4432769ac309c0c14eeed2769cbbfabef7f9e59a.tar.bz2 | |
Fix layering violation in content\browser\renderer_host. That directory shouldn't be including web_contents.
It looks like this regressed because we forgot to update the DEPS exclusion after the tab_contents->web_contents rename. I've fixed the non test case, added exclusion for tests, and put a temporary workaround for the media subdirectory.
It seems that the media subdirectory should move to content\browser since it has dependencies on web_contents.
Review URL: https://codereview.chromium.org/12319119
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@184735 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/browser/renderer_host/DEPS')
| -rw-r--r-- | content/browser/renderer_host/DEPS | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/content/browser/renderer_host/DEPS b/content/browser/renderer_host/DEPS index 2f890a9..2d80d38 100644 --- a/content/browser/renderer_host/DEPS +++ b/content/browser/renderer_host/DEPS @@ -8,5 +8,11 @@ include_rules = [ # The renderer_host files should only call upwards in the layering via the # delegate interfaces. - "-content/browser/tab_contents", + "-content/browser/web_contents", ] + +specific_include_rules = { + ".*_(unit|browser)test\.cc": [ + "+content/browser/web_contents", + ], +} |