Reenable BrowserTest.WindowOpenClose

ui/base/gtk/owned_widget_gtk.cc

Index: ui/base/gtk/owned_widget_gtk.cc
diff --git a/ui/base/gtk/owned_widget_gtk.cc b/ui/base/gtk/owned_widget_gtk.cc
index 6a129a21807cf63f60044089402a4de9a47a2629..2324e53286a32efe7aec455d534c8733d42d2fad 100644
--- a/ui/base/gtk/owned_widget_gtk.cc
+++ b/ui/base/gtk/owned_widget_gtk.cc
@@ -37,8 +37,6 @@ void OwnedWidgetGtk::Destroy() {
   gtk_widget_destroy(widget);
 
   DCHECK(!g_object_is_floating(widget));
-  // NOTE: Assumes some implementation details about glib internals.
-  DCHECK_EQ(G_OBJECT(widget)->ref_count, 1U);

[Elliot Glaysher, 2013/07/30 18:13:49]

While I really really want this line gone (and is probably wrong now that Ubuntu
is loading multiple other libraries into our process that hold on to our
widgets), have you discussed this with other people? I don't feel comfortable
dropping this w/o discussion because estade has fought to keep this here.

[Evan Stade, 2013/07/30 22:08:50]

This DCHECK is not spurious. It was intentionally put here and is a large part
of why this class even exists. Without this check, the class serves a different
purpose: it goes from "OwnedWidget" to "ScopedWidgetReference". It has served us
well for years (although yes it can be annoying). The fact that the DCHECK is
being hit 100% of the time indicates that someone's code is wrong.

   g_object_unref(widget);
 }