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

chrome/browser/ui/browser_navigator.h

Index: chrome/browser/ui/browser_navigator.h
diff --git a/chrome/browser/ui/browser_navigator.h b/chrome/browser/ui/browser_navigator.h
index 4209927811452747814160386e8e2e8d14e6af60..c407d7f62186c73c42af42e21e1eebc035ed39eb 100644
--- a/chrome/browser/ui/browser_navigator.h
+++ b/chrome/browser/ui/browser_navigator.h
@@ -13,6 +13,7 @@
 #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"
@@ -65,6 +66,9 @@ struct NavigateParams {
   GURL url;
   content::Referrer referrer;
 
+  // Optional site instance that initiated the navigation.

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

This should be more precise.  When should callers set this?  The name should
probably more carefully indicate what it's for as well.

I'm concerned that a bug in setting this incorrectly will lead to a security bug
(putting the wrong page into a SiteInstance), which means we may want to take a
different approach.

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

Improved the commend and renamed it to source_site_instance.

+  scoped_refptr<content::SiteInstance> site_instance;
+
   // The browser-global ID of the frame to navigate, or -1 for the main frame.
   int64 frame_tree_node_id;