Add ContextualSuggestionsService to Omnibox

components/omnibox/browser/DEPS

Index: components/omnibox/browser/DEPS
diff --git a/components/omnibox/browser/DEPS b/components/omnibox/browser/DEPS
index 8403b4f979607b9215a0c76ef6c66e1444564897..12eebed0f81586a43a3ef69868d15d09e86f3adc 100644
--- a/components/omnibox/browser/DEPS
+++ b/components/omnibox/browser/DEPS
@@ -1,6 +1,7 @@
 include_rules = [
   "+components/bookmarks/browser",
   "+components/bookmarks/test",
+  "+components/contextual_suggestions",

[Mark P, 2017/07/17 23:36:12]

This isn't necessary anymore.

   "+components/data_use_measurement/core",
   "+components/grit/components_scaled_resources.h",
   "+components/history/core/browser",
@@ -15,6 +16,7 @@ include_rules = [
   "+components/search",
   "+components/search_engines",
   "+components/sessions",
+  "+components/signin/core/browser",

[Mark P, 2017/07/17 23:36:12]

I'm not sure it's okay to include components/.../browser stuffy in this
non-browser component.  I think it is because this component is basically only
building a "browser" library plus some tests, but I'd like someone who
understands what layers we're supposed to have to comment.

[blundell, 2017/07/18 15:21:14]

This should be absolutely fine unless there's something I'm missing. Why do you
call //components/omnibox/browser a non-browser component?

[Mark P, 2017/07/18 19:11:28]

No, you're not missing something.  I'm missing something ... :-)

(I missed the BROWSER part of the filename for components/omnibox/browser/DEPS.)

   "+components/strings/grit/components_strings.h",
   "+components/toolbar",
   "+components/url_formatter",