PlzNavigate: Get StreamPrivate API to work.

chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc

Index: chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
diff --git a/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc b/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
index bc8d974c05a46b49907bab0b74e34a4ce9f95984..8a6b269c38ab13514f0ff4347fed17715c05a18c 100644
--- a/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
+++ b/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
@@ -181,15 +181,16 @@ void UpdatePrerenderNetworkBytesCallback(content::WebContents* web_contents,
 void SendExecuteMimeTypeHandlerEvent(
     std::unique_ptr<content::StreamInfo> stream,
     int64_t expected_content_size,
-    int render_process_id,
-    int render_frame_id,
     const std::string& extension_id,
     const std::string& view_id,
-    bool embedded) {
+    bool embedded,
+    const ResourceRequestInfo::WebContentsGetter& web_contents_getter,

[jam, 2016/09/20 21:29:35]

it seems redundant passing both this and FTNID and RPID/RFID. i.e. what happens
if it's not the same?

nasko: if we're exposing FTNID outside of content/public, how about having a
static WebContents getter from FTNID?

[ananta, 2016/09/21 05:26:32]

Added a static getter to get the WebContents associated with a FTNID. Currently
this leads to duplicated code  trying to fetch the WC in three places. Here, in
StreamsPrivateAPI::ExecuteMimeTypeHandler and the EmbedderObserver ctor. If we
add a helper returning the WC given a FTNID, RFID, RPID then it looks like
ResourceRequestInfoImpl::GetWebContentsGetter.

I updated the patch anyways. Please let me know what you think. I agree that
passing in a WC along with the various IDs seems redundant.

+    int frame_tree_node_id,
+    int render_process_id,
+    int render_frame_id) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
 
-  content::WebContents* web_contents =
-      tab_util::GetWebContentsByFrameID(render_process_id, render_frame_id);
+  content::WebContents* web_contents = web_contents_getter.Run();
   if (!web_contents)
     return;
 
@@ -210,7 +211,8 @@ void SendExecuteMimeTypeHandlerEvent(
     return;
   streams_private->ExecuteMimeTypeHandler(
       extension_id, web_contents, std::move(stream), view_id,
-      expected_content_size, embedded, render_process_id, render_frame_id);
+      expected_content_size, embedded, frame_tree_node_id, render_process_id,
+      render_frame_id);
 }
 #endif  // !defined(ENABLE_EXTENSIONS)
 
@@ -722,9 +724,11 @@ void ChromeResourceDispatcherHostDelegate::OnStreamCreated(
   content::BrowserThread::PostTask(
       content::BrowserThread::UI, FROM_HERE,
       base::Bind(&SendExecuteMimeTypeHandlerEvent, base::Passed(&stream),
-                 request->GetExpectedContentSize(), info->GetChildID(),
-                 info->GetRenderFrameID(), ix->second.extension_id,
-                 ix->second.view_id, embedded));
+                 request->GetExpectedContentSize(), ix->second.extension_id,
+                 ix->second.view_id, embedded,
+                 info->GetWebContentsGetterForRequest(),
+                 info->GetFrameTreeNodeId(), info->GetChildID(),
+                 info->GetRenderFrameID()));
   stream_target_info_.erase(request);
 #endif
 }