Don't call DidLoadResourceFromMemoryCache() on data: URLs.

content/renderer/render_view_impl.cc

Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index c4d1fed43c0b6e7735893f698168704399cd6f7c..eaba3197170ad7723b5b576f3cc66edb11932e7d 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -2764,9 +2764,13 @@ void RenderViewImpl::didLoadResourceFromMemoryCache(
     const WebURLResponse& response) {
   // Let the browser know we loaded a resource from the memory cache.  This
   // message is needed to display the correct SSL indicators.
+  GURL url(request.url());
+  if (url.SchemeIs("data")) {
+    return;
+  }

[abarth-chromium, 2011/10/27 20:36:35]

I guess this is ok.  It's kind of a hack though.  (Also, no braces on
single-line if statements.)  Is there something concrete we gain by doing this?

If you're going to do this, please add a big comment explaining why we're doing
this.

   Send(new ViewHostMsg_DidLoadResourceFromMemoryCache(
       routing_id_,
-      request.url(),
+      url,
       response.securityInfo(),
       request.httpMethod().utf8(),
       ResourceType::FromTargetType(request.targetType())));