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/08 15:18:08]

This feels like the reader should be in some common code then, no?!

This should at least be a TODO to make both the chrome and the utility DLL
depend on the bookmarks component (see comment at top of this file) once it's
here.

[scottmg, 2013/07/09 16:20:49]

The problem is that it depends on blink to reencode favicons, so can't go in
common or a component.

The reader isn't used outside of the importer other than
bookmarks_html_writer_unittest (where it's used to check that the bookmarks
roundtrip).
 
I think the "html reader for bookmarks" is reasonable to be distinct from the
bookmarks component. I'm not sure what the right resolution is, but perhaps
there should be a bookmarks_io component or something. The dependency on blink
is the difficult part to resolve though.

+
   # 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.