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

content/browser/frame_host/render_frame_host_impl.h

Index: content/browser/frame_host/render_frame_host_impl.h
diff --git a/content/browser/frame_host/render_frame_host_impl.h b/content/browser/frame_host/render_frame_host_impl.h
index 052da3b4366107390320f166275346c975e2fb4d..396d3747404f3b9b90ef46fd57dc69a3fd60fb0e 100644
--- a/content/browser/frame_host/render_frame_host_impl.h
+++ b/content/browser/frame_host/render_frame_host_impl.h
@@ -270,10 +270,14 @@ class CONTENT_EXPORT RenderFrameHostImpl
   // Load the specified URL; this is a shortcut for Navigate().
   void NavigateToURL(const GURL& url);
 
-  // Treat this prospective navigation as thought it originated from the
-  // frame. Used, e.g., for a navigation request that originated from
-  // a RemoteFrame.
-  void OpenURL(const FrameHostMsg_OpenURL_Params& params);
+  // TODO(creis): Remove this method and have RenderFrameProxyHost call

[Charlie Reis, 2014/12/09 19:14:37]

nit: Move TODO to the end of the comment (since the rest will stick around when
we remove it).

[lfg, 2014/12/10 21:53:00]

Done.

+  // RequestOpenURL with its FrameTreeNode.
+  // Treat this prospective navigation as thought it originated from the frame.

[Charlie Reis, 2014/12/09 19:14:37]

nit: thought -> though

[lfg, 2014/12/10 21:53:00]

Done.

+  // Used, e.g., for a navigation request that originated from a a RemoteFrame.

[Charlie Reis, 2014/12/09 19:14:37]

nit: fix "a a"

[lfg, 2014/12/10 21:53:00]

Done.

+  // |source_site_instance| is the site instance of the frame that initiated the
+  // navigation.
+  void OpenURL(const FrameHostMsg_OpenURL_Params& params,
+               SiteInstance* source_site_instance);
 
   // Stop the load in progress.
   void Stop();