Handle the ViewHostMsg_RunFileChooser IPC message which is sent by content(renderer) in RVH and

content/browser/renderer_host/render_view_host.cc

Index: content/browser/renderer_host/render_view_host.cc
===================================================================
--- content/browser/renderer_host/render_view_host.cc	(revision 98068)
+++ content/browser/renderer_host/render_view_host.cc	(working copy)
@@ -718,10 +718,14 @@
 #if defined(OS_MACOSX)
     IPC_MESSAGE_HANDLER(ViewHostMsg_ShowPopup, OnMsgShowPopup)
 #endif
+    IPC_MESSAGE_HANDLER(ViewHostMsg_RunFileChooser, OnRunFileChooser)
+
     // Have the super handle all other messages.
     IPC_MESSAGE_UNHANDLED(handled = RenderWidgetHost::OnMessageReceived(msg))
-    // NOTE: Do not add a message handler that just calls the delegate!
+

[jam, 2011/08/24 20:30:29]

why add this blank line? and also above

[ananta, 2011/08/24 20:38:05]

Removed both lines

+    // NOTE: Do not add a message handler that just calls the delegate!,
     // Dispatch the message directly there instead.
+    // This does not apply to messages sent by content.

[jam, 2011/08/24 20:30:29]

if you add this line, please explain why that's the case (i.e. because content
messages need to be dispatched in content)

[ananta, 2011/08/24 20:38:05]

I removed the whole comment as delegates can live in chrome. For e.g.
ExtensionHost, in which case we don't want them to dispatch content messages.

   IPC_END_MESSAGE_MAP_EX()
 
   if (!msg_is_ok) {
@@ -1357,3 +1361,9 @@
   }
 }
 #endif
+
+void RenderViewHost::OnRunFileChooser(
+    const ViewHostMsg_RunFileChooser_Params& params) {
+  delegate_->RunFileChooser(this, params);
+}
+