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

content/browser/site_instance_impl.cc

Index: content/browser/site_instance_impl.cc
diff --git a/content/browser/site_instance_impl.cc b/content/browser/site_instance_impl.cc
index f1555dc1af8f627c7eb50d811a3ae445adec9678..d1d15bdd05df4f36676dd584193fa6cabb563c13 100644
--- a/content/browser/site_instance_impl.cc
+++ b/content/browser/site_instance_impl.cc
@@ -273,6 +273,9 @@ bool SiteInstance::IsSameWebSite(BrowserContext* browser_context,
   if (dest_url == blank_page)
     return true;
 
+  if (dest_url.scheme() == url::kDataScheme)

[Charlie Reis, 2014/12/06 00:18:50]

I'd like to avoid this if we can.  Generally, typing a data URL into the omnibox
should cause a process swap.  In fact, I'd almost like the reverse (always
return false if the dest is a data URL), since two data URLs are effectively
different sites.  That should be part of a separate CL, though.

It would be better to treat data URLs and about URLs as a special case in RFHM
before getting to IsSameWebSite.

[lfg, 2014/12/08 20:45:33]

Should we also remove about:blank urls from IsSameSiteInstance() ?

+    return true;
+
   // If the schemes differ, they aren't part of the same site.
   if (src_url.scheme() != dest_url.scheme())
     return false;