X message pump: Remove the GDK window event filter.

base/message_pump_glib_x.cc

Index: base/message_pump_glib_x.cc
diff --git a/base/message_pump_glib_x.cc b/base/message_pump_glib_x.cc
index c694114d2933b97253723f4921ff5919a2f56fd1..b4c8b7d403648f9a6fb0c544558a1e8e2823b7db 100644
--- a/base/message_pump_glib_x.cc
+++ b/base/message_pump_glib_x.cc
@@ -43,6 +43,12 @@ MessagePumpGlibX::MessagePumpGlibX() : base::MessagePumpForUI(),
 }
 
 MessagePumpGlibX::~MessagePumpGlibX() {
+  // Remove the filter.

[Evan Martin, 2011/05/25 18:56:06]

I guess this comment isn't very useful, since it's self-evident from the
following line.  Consider either deleting it or saying something about why
you're doing it (e.g. "clean up the filter that we installed in the ctor")

[sadrul, 2011/05/25 19:00:05]

I agree; it's fairly clear what's happening. Removed the comment.

+  gdk_window_remove_filter(NULL, &GdkEventFilter, this);
+
+  // It is not necessary to reset the GDK event handler using
+  // gdk_event_handler_set since it's done in the destructor for
+  // MessagePumpForUI.
 }
 
 bool MessagePumpGlibX::ShouldCaptureXEvent(XEvent* xev) {