summaryrefslogtreecommitdiffstats
path: root/extensions/DEPS
diff options
context:
space:
mode:
authortapted <tapted@chromium.org>2014-10-02 16:45:52 -0700
committerCommit bot <commit-bot@chromium.org>2014-10-02 23:46:03 +0000
commit344272ddd4ca47fb56fd6a1ca6a4f1418df2c426 (patch)
treea711d865e73a7a38e3b07b3c28f486671367a5a2 /extensions/DEPS
parentc4c7e842b3ba77671b74ac263e5fb7217c3dfbfb (diff)
downloadchromium_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/DEPS1
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",