Popular sites on the NTP: Favicons!

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 5a808145ca76498556a03e85b886faf8d55c87d2..7c8a6ae5d827610776fd1868912f0a4c6f9dcda0 100644
--- a/chrome/browser/android/most_visited_sites.cc
+++ b/chrome/browser/android/most_visited_sites.cc
@@ -452,8 +452,21 @@ void MostVisitedSites::OnPopularSitesAvailable(bool success) {
     return;
   }
 
-  if (!observer_.is_null())
-    QueryMostVisitedURLs();
+  if (observer_.is_null())
+    return;
+
+  std::vector<std::string> urls;
+  std::vector<std::string> favicon_urls;
+  for (const PopularSites::Site& popular_site : popular_sites_->sites()) {
+    urls.push_back(popular_site.url.spec());
+    favicon_urls.push_back(popular_site.favicon_url.spec());
+  }
+  JNIEnv* env = AttachCurrentThread();
+  Java_MostVisitedURLsObserver_onPopularURLsAvailable(
+      env, observer_.obj(), ToJavaArrayOfStrings(env, urls).obj(),
+      ToJavaArrayOfStrings(env, favicon_urls).obj());
+
+  QueryMostVisitedURLs();
 }
 
 void MostVisitedSites::RecordUMAMetrics() {