Move most importer code to chrome/utility/importer

chrome/browser/bookmarks/DEPS

Index: chrome/browser/bookmarks/DEPS
diff --git a/chrome/browser/bookmarks/DEPS b/chrome/browser/bookmarks/DEPS
index 0086137cef380e7f2e14e4e7647be39f220b6887..b73c159ed07a51fc828697375a39faa082f77480 100644
--- a/chrome/browser/bookmarks/DEPS
+++ b/chrome/browser/bookmarks/DEPS
@@ -7,6 +7,9 @@ include_rules = [
   "+chrome/browser/bookmarks",
   "+chrome/browser/favicon",
 
+  # For unit tests.
+  "+chrome/utility/importer/bookmark_html_reader.h",

[gab, 2013/07/10 13:40:44]

I still don't like this as it allows anyone in bookmarks/ to include the reader
which is wrong.

Perhaps make this a '!' rule (this is a warning if my previous assumptions are
correct?)?

Otherwise we could add a c/b/bookmarks/test directory, but that might be
overkill for a single test..?

[tfarina, 2013/07/10 13:46:55]

you can use specific_include_rules for this.
see a good example of how to do this for a single unittest that needs this
include like here: components/autofill/content/browser/DEPS

[scottmg, 2013/07/15 16:06:01]

Thanks Thiago, changed to a specific_include_rules.

+
   # TODO(kaiwang): Bring this list to zero.
   # Do not add to the list of temporarily-allowed dependencies below,
   # and please do not introduce more #includes of these files.