Fix leak of GtkWidgets in NativeViewHost, and in doing so fix a crash on shut...

views/controls/native/native_view_host_gtk.h

Index: views/controls/native/native_view_host_gtk.h
===================================================================
--- views/controls/native/native_view_host_gtk.h	(revision 22173)
+++ views/controls/native/native_view_host_gtk.h	(working copy)
@@ -17,6 +17,10 @@
 class View;
 class WidgetGtk;
 
+// Note that the NativeViewHostGtk assumes ownership of the GtkWidget attached
+// to it for the duration of its attachment. This is so the NativeViewHostGtk
+// can safely reparent the GtkWidget in multiple passes without having to worry
+// about the GtkWidget's refcnt falling to 0.
 class NativeViewHostGtk : public NativeViewHostWrapper {
  public:
   explicit NativeViewHostGtk(NativeViewHost* host);
@@ -45,10 +49,8 @@
   // DestroyFixed returns true if an associated GtkWidget was addref'ed.

[jcampan, 2009/07/31 23:51:56]

It does not return anything anymore, fix comment.

   // It does this because when the fixed is destroyed the refcount for the
   // contained GtkWidget is decremented, which may cause it to be destroyed
-  // which we do not want. If this function returns true, the caller is
-  // responsible for unrefing the GtkWidget after it has been added to the new
-  // container.
-  bool DestroyFixed();
+  // which we do not want.
+  void DestroyFixed();
 
   WidgetGtk* GetHostWidget() const;