From a8cf61b1bd02bf47aca467f64f9ca91916252f75 Mon Sep 17 00:00:00 2001 From: "stoyan@chromium.org" Date: Thu, 5 Feb 2009 17:41:38 +0000 Subject: Fix the dependency checker tool. Rules for a directory did modify their directory parent rules. Using copy.copy() solves the problem.Additional fix when "allow-current-directory" rules was not applied if DEPS file is missing. Updated few DEPS file with reasonable rules. To prevent tree closing other dependencies are added. These need to be either legitimated or dependency removed. Review URL: http://codereview.chromium.org/21025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@9221 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/DEPS | 7 +++++++ chrome/common/DEPS | 18 +++++++++++++++--- chrome/common/gfx/DEPS | 4 ++++ chrome/installer/setup/DEPS | 3 +++ chrome/plugin/DEPS | 9 +++++++++ chrome/renderer/DEPS | 15 ++++++++++++++- chrome/test/DEPS | 3 +++ chrome/third_party/hunspell/google/DEPS | 3 +++ chrome/tools/DEPS | 5 +++++ chrome/views/DEPS | 9 +++++++++ 10 files changed, 72 insertions(+), 4 deletions(-) create mode 100644 chrome/common/gfx/DEPS create mode 100644 chrome/installer/setup/DEPS create mode 100644 chrome/plugin/DEPS create mode 100644 chrome/third_party/hunspell/google/DEPS create mode 100644 chrome/tools/DEPS create mode 100644 chrome/views/DEPS (limited to 'chrome') diff --git a/chrome/browser/DEPS b/chrome/browser/DEPS index f662673..cb177e3 100755 --- a/chrome/browser/DEPS +++ b/chrome/browser/DEPS @@ -1,8 +1,15 @@ include_rules = [ + "+chrome/app", + "+chrome/app/theme", # For resource identifiers + "+chrome/app/locales", "+chrome/installer", + "+chrome/personalization", "+chrome/tools", "+chrome/tools/profiles", # For history unit tests. "+chrome/views", + "+sandbox/src", + "+skia/include", + "+skia/ext", "+webkit/activex_shim", "+webkit/default_plugin", "+webkit/glue", # Defines some types that are marshalled over IPC. diff --git a/chrome/common/DEPS b/chrome/common/DEPS index fdd760a..3fd1671 100644 --- a/chrome/common/DEPS +++ b/chrome/common/DEPS @@ -1,8 +1,20 @@ include_rules = [ "+chrome/plugin", # For checking whether we're a plugin process. - + "+libxml", + "+sandbox/src", + "+skia/include", + "+webkit/glue", + # Other libraries. "+third_party/bzip2", "+third_party/npapi", -] - + "+third_party/sqlite", + "+third_party/zlib", + + # FIXME - refactor code and remove these dependencies + "+chrome/app", + "+chrome/browser", + "+chrome/browser/net", + "+chrome/renderer", + "+chrome/views", +] \ No newline at end of file diff --git a/chrome/common/gfx/DEPS b/chrome/common/gfx/DEPS new file mode 100644 index 0000000..88f24d3 --- /dev/null +++ b/chrome/common/gfx/DEPS @@ -0,0 +1,4 @@ +include_rules = [ + "+skia/include", + "+skia/ext", +]; \ No newline at end of file diff --git a/chrome/installer/setup/DEPS b/chrome/installer/setup/DEPS new file mode 100644 index 0000000..ab2a48a --- /dev/null +++ b/chrome/installer/setup/DEPS @@ -0,0 +1,3 @@ +include_rules = [ + "+chrome/app", +] diff --git a/chrome/plugin/DEPS b/chrome/plugin/DEPS new file mode 100644 index 0000000..0d9d3a6 --- /dev/null +++ b/chrome/plugin/DEPS @@ -0,0 +1,9 @@ +include_rules = [ + "+chrome/renderer", + "+sandbox/src", + "+skia/ext", + "+third_party/npapi", + "+webkit/glue", + "+webkit/glue/plugins", +] + diff --git a/chrome/renderer/DEPS b/chrome/renderer/DEPS index a9c8e70..d533df4 100644 --- a/chrome/renderer/DEPS +++ b/chrome/renderer/DEPS @@ -1,5 +1,18 @@ include_rules = [ + "+chrome/app", # The plugins use the theme library to get the sad plugin bitmap. + "+chrome/personalization", + "+chrome/plugin", "+media/base", # For HTML5 media rendering. - "+webkit/port", # For certain drawing, would be nice to not have. + "+sandbox/src", + "+skia/ext", + "+skia/include", + "+webkit/default_plugin", + "+webkit/glue", + "+webkit/glue/plugins", + + # FIXME - refactor code and remove these dependencies + "+chrome/app/theme", + "+chrome/browser/net", + "+chrome/views", ] diff --git a/chrome/test/DEPS b/chrome/test/DEPS index 70906cf..f6a76e9 100644 --- a/chrome/test/DEPS +++ b/chrome/test/DEPS @@ -1,4 +1,7 @@ include_rules = [ # The test directory can do whatever it wants in chrome. "+chrome", + "+sandbox/src", + "+sandbox/tests", + "+webkit/glue", ] diff --git a/chrome/third_party/hunspell/google/DEPS b/chrome/third_party/hunspell/google/DEPS new file mode 100644 index 0000000..ab03e0d4 --- /dev/null +++ b/chrome/third_party/hunspell/google/DEPS @@ -0,0 +1,3 @@ +include_rules = [ + "+chrome/tools/convert_dict", +] diff --git a/chrome/tools/DEPS b/chrome/tools/DEPS new file mode 100644 index 0000000..376e9fc --- /dev/null +++ b/chrome/tools/DEPS @@ -0,0 +1,5 @@ +include_rules = [ + "+chrome/browser/history", + "+chrome/third_party/hunspell/google", + "+skia/ext", +] diff --git a/chrome/views/DEPS b/chrome/views/DEPS new file mode 100644 index 0000000..7256113 --- /dev/null +++ b/chrome/views/DEPS @@ -0,0 +1,9 @@ +include_rules = [ + "+chrome/app", + "+chrome/app/theme", + "+chrome/browser", + "+chrome/browser/views", + "+skia/ext", + "+skia/include", + "+webkit/glue", +] -- cgit v1.1