diff options
author | tapted <tapted@chromium.org> | 2014-10-02 16:45:52 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-10-02 23:46:03 +0000 |
commit | 344272ddd4ca47fb56fd6a1ca6a4f1418df2c426 (patch) | |
tree | a711d865e73a7a38e3b07b3c28f486671367a5a2 /extensions/DEPS | |
parent | c4c7e842b3ba77671b74ac263e5fb7217c3dfbfb (diff) | |
download | chromium_src-344272ddd4ca47fb56fd6a1ca6a4f1418df2c426.zip chromium_src-344272ddd4ca47fb56fd6a1ca6a4f1418df2c426.tar.gz chromium_src-344272ddd4ca47fb56fd6a1ca6a4f1418df2c426.tar.bz2 |
Move components/native_app_window to extensions/components/native_app_window
There's currently a dependency cycle between gyp files: components.gyp
and extensions.gyp. extensions.gyp depends on a number of components,
and two components depend on exetensions.gyp. These are:
- components/renderer_context_menu/
- components/native_app_window/ (more recently)
renderer_context_menu has an "optional" extensions dependency, which can
be skipped when ENABLE_EXTENSIONS is not defined. Still, it is necessary
for renderer_context_menu.gyp to omit its extensions.gyp dependency and
build as a static library.
For native_app_window, since it is not compiled on Mac, the gyp circular
check is not performed on the bots.
This CL fixes the layering by adding a folder,
src/extensions/components. Things here may depend on extensions, but not
on other extensions/components folders in a way that creates a cycle.
This also allows the benefits of a component-architecture to make sense
of interdependencies between the ~1337 files under src/extensions.
This layout also makes it clear that by depending on one of these
components, a target is also depending on src/extensions. Currently,
this is not clear.
BUG=35878, 418455
TBR=reed@google.com
Review URL: https://codereview.chromium.org/606953002
Cr-Commit-Position: refs/heads/master@{#297952}
Diffstat (limited to 'extensions/DEPS')
-rw-r--r-- | extensions/DEPS | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/extensions/DEPS b/extensions/DEPS index 2286ed4..d2bbec8 100644 --- a/extensions/DEPS +++ b/extensions/DEPS @@ -7,6 +7,7 @@ include_rules = [ "+content/public/common", "+content/public/test", "+crypto", + "-extensions/components", "+extensions/test", "+grit/extensions_renderer_resources.h", "+grit/extensions_resources.h", |