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

content/browser/frame_host/navigation_controller_impl.cc

 // Copyright 2013 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/browser/frame_host/navigation_controller_impl.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
@@ skipping 703 matching lines @@
   NavigationEntryImpl* entry = NavigationEntryImpl::FromNavigationEntry(
       CreateNavigationEntry(
           params.url,
           params.referrer,
           params.transition_type,
           params.is_renderer_initiated,
           params.extra_headers,
           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)
     entry->set_should_replace_entry(true);
   entry->set_should_clear_history_list(params.should_clear_history_list);
   entry->SetIsOverridingUserAgent(override);
   entry->set_transferred_global_request_id(
       params.transferred_global_request_id);
   entry->SetFrameToNavigate(params.frame_name);
 
@@ skipping 1056 matching lines @@
     }
   }
 }
 
 void NavigationControllerImpl::SetGetTimestampCallbackForTest(
     const base::Callback<base::Time()>& get_timestamp_callback) {
   get_timestamp_callback_ = get_timestamp_callback;
 }
 
 }  // namespace content