Popular sites on the NTP: add thumbnails!

components/suggestions/suggestions_service.h

Index: components/suggestions/suggestions_service.h
diff --git a/components/suggestions/suggestions_service.h b/components/suggestions/suggestions_service.h
index 7f0aab3525adaa6cd9cefb2e16db3910f763b57c..1dc66a033f3cc775cad4fce2fb64e077bb549eeb 100644
--- a/components/suggestions/suggestions_service.h
+++ b/components/suggestions/suggestions_service.h
@@ -75,6 +75,14 @@ class SuggestionsService : public KeyedService, public net::URLFetcherDelegate {
       const GURL& url,
       base::Callback<void(const GURL&, const SkBitmap*)> callback);
 
+  // A version of |GetPageThumbnail| that explicitly supplies the download URL
+  // for the thumbnail. Replaces any pre-existing thumbnail URL with the
+  // supplied one.
+  void GetPageThumbnailWithURL(
+      const GURL& url,
+      const GURL& thumbnail_url,
+      base::Callback<void(const GURL&, const SkBitmap*)> callback);

[Bernhard Bauer, 2015/09/04 13:42:41]

Pass the callback by const reference?

[Marc Treib, 2015/09/04 13:49:03]

Done.
I was just blindly copying GetPageThumbnail here.. changed that to const ref as
well.

+
   // Adds a URL to the blacklist cache, invoking |callback| on success or
   // |fail_callback| otherwise. The URL will eventually be uploaded to the
   // server.