PlzNavigate: Get StreamPrivate API to work.

chrome/browser/extensions/api/streams_private/streams_private_api.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_EXTENSIONS_API_STREAMS_PRIVATE_STREAMS_PRIVATE_API_H_
 #define CHROME_BROWSER_EXTENSIONS_API_STREAMS_PRIVATE_STREAMS_PRIVATE_API_H_
 
 #include <stdint.h>
 
 #include <map>
@@ skipping 21 matching lines @@
 
   explicit StreamsPrivateAPI(content::BrowserContext* context);
   ~StreamsPrivateAPI() override;
 
   // Send the onExecuteMimeTypeHandler event to |extension_id|.
   // |web_contents| is used to determine the tabId where the document is being
   // opened. The data for the document will be readable from |stream|, and
   // should be |expected_content_size| bytes long. If the viewer is being opened
   // in a BrowserPlugin, specify a non-empty |view_id| of the plugin. |embedded|
   // should be set to whether the document is embedded within another document.
+  // TODO(ananta)
+  // Remove the |render_process_id| and |render_frame_id| parameters when
+  // PlzNavigate ships.

[jam, 2016/09/17 00:10:07]

is this true? i.e. we'll still need them for the non-top level plugin case?

nit: please document the three different IDs (i.e. mentioning that for
plznavigate top level plugins, FTNID has the value, otherwise it's the two other
IDs)

[ananta, 2016/09/17 00:23:27]

Thanks. Changed the comment

[ananta, 2016/09/17 02:43:39]

I debugged the pdf case inside an iframe. The render_frame_id and
render_process_id are still -1 in this case. That should not be the top level
case which you mentioned?

   void ExecuteMimeTypeHandler(const std::string& extension_id,
                               content::WebContents* web_contents,
                               std::unique_ptr<content::StreamInfo> stream,
                               const std::string& view_id,
                               int64_t expected_content_size,
                               bool embedded,
+                              int frame_tree_node_id,
                               int render_process_id,
                               int render_frame_id);
 
   void AbortStream(const std::string& extension_id,
                    const GURL& stream_url,
                    const base::Closure& callback);
 
   // BrowserContextKeyedAPI implementation.
   static BrowserContextKeyedAPIFactory<StreamsPrivateAPI>* GetFactoryInstance();
 
@@ skipping 39 matching lines @@
 
  private:
   void OnClose();
 
   std::string stream_url_;
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_STREAMS_PRIVATE_STREAMS_PRIVATE_API_H_