Add favicon and publisher name to snippet cards

chrome/browser/android/ntp/ntp_snippets_bridge.cc

Index: chrome/browser/android/ntp/ntp_snippets_bridge.cc
diff --git a/chrome/browser/android/ntp/ntp_snippets_bridge.cc b/chrome/browser/android/ntp/ntp_snippets_bridge.cc
index 569b3c7ad2a00554b89b7d5e38618b69b345efed..d3cccaeb29358a8766d604e01b378a102c07531d 100644
--- a/chrome/browser/android/ntp/ntp_snippets_bridge.cc
+++ b/chrome/browser/android/ntp/ntp_snippets_bridge.cc
@@ -112,24 +112,35 @@ void NTPSnippetsBridge::NTPSnippetsServiceLoaded() {
 
   std::vector<std::string> titles;
   std::vector<std::string> urls;
+  std::vector<std::string> amp_urls;
   std::vector<std::string> thumbnail_urls;
   std::vector<std::string> snippets;
   std::vector<int64_t> timestamps;
+  std::vector<std::string> publishers;
   for (const ntp_snippets::NTPSnippet& snippet : *ntp_snippets_service_) {
+    // The url from source_info is a url for a site that is one of the
+    // HOST_RESTRICT parameters, so this is preferred. If there's an AMP url,
+    // prefer that over the plain url
+    urls.push_back(snippet.best_source().url.spec());
+    amp_urls.push_back(snippet.best_source().amp_url.is_empty()
+                           ? snippet.best_source().url.spec()

[Marc Treib, 2016/04/28 08:50:13]

Wait, if we don't have an AMP URL, then shouldn't the amp_urls entry stay empty?

[May, 2016/04/28 18:01:52]

Ah, yeah it should. Good catch.

+                           : snippet.best_source().amp_url.spec());
     titles.push_back(snippet.title());
-    urls.push_back(snippet.url().spec());
     thumbnail_urls.push_back(snippet.salient_image_url().spec());
     snippets.push_back(snippet.snippet());
     timestamps.push_back(snippet.publish_date().ToJavaTime());
+    publishers.push_back(snippet.best_source().publisher_name);
   }
 
   JNIEnv* env = base::android::AttachCurrentThread();
   Java_SnippetsBridge_onSnippetsAvailable(
       env, observer_.obj(), ToJavaArrayOfStrings(env, titles).obj(),
       ToJavaArrayOfStrings(env, urls).obj(),
+      ToJavaArrayOfStrings(env, amp_urls).obj(),
       ToJavaArrayOfStrings(env, thumbnail_urls).obj(),
       ToJavaArrayOfStrings(env, snippets).obj(),
-      ToJavaLongArray(env, timestamps).obj());
+      ToJavaLongArray(env, timestamps).obj(),
+      ToJavaArrayOfStrings(env, publishers).obj());
 }
 
 void NTPSnippetsBridge::NTPSnippetsServiceShutdown() {