Most visited thumbnails and favicons need id-based urls

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

Index: chrome/browser/ui/webui/ntp/thumbnail_source.cc
diff --git a/chrome/browser/ui/webui/ntp/thumbnail_source.cc b/chrome/browser/ui/webui/ntp/thumbnail_source.cc
index 706b760592df07346b0058a3aa2c0cbb9eee0d0b..0848da3d4e2593fa79c2b6773a913307f9960ecb 100644
--- a/chrome/browser/ui/webui/ntp/thumbnail_source.cc
+++ b/chrome/browser/ui/webui/ntp/thumbnail_source.cc
@@ -8,6 +8,8 @@
 #include "base/message_loop.h"
 #include "base/memory/ref_counted_memory.h"
 #include "chrome/browser/instant/instant_io_context.h"
+#include "chrome/browser/instant/instant_service.h"
+#include "chrome/browser/instant/instant_service_factory.h"
 #include "chrome/browser/thumbnails/thumbnail_service.h"
 #include "chrome/browser/thumbnails/thumbnail_service_factory.h"
 #include "chrome/browser/profiles/profile.h"
@@ -17,9 +19,13 @@
 #include "net/url_request/url_request.h"
 #include "ui/base/resource/resource_bundle.h"
 
+using content::BrowserThread;
+
 // Set ThumbnailService now as Profile isn't thread safe.
 ThumbnailSource::ThumbnailSource(Profile* profile)
-    : thumbnail_service_(ThumbnailServiceFactory::GetForProfile(profile)) {
+    : thumbnail_service_(ThumbnailServiceFactory::GetForProfile(profile)),
+      current_request_(NULL),
+      profile_(profile) {
 }
 
 ThumbnailSource::~ThumbnailSource() {
@@ -30,9 +36,19 @@ std::string ThumbnailSource::GetSource() {
 }
 
 void ThumbnailSource::StartDataRequest(
-    const std::string& path,
+    const std::string& raw_path,
     bool is_incognito,
     const content::URLDataSource::GotDataCallback& callback) {
+  // Translate to regular path if |raw_path| is of the form
+  // chrome-search://favicon/<rid>, where rid is a uint64.
+  std::string path = raw_path;
+  if (BrowserThread::CurrentlyOn(BrowserThread::IO)) {
+    path = InstantService::MaybeTranslateInstantPathOnIO(
+        current_request_, raw_path);
+  } else if (BrowserThread::CurrentlyOn(BrowserThread::UI)) {
+    path = InstantService::MaybeTranslateInstantPathOnUI(profile_, raw_path);
+  }
+
   scoped_refptr<base::RefCountedMemory> data;
   if (thumbnail_service_->GetPageThumbnail(GURL(path), &data)) {
     // We have the thumbnail.
@@ -40,6 +56,7 @@ void ThumbnailSource::StartDataRequest(
   } else {
     callback.Run(default_thumbnail_);
   }
+  current_request_ = NULL;
 }
 
 std::string ThumbnailSource::GetMimeType(const std::string&) const {
@@ -57,7 +74,10 @@ MessageLoop* ThumbnailSource::MessageLoopForRequestPath(
 
 bool ThumbnailSource::ShouldServiceRequest(
     const net::URLRequest* request) const {
-  if (request->url().SchemeIs(chrome::kChromeSearchScheme))
-    return InstantIOContext::ShouldServiceRequest(request);
+  current_request_ = request;
+  if (request->url().SchemeIs(chrome::kChromeSearchScheme)) {
+    return InstantService::IsInstantPath(request->url()) &&
+        InstantIOContext::ShouldServiceRequest(request);
+  }
   return URLDataSource::ShouldServiceRequest(request);
 }