OOPIF: Data URLs are now rendered in the renderer that initiated the navigation.

content/browser/frame_host/navigation_controller_impl.cc

Index: content/browser/frame_host/navigation_controller_impl.cc
diff --git a/content/browser/frame_host/navigation_controller_impl.cc b/content/browser/frame_host/navigation_controller_impl.cc
index 4be5a3ce4a009f07a74d87de92730bec3740c2b7..3ca4525f1e74e296181acc3ab3a727a037021b12 100644
--- a/content/browser/frame_host/navigation_controller_impl.cc
+++ b/content/browser/frame_host/navigation_controller_impl.cc
@@ -721,6 +721,10 @@ void NavigationControllerImpl::LoadURLWithParams(const LoadURLParams& params) {
           browser_context_));
   if (params.frame_tree_node_id != -1)
     entry->set_frame_tree_node_id(params.frame_tree_node_id);
+  if (params.url.scheme() == url::kDataScheme && params.site_instance.get()) {

[Charlie Reis, 2014/11/20 19:58:43]

Hmm, this doesn't feel right to me.  Process model decisions should generally be
made in RenderFrameHostManager, not NavigationController.  This feels like a
corner case that is better handled there (e.g., something like
GetSiteInstanceForNavigation), if we can.

That said, I don't yet see a clean way to do that in the current API.

[lfg, 2014/12/05 21:55:42]

Done that, though there's a scary comment about the serialized session, would
the source site instance need to be serialized in the session?

+    entry->set_site_instance(
+        static_cast<SiteInstanceImpl*>(params.site_instance.get()));
+  }
   if (params.redirect_chain.size() > 0)
     entry->SetRedirectChain(params.redirect_chain);
   if (params.should_replace_current_entry)