Rip out browser-side RID caching for most visited items.

chrome/renderer/chrome_content_renderer_client.cc

 // Copyright (c) 2012 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/renderer/chrome_content_renderer_client.h"
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
 #include "base/path_service.h"
 #include "base/string_util.h"
+#include "base/strings/string_number_conversions.h"
 #include "base/utf_string_conversions.h"
 #include "base/values.h"
 #include "chrome/common/child_process_logging.h"
 #include "chrome/common/chrome_content_client.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/content_settings_pattern.h"
 #include "chrome/common/extensions/chrome_manifest_handlers.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/extensions/extension_constants.h"
@@ skipping 172 matching lines @@
   // Treat CDM invocations like JavaScript.
   if (plugin.name == ASCIIToUTF16(kWidevineCdmDisplayName)) {
     DCHECK(plugin.type == WebPluginInfo::PLUGIN_TYPE_PEPPER_OUT_OF_PROCESS);
     return true;
   }
 #endif  // WIDEVINE_CDM_AVAILABLE
 
   return false;
 }
 
+content::RenderView* GetRenderViewFromWebFrame(WebKit::WebFrame* frame) {

[jam, 2013/06/05 16:36:01]

nit: no need for this method. we know that frame is not null. we also know that
view is not null. so just inline content::RenderView::FromWebView in line 1006.

[kmadhusu, 2013/06/05 18:40:39]

Done.

+  if (!frame)
+    return NULL;
+
+  WebKit::WebView* view = frame->view();
+  if (!view)
+    return NULL;
+  return content::RenderView::FromWebView(view);
+}
+
 }  // namespace
 
 namespace chrome {
 
 ChromeContentRendererClient::ChromeContentRendererClient() {
   g_current_client = this;
 }
 
 ChromeContentRendererClient::~ChromeContentRendererClient() {
   g_current_client = NULL;
@@ skipping 771 matching lines @@
   }
 
   if (url.SchemeIs(chrome::kExtensionResourceScheme) &&
       !extensions::ResourceRequestPolicy::CanRequestExtensionResourceScheme(
           url,
           frame)) {
     *new_url = GURL(chrome::kExtensionResourceInvalidRequestURL);
     return true;
   }
 
+  const content::RenderView* render_view = GetRenderViewFromWebFrame(frame);
+  if (SearchBox* search_box = SearchBox::Get(render_view)) {
+    if (url.SchemeIs(chrome::kChromeSearchScheme)) {
+      if (url.host() == chrome::kChromeUIThumbnailHost)
+        return search_box->GenerateThumbnailURLFromTransientURL(url, new_url);
+      else if (url.host() == chrome::kChromeUIFaviconHost)
+        return search_box->GenerateFaviconURLFromTransientURL(url, new_url);
+    }
+  }
+
   return false;
 }
 
 bool ChromeContentRendererClient::ShouldPumpEventsDuringCookieMessage() {
   // We no longer pump messages, even under Chrome Frame. We rely on cookie
   // read requests handled by CF not putting up UI or causing other actions
   // that would require us to pump messages. This fixes http://crbug.com/110090.
   return false;
 }
 
@@ skipping 189 matching lines @@
 
   if (container->element().shadowHost().isNull())
     return false;
 
   WebString tag_name = container->element().shadowHost().tagName();
   return tag_name.equals(WebString::fromUTF8(kWebViewTagName)) ||
     tag_name.equals(WebString::fromUTF8(kAdViewTagName));
 }
 
 }  // namespace chrome