Make GTK file dialog box modal for parent window, instead of for the entire...

chrome/browser/renderer_host/render_widget_host_view_gtk.cc

 // Copyright (c) 2006-2008 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.
 
 #include "chrome/browser/renderer_host/render_widget_host_view_gtk.h"
 
 #include <gtk/gtk.h>
 #include <gdk/gdk.h>
 #include <gdk/gdkkeysyms.h>
 #include <gdk/gdkx.h>
@@ skipping 403 matching lines @@
   parent_ = parent_host_view->GetNativeView();
   GtkWidget* popup = gtk_window_new(GTK_WINDOW_POPUP);
   view_.Own(RenderWidgetHostViewGtkWidget::CreateNewWidget(this));
   plugin_container_manager_.set_host_widget(view_.get());
   gtk_container_add(GTK_CONTAINER(popup), view_.get());
 
   // If we are not activatable, we don't want to grab keyboard input,
   // and webkit will manage our destruction.
   if (activatable()) {
     // Grab all input for the app. If a click lands outside the bounds of the
-    // popup, WebKit will notice and destroy us.
+    // popup, WebKit will notice and destroy us. Before doing this we need
+    // to ensure that the the popup is added to the browser's window group,
+    // to allow for the grabs to work correctly.
+    gtk_window_group_add_window(gtk_window_get_group(
+        GTK_WINDOW(gtk_widget_get_toplevel(parent_))), GTK_WINDOW(popup));
     gtk_grab_add(view_.get());
+
     // Now grab all of X's input.
     gdk_pointer_grab(
         parent_->window,
         TRUE,  // Only events outside of the window are reported with respect
                // to |parent_->window|.
         static_cast<GdkEventMask>(GDK_BUTTON_PRESS_MASK |
             GDK_BUTTON_RELEASE_MASK | GDK_POINTER_MOTION_MASK),
         NULL,
         NULL,
         GDK_CURRENT_TIME);
@@ skipping 324 matching lines @@
   }
 }
 
 void RenderWidgetHostViewGtk::PluginProcessCrashed(base::ProcessId pid) {
   for (PluginPidMap::iterator i = plugin_pid_map_.find(pid);
        i != plugin_pid_map_.end() && i->first == pid; ++i) {
     plugin_container_manager_.DestroyPluginContainer(i->second);
   }
   plugin_pid_map_.erase(pid);
 }