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

content/browser/site_instance_impl.cc

 // Copyright (c) 2012 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/site_instance_impl.h"
 
 #include "base/command_line.h"
 #include "content/browser/browsing_instance.h"
 #include "content/browser/child_process_security_policy_impl.h"
 #include "content/browser/frame_host/debug_urls.h"
@@ skipping 255 matching lines @@
   // If either URL is invalid, they aren't part of the same site.
   if (!src_url.is_valid() || !dest_url.is_valid())
     return false;
 
   // If the destination url is just a blank page, we treat them as part of the
   // same site.
   GURL blank_page(url::kAboutBlankURL);
   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;
 
   return net::registry_controlled_domains::SameDomainOrHost(
       src_url,
       dest_url,
       net::registry_controlled_domains::INCLUDE_PRIVATE_REGISTRIES);
 }
 
@@ skipping 59 matching lines @@
       *base::CommandLine::ForCurrentProcess();
   if (command_line.HasSwitch(switches::kEnableStrictSiteIsolation) ||
       command_line.HasSwitch(switches::kSitePerProcess)) {
     ChildProcessSecurityPolicyImpl* policy =
         ChildProcessSecurityPolicyImpl::GetInstance();
     policy->LockToOrigin(process_->GetID(), site_);
   }
 }
 
 }  // namespace content