[Content suggestions] Reveal URL with favicon in snippets-internals

chrome/browser/ui/webui/snippets_internals_message_handler.cc

Index: chrome/browser/ui/webui/snippets_internals_message_handler.cc
diff --git a/chrome/browser/ui/webui/snippets_internals_message_handler.cc b/chrome/browser/ui/webui/snippets_internals_message_handler.cc
index 9b7798098caa38115c50019ad18ed5fc5be22f83..08781a31e8888f4af656f3d1be3c2c0bb218ac73 100644
--- a/chrome/browser/ui/webui/snippets_internals_message_handler.cc
+++ b/chrome/browser/ui/webui/snippets_internals_message_handler.cc
@@ -53,6 +53,8 @@ std::unique_ptr<base::DictionaryValue> PrepareSuggestion(
   auto entry = base::MakeUnique<base::DictionaryValue>();
   entry->SetString("idWithinCategory", suggestion.id().id_within_category());
   entry->SetString("url", suggestion.url().spec());
+  entry->SetString("urlWithFavicon",
+                   suggestion.url_with_favicon().GetWithEmptyPath().spec());

[vitaliii, 2017/04/19 09:02:10]

Why do you remove path?

[jkrcal, 2017/04/19 10:26:48]

To do the same what is done in ContentSuggestionsService. I think this is not
worth introducing another ForDebugging method in the service.

[vitaliii, 2017/04/19 10:47:44]

Do you mean that ContentSuggestionsService ignores path (I could not see
anything related in CS)?
If yes, then you duplicating the logic here and it will become obsolete and
confusing when ContentSuggestionsService starts doing something else. 

Have you considered ignoring path inside url_with_favicon()?

[jkrcal, 2017/04/19 11:44:07]

Okay, done.

   entry->SetString("title", suggestion.title());
   entry->SetString("snippetText", suggestion.snippet_text());
   entry->SetString("publishDate",