Popular sites on the NTP: add thumbnails!

chrome/browser/android/most_visited_sites.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/android/most_visited_sites.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_array.h"
 #include "base/android/jni_string.h"
 #include "base/android/scoped_java_ref.h"
@@ skipping 211 matching lines @@
     const GURL& url,
     scoped_ptr<ScopedJavaGlobalRef<jobject>> j_callback,
     scoped_ptr<SkBitmap> bitmap) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   if (!bitmap.get()) {
     // A thumbnail is not locally available for |url|. Make sure it is put in
     // the list to be fetched at the next visit to this site.
     scoped_refptr<TopSites> top_sites(TopSitesFactory::GetForProfile(profile_));
     if (top_sites)
       top_sites->AddForcedURL(url, base::Time::Now());
+    // Also fetch a remote thumbnail if possible. PopularSites or the
+    // SuggestionsService can supply a thumbnail download URL.
     SuggestionsService* suggestions_service =
-        (mv_source_ == SUGGESTIONS_SERVICE)
-            ? SuggestionsServiceFactory::GetForProfile(profile_)
-            : nullptr;
+        SuggestionsServiceFactory::GetForProfile(profile_);
     if (suggestions_service) {
-      return suggestions_service->GetPageThumbnail(
-          url, base::Bind(&MostVisitedSites::OnObtainedThumbnail,
-                          weak_ptr_factory_.GetWeakPtr(), false,
-                          base::Passed(&j_callback)));
+      if (popular_sites_) {
+        const std::vector<PopularSites::Site>& sites = popular_sites_->sites();
+        auto it = std::find_if(sites.begin(), sites.end(),
+                               [&url](const PopularSites::Site& site) {
+                                 return site.url == url;
+                               });
+        if (it != sites.end() && it->thumbnail_url.is_valid()) {
+          return suggestions_service->GetPageThumbnailWithURL(

[Marc Treib, 2015/09/04 13:29:12]

I guess it's a bit hacky to go through the SuggestionsService here, even though
it might not be active. But it has all the thumbnail fetching logic set up
nicely. :)
The alternative would be to pull all of that out of components/suggestions and
put it somewhere else, probably a standalone component? That doesn't seem worth
the trouble, especially if we eventually want to move to ML.

+              url, it->thumbnail_url,
+              base::Bind(&MostVisitedSites::OnObtainedThumbnail,
+                         weak_ptr_factory_.GetWeakPtr(), false,
+                         base::Passed(&j_callback)));
+        }
+      }
+      if (mv_source_ == SUGGESTIONS_SERVICE) {
+        return suggestions_service->GetPageThumbnail(
+            url, base::Bind(&MostVisitedSites::OnObtainedThumbnail,
+                            weak_ptr_factory_.GetWeakPtr(), false,
+                            base::Passed(&j_callback)));
+      }
     }
   }
   OnObtainedThumbnail(true, j_callback.Pass(), url, bitmap.get());
 }
 
 void MostVisitedSites::OnObtainedThumbnail(
     bool is_local_thumbnail,
     scoped_ptr<ScopedJavaGlobalRef<jobject>> j_callback,
     const GURL& url,
     const SkBitmap* bitmap) {
@@ skipping 320 matching lines @@
   }
 }
 
 static jlong Init(JNIEnv* env,
                   const JavaParamRef<jobject>& obj,
                   const JavaParamRef<jobject>& jprofile) {
   MostVisitedSites* most_visited_sites =
       new MostVisitedSites(ProfileAndroid::FromProfileAndroid(jprofile));
   return reinterpret_cast<intptr_t>(most_visited_sites);
 }