Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(371)

Unified Diff: webkit/DEPS

Issue 739793004: Cleanup webkit DEPS list. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | webkit/common/DEPS » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/DEPS
diff --git a/webkit/DEPS b/webkit/DEPS
index 2d45bbb05a850285acfb1977aea057cf01490742..e1e03d4ac47fd5d60f7dde1401fbdd156afc671c 100644
--- a/webkit/DEPS
+++ b/webkit/DEPS
@@ -1,46 +1,16 @@
-# checkdeps.py shouldn't check include paths for files in these dirs:
-skip_child_includes = [
- "build",
-]
-
-# Do NOT add chrome to the list below. We shouldn't be including files from
+# Do NOT add chrome to the list below. We shouldn't be including files from
# src/chrome in src/webkit.
include_rules = [
- # For bridge/c/c_utility.h in npruntime_util.cc
- "+bridge",
- "+cc",
- "-cc/surfaces",
- "+grit", # For generated headers
- "+skia",
- "+third_party/angle",
- "+third_party/hyphen",
- "+third_party/leveldatabase",
- "+third_party/skia",
- "+third_party/sqlite",
- "+third_party/tcmalloc",
- "+third_party/WebKit/public/platform",
- "+third_party/WebKit/public/web",
- "+v8",
-
- # For databases/
- "+sql",
- "+storage",
-
- # For gpu/
danakj 2014/11/20 17:39:39 is this comment wrong now?
tfarina 2014/11/20 17:46:05 well, that is all it is left now: webkit/common/gp
danakj 2014/11/20 17:48:12 Ah, I see. LGTM
- "+ui/gfx",
- "+ui/gl",
+ "+cc/output",
"+gpu/GLES2",
"+gpu/command_buffer/client",
"+gpu/command_buffer/common",
"+gpu/command_buffer/service",
"+gpu/skia_bindings",
+ "+skia",
+ "+third_party/WebKit/public/platform",
"+third_party/khronos/GLES2",
-
- # TODO(brettw) - review these; move up if it's ok, or remove the dependency
- "+net",
- "+third_party/npapi/bindings",
-
- # webkit/support is only test support code, nothing outside of webkit/support
- # should pull it in.
- "-webkit/support",
+ "+third_party/skia",
+ "+ui/gfx",
+ "+ui/gl",
]
« no previous file with comments | « no previous file | webkit/common/DEPS » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698