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

Unified Diff: components/sync/engine_impl/DEPS

Issue 2245073003: [Sync] Clean up intra-sync DEPS significantly. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 months 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
Index: components/sync/engine_impl/DEPS
diff --git a/components/sync/engine_impl/DEPS b/components/sync/engine_impl/DEPS
index f0c51dd45129c49227bd72ad1e283189059d87bf..aa8fb64e026a884ac5027a24747ae9af7c4f43e7 100644
--- a/components/sync/engine_impl/DEPS
+++ b/components/sync/engine_impl/DEPS
@@ -1,13 +1,7 @@
include_rules = [
"+components/sync/base",
- # TODO(maxbogue): Move to engine.
- "+components/sync/core/model_type_processor.h",
- # TODO(maxbogue): Move to base.
- "+components/sync/core/non_blocking_sync_common.h",
- # TODO(maxbogue): Move to engine.
- "+components/sync/core/sync_encryption_handler.h",
- # TODO(maxbogue): Move to test.
- "+components/sync/core/test/test_entry_factory.h",
+ # TODO(crbug.com/631271): core should be split into driver/engine/api.
skym 2016/08/15 22:17:53 Should the 'c' in core be capitalized?
maxbogue 2016/08/15 22:35:07 ¯\_(ツ)_/¯ Sure, why not. Done.
+ "+components/sync/core",
"+components/sync/engine",
"+components/sync/protocol",
"+components/sync/sessions",

Powered by Google App Engine
This is Rietveld 408576698