Improvements to Linux InfoBubble and BookmarkBubble.

chrome/browser/gtk/info_bubble_gtk.cc

 // Copyright (c) 2009 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/gtk/info_bubble_gtk.h"
 
 #include <gtk/gtk.h>
 
 #include "base/basictypes.h"
 #include "base/gfx/gtk_util.h"
@@ skipping 98 matching lines @@
       widget->allocation.width, widget->allocation.height, FRAME_STROKE);
   gdk_draw_polygon(drawable, gc, FALSE, &points[0], points.size());
 
   g_object_unref(gc);
   return FALSE;  // Propagate so our children paint, etc.
 }
 
 }  // namespace
 
 // static
-InfoBubbleGtk* InfoBubbleGtk::Show(const gfx::Rect& rect, GtkWidget* content) {
+InfoBubbleGtk* InfoBubbleGtk::Show(const gfx::Rect& rect,
+                                   GtkWidget* content,
+                                   InfoBubbleGtkDelegate* delegate) {
   InfoBubbleGtk* bubble = new InfoBubbleGtk();
   bubble->Init(rect, content);
+  bubble->set_delegate(delegate);
   return bubble;
 }
 
 InfoBubbleGtk::InfoBubbleGtk()
-    : window_(NULL),
+    : delegate_(NULL),
+      window_(NULL),
       screen_x_(0),
-      screen_y_(0),
-      closed_(false) {
+      screen_y_(0) {
+      
 }
 
 InfoBubbleGtk::~InfoBubbleGtk() {
 }
 
 void InfoBubbleGtk::Init(const gfx::Rect& rect, GtkWidget* content) {
   DCHECK(!window_);
   screen_x_ = rect.x() + (rect.width() / 2) - kArrowX;
   screen_y_ = rect.y() + rect.height() + kArrowToContentPadding;
 
@@ skipping 26 matching lines @@
                                  GDK_BUTTON_RELEASE_MASK);
 
   g_signal_connect(window_, "size-allocate",
                    G_CALLBACK(HandleSizeAllocate), NULL);
   g_signal_connect(window_, "expose-event",
                    G_CALLBACK(HandleExpose), NULL);
   g_signal_connect(window_, "configure-event",
                    G_CALLBACK(&HandleConfigureThunk), this);
   g_signal_connect(window_, "button-press-event",
                    G_CALLBACK(&HandleButtonPressThunk), this);
+  g_signal_connect(window_, "destroy",
+                   G_CALLBACK(&HandleDestroyThunk), this);
 
   gtk_widget_show_all(window_);
+  // Make sure our window has focus, is brought to the top, etc.
   gtk_window_present(GTK_WINDOW(window_));
+  // We add a GTK (application level) grab.  This means we will get all
+  // keyboard and mouse events for our application, even if they were delivered
+  // on another window.  This allows us to close when the user clicks outside
+  // of the info bubble.  We don't use an X grab since that would steal
+  // keystrokes from your window manager, prevent you from interacting with
+  // other applications, etc.
   gtk_grab_add(window_);
 }
 
-void InfoBubbleGtk::Close() {
-  DCHECK(!closed_);
+void InfoBubbleGtk::Close(bool closed_by_escape) {
+  // Notify the delegate that we're about to close.  This gives the chance
+  // to save state / etc from the hosted widget before it's destroyed.
+  if (delegate_)
+    delegate_->InfoBubbleClosing(this, closed_by_escape);
+
   DCHECK(window_);
   gtk_widget_destroy(window_);
-  window_ = NULL;
-  closed_ = true;
+  // |this| has been deleted, see HandleDestroy.
 }
 
 gboolean InfoBubbleGtk::HandleConfigure(GdkEventConfigure* event) {
   // If the window is moved someplace besides where we want it, move it back.
   // TODO(deanm): In the end, I will probably remove this code and just let
   // the user move around the bubble like a normal dialog.  I want to try
   // this for now and see if it causes problems when any window managers.
   if (event->x != screen_x_ || event->y != screen_y_)
     gtk_window_move(GTK_WINDOW(window_), screen_x_, screen_y_);
   return FALSE;
 }
 
 gboolean InfoBubbleGtk::HandleButtonPress(GdkEventButton* event) {
   // If we got a click in our own window, that's ok.
   if (event->window == window_->window)
     return FALSE;  // Propagate.
 
   // Otherwise we had a click outside of our window, close ourself.
   Close();
   return TRUE;
 }
+
+gboolean InfoBubbleGtk::HandleDestroy() {
+  // We are self deleting, we have a destroy signal setup to catch when we
+  // destroy the widget manually, or the window was closed via X.  This will
+  // delete the InfoBubbleGtk object.
+  delete this;
+  return FALSE;  // Propagate.
+}