Browser Plugin: Enable File Chooser (<input type="file">)

content/browser/browser_plugin/browser_plugin_embedder.cc

Index: content/browser/browser_plugin/browser_plugin_embedder.cc
diff --git a/content/browser/browser_plugin/browser_plugin_embedder.cc b/content/browser/browser_plugin/browser_plugin_embedder.cc
index 091c00c90b0bb8c400514743bb4431de4d7e1599..5b8d1cdd2a0b298b2c65989794d9c3776fc9898b 100644
--- a/content/browser/browser_plugin/browser_plugin_embedder.cc
+++ b/content/browser/browser_plugin/browser_plugin_embedder.cc
@@ -95,7 +95,7 @@ void BrowserPluginEmbedder::CreateGuest(RenderViewHost* render_view_host,
 
   guest = guest_web_contents->GetBrowserPluginGuest();
   guest->set_embedder_render_process_host(render_view_host->GetProcess());
-
+  guest->set_embedder_web_contents(web_contents());

[Charlie Reis, 2012/10/16 21:07:21]

Do we really want to set so many fields on the guest (RenderProcessHost,
WebContents, RenderViewHost from Sadrul's CL)?  I worry about them getting out
of sync in the future.

Seems like we only need to set the WebContents, since you can call
GetRenderViewHost() and GetRenderProcessHost() on the WebContents.  Less room
for error.

[Fady Samuel, 2012/10/17 00:28:06]

Done.

   RendererPreferences* guest_renderer_prefs =
       guest_web_contents->GetMutableRendererPrefs();
   // Copy renderer preferences (and nothing else) from the embedder's