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

chrome/browser/ui/browser_navigator.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_BROWSER_NAVIGATOR_H_
 #define CHROME_BROWSER_UI_BROWSER_NAVIGATOR_H_
 
 #include <string>
 
 #include "base/memory/ref_counted.h"
 #include "base/memory/ref_counted_memory.h"
 #include "chrome/browser/ui/host_desktop.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/global_request_id.h"
 #include "content/public/browser/page_navigator.h"
+#include "content/public/browser/site_instance.h"
 #include "content/public/common/referrer.h"
 #include "ui/base/page_transition_types.h"
 #include "ui/base/window_open_disposition.h"
 #include "ui/gfx/rect.h"
 #include "url/gurl.h"
 
 class Browser;
 class Profile;
 
 namespace content {
@@ skipping 32 matching lines @@
                  content::WebContents* a_target_contents);
   NavigateParams(Profile* profile,
                  const GURL& a_url,
                  ui::PageTransition a_transition);
   ~NavigateParams();
 
   // The URL/referrer to be loaded. Ignored if |target_contents| is non-NULL.
   GURL url;
   content::Referrer referrer;
 
+  // Site instance that initiated the navigation. Should only be used when a
+  // cross-site navigation is initiated by a remote frame.

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

Why?  It may only matter in that case for now, but it would be more consistent
to set it in all cases.

I'd suggest simplifying the comment to "SiteInstance of the frame that initiated
the navigation" and setting it in the LocalFrame case as well.

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

Done.

+  scoped_refptr<content::SiteInstance> source_site_instance;
+
   // The browser-global ID of the frame to navigate, or -1 for the main frame.
   int64 frame_tree_node_id;
 
   // Any redirect URLs that occurred for this navigation before |url|.
   // Usually empty.
   std::vector<GURL> redirect_chain;
 
   // Indicates whether this navigation will be sent using POST.
   // The POST method is limited support for basic POST data by leveraging
   // NavigationController::LOAD_TYPE_BROWSER_INITIATED_HTTP_POST.
@@ skipping 167 matching lines @@
 // Navigates according to the configuration specified in |params|.
 void Navigate(NavigateParams* params);
 
 // Returns true if the url is allowed to open in incognito window.
 bool IsURLAllowedInIncognito(const GURL& url,
                              content::BrowserContext* browser_context);
 
 }  // namespace chrome
 
 #endif  // CHROME_BROWSER_UI_BROWSER_NAVIGATOR_H_