Use whitelist large icon for corresponding most visited suggestions.

chrome/browser/android/most_visited_sites.cc

Index: chrome/browser/android/most_visited_sites.cc
diff --git a/chrome/browser/android/most_visited_sites.cc b/chrome/browser/android/most_visited_sites.cc
index 5aeceb28b788e31fd580548aaba28adf78bcf74c..31708f3d3a6a305bfcc77ae762c7899aa5f26d6b 100644
--- a/chrome/browser/android/most_visited_sites.cc
+++ b/chrome/browser/android/most_visited_sites.cc
@@ -458,11 +458,23 @@ void MostVisitedSites::InitiateTopSitesQuery() {
       false);
 }
 
+base::FilePath MostVisitedSites::GetWhitelistLargeIconPath(const GURL& url) {
+  SupervisedUserService* supervised_user_service =
+      SupervisedUserServiceFactory::GetForProfile(profile_);
+
+  for (const auto& whitelist : supervised_user_service->whitelists()) {
+    if (whitelist->entry_point().GetContent() == url.GetContent())

[Bernhard Bauer, 2016/03/17 15:49:00]

Hm... content is everything after the scheme, so including query parameters and
fragment identifier, but _not_ the scheme... Is that what we want?

(I mean, the question of when two URLs are the "same" is surprisingly
complicated, so we might have to compromise in some form anyway, but I'd like to
at least do that consciously.)

[atanasova, 2016/03/17 16:02:08]

Marc and I discussed in the previous comments that using host() will not be
appropriate, since we want to use the whitelist icon when it is the same url. I
personally think that using everything after the scheme will be enough to
distinguish the URLs. Since this is a feature that is going to be used by
children who might not be able to fully type out addresses, I think that
different schemes in URLs will still lead to the same source.
However, I am open to suggestions to how we can improve this and we can
definitely discuss how exact do we want to be vs. trying to "anticipate" what
the child meant.

[Marc Treib, 2016/03/17 16:29:02]

I think Bernhard's suggesting to use less than the full GetContent() - we're
probably not interested in query params.

[Bernhard Bauer, 2016/03/17 17:44:30]

Exactly. And in particular, I'd like to explicitly list the things we are
looking at for equality purposes here, so for example host and path?

[atanasova, 2016/03/22 12:21:49]

Changed to use host+path as discussed here and in the email thread.

+      return whitelist->large_icon_path();
+  }
+  return base::FilePath();
+}
+
 void MostVisitedSites::OnMostVisitedURLsAvailable(
     const history::MostVisitedURLList& visited_list) {
   SupervisedUserURLFilter* url_filter =
       SupervisedUserServiceFactory::GetForProfile(profile_)
           ->GetURLFilterForUIThread();
+
   MostVisitedSites::SuggestionsVector suggestions;
   size_t num_tiles =
       std::min(visited_list.size(), static_cast<size_t>(num_sites_));
@@ -481,6 +493,7 @@ void MostVisitedSites::OnMostVisitedURLsAvailable(
     suggestion->title = visited.title;
     suggestion->url = visited.url;
     suggestion->source = TOP_SITES;
+    suggestion->whitelist_icon_path = GetWhitelistLargeIconPath(visited.url);
 
     suggestions.push_back(std::move(suggestion));
   }
@@ -518,6 +531,8 @@ void MostVisitedSites::OnSuggestionsProfileAvailable(
     generated_suggestion->title = base::UTF8ToUTF16(suggestion.title());
     generated_suggestion->url = GURL(suggestion.url());
     generated_suggestion->source = SUGGESTIONS_SERVICE;
+    generated_suggestion->whitelist_icon_path = GetWhitelistLargeIconPath(
+        GURL(suggestion.url()));
     if (suggestion.providers_size() > 0)
       generated_suggestion->provider_index = suggestion.providers(0);