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"
 #include "chrome/common/extensions/extension_manifest_constants.h"
 #include "chrome/common/extensions/extension_process_policy.h"
 #include "chrome/common/extensions/extension_set.h"
 #include "chrome/common/extensions/permissions/chrome_api_permissions.h"
 #include "chrome/common/external_ipc_fuzzer.h"
+#include "chrome/common/instant_types.h"
 #include "chrome/common/localized_error.h"
 #include "chrome/common/render_messages.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/renderer/benchmarking_extension.h"
 #include "chrome/renderer/chrome_render_process_observer.h"
 #include "chrome/renderer/chrome_render_view_observer.h"
 #include "chrome/renderer/content_settings_observer.h"
 #include "chrome/renderer/extensions/chrome_v8_context.h"
 #include "chrome/renderer/extensions/chrome_v8_extension.h"
 #include "chrome/renderer/extensions/dispatcher.h"
@@ skipping 157 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;
 }
 
+// Parses |url| and fills in |id| with the InstantRestrictedID obtained from the
+// |url|. |render_view_id| is the ID of the associated RenderView.
+//
+// Valid |url| forms:
+// chrome-search://favicon/<view_id>/<restriced_id> or
+// chrome-search://thumb/<view_id>/<restricted_id>
+//
+// If the |url| is valid, returns true and fills in |id| with restricted_id
+// value. If the |url| is invalid, returns false and |id| is not set.
+bool GetInstantRestrictedIDFromURL(int render_view_id,

[Jered, 2013/06/04 23:27:23]

I realized this function might make more sense in searchbox or in instant_types
than here. What do you think?

[kmadhusu, 2013/06/05 00:48:30]

Moved to SearchBox.

+                                   const GURL& url,
+                                   InstantRestrictedID* id) {
+  // Strip leading slash.
+  std::string path = url.path().substr(1);
+
+  // Check that the path is of Most visited item ID form.
+  std::vector<std::string> tokens;
+  if (Tokenize(path, "/", &tokens) != 2)
+    return false;
+
+  int view_id = 0;
+  if (!base::StringToInt(tokens[0], &view_id) || view_id != render_view_id)
+    return false;
+  return base::StringToInt(tokens[1], id);
+}
+
+content::RenderView* GetRenderViewFromWebFrame(WebKit::WebFrame* frame) {
+  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) {
+        InstantRestrictedID rid = 0;
+        return GetInstantRestrictedIDFromURL(render_view->GetRoutingID(), url,

[Jered, 2013/06/04 23:19:20]

If the rewrite fails, we'll send a bad URL to chrome-search://{favicon,thumb}.
Have you checked that they don't fall over in that case?

[kmadhusu, 2013/06/05 00:48:30]

If rewrite fails, we will send a bad URL to FaviconSource/ThumbnailSource. 

FaviconSource/ThumbnailSource::StartDataRequest will handle the situation
gracefully and return default favicon/thumbnail.

+                                             &rid) &&
+            search_box->GenerateThumbnailURLFromRID(rid, new_url);
+      } else if (url.host() == chrome::kChromeUIFaviconHost) {
+        InstantRestrictedID rid = 0;
+        return GetInstantRestrictedIDFromURL(render_view->GetRoutingID(), url,
+                                             &rid) &&
+            search_box->GenerateFaviconURLFromRID(rid, 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