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

content/renderer/render_view_impl.cc

 // Copyright (c) 2011 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 "content/renderer/render_view_impl.h"
 
 #include <algorithm>
 #include <cmath>
 #include <string>
 #include <vector>
@@ skipping 2746 matching lines @@
 void RenderViewImpl::didFailResourceLoad(
     WebFrame* frame, unsigned identifier, const WebURLError& error) {
   // Ignore
 }
 
 void RenderViewImpl::didLoadResourceFromMemoryCache(
     WebFrame* frame, const WebURLRequest& request,
     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())));
 }
 
 void RenderViewImpl::didDisplayInsecureContent(WebFrame* frame) {
   Send(new ViewHostMsg_DidDisplayInsecureContent(routing_id_));
 }
 
 void RenderViewImpl::didRunInsecureContent(
@@ skipping 1887 matching lines @@
   return webview()->settings()->useThreadedCompositor();
 }
 
 void RenderViewImpl::OnJavaBridgeInit(
     const IPC::ChannelHandle& channel_handle) {
   DCHECK(!java_bridge_dispatcher_.get());
 #if defined(ENABLE_JAVA_BRIDGE)
   java_bridge_dispatcher_.reset(new JavaBridgeDispatcher(this, channel_handle));
 #endif
 }