Ensure using chrome://thumb2/... adds the URL as a forced URL in TopSites so that a thumbnail is ca…

chrome/browser/ui/webui/ntp/thumbnail_source.h

Index: chrome/browser/ui/webui/ntp/thumbnail_source.h
diff --git a/chrome/browser/ui/webui/ntp/thumbnail_source.h b/chrome/browser/ui/webui/ntp/thumbnail_source.h
index f4ff8f014388f4cc9752889265219f08650ed7c4..c280c4cb219b5b0db3bd25a004b7e12fb74b377a 100644
--- a/chrome/browser/ui/webui/ntp/thumbnail_source.h
+++ b/chrome/browser/ui/webui/ntp/thumbnail_source.h
@@ -25,7 +25,7 @@ class ThumbnailService;
 // thumbnails and the history/top-sites backend that serves these.
 class ThumbnailSource : public content::URLDataSource {
  public:
-  ThumbnailSource(Profile* profile, bool prefix_match);
+  ThumbnailSource(Profile* profile, bool forced_thumbnails);
 
   // content::URLDataSource implementation.
   virtual std::string GetSource() const OVERRIDE;
@@ -53,10 +53,11 @@ class ThumbnailSource : public content::URLDataSource {
   // Only used when servicing requests on the UI thread.
   Profile* const profile_;
 
-  // If an exact thumbnail URL match fails, specifies whether or not to try
-  // harder by matching the query thumbnail URL as URL prefix. This affects
-  // GetSource().
-  const bool prefix_match_;
+  // Indicate that this source will force the eventual capture of requested URLs

[Evan Stade, 2013/11/20 22:32:36]

I can't understand this comment. Can you rewrite it with more context?

"forced" doesn't seem like the best choice of adjective. Maybe "eager" or
"aggressive"?

[beaudoin, 2013/11/21 00:06:46]

You're right, it's a bit opaque. Is the new one better?

+  // if they are not found. Such a source will also be more lenient when
+  // matching thumbnail URL, by checking for existing thumbnails in the database
+  // that contain URL matching the prefix of the requested URL.
+  const bool forced_thumbnails_;
 
   DISALLOW_COPY_AND_ASSIGN(ThumbnailSource);
 };