Creates a new TabContentsViewGtk for views based FE.

views/controls/native/native_view_host_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 "views/controls/native/native_view_host_gtk.h"
 
 #include <gtk/gtk.h>
 
 #include "base/logging.h"
 #include "views/controls/native/native_view_host.h"
 #include "views/widget/widget_gtk.h"
 
 namespace views {
 
 ////////////////////////////////////////////////////////////////////////////////
 // NativeViewHostGtk, public:
 
 NativeViewHostGtk::NativeViewHostGtk(NativeViewHost* host)
     : host_(host),
       installed_clip_(false),
       destroy_signal_id_(0) {
 }
 
 NativeViewHostGtk::~NativeViewHostGtk() {
 }
 
-// static
-View* NativeViewHostGtk::GetViewForNative(GtkWidget* widget) {
-  gpointer user_data = g_object_get_data(G_OBJECT(widget), "chrome-view");
-  return static_cast<View*>(user_data);
-}
-
-// static
-void NativeViewHostGtk::SetViewForNative(GtkWidget* widget, View* view) {
-  g_object_set_data(G_OBJECT(widget), "chrome-view", view);
-}
-
 ////////////////////////////////////////////////////////////////////////////////
 // NativeViewHostGtk, NativeViewHostWrapper implementation:
 
 void NativeViewHostGtk::NativeViewAttached() {
   DCHECK(host_->native_view());
 
-  GtkWidget* current_parent = gtk_widget_get_parent(host_->native_view());
-  GtkWidget* new_parent = host_->GetWidget()->GetNativeView();
-  // Only adjust the parent if the parent actually changed.
-  if (current_parent != new_parent) {
-    // First hide the new window. We don't want anything to draw (like sub-hwnd
-    // borders), when we change the parent below.
-    gtk_widget_hide(host_->native_view());
+  if (gtk_widget_get_parent(host_->native_view())) {

[sky, 2009/06/16 16:01:45]

NOTE: rather than invoking ReparentChild you could ref and unref as is done in
AddedToWidget.

I suspect the ownership of the widget isn't quite right in this class if the
widget is removed and then parented again. But that can be dealt with later.

+    static_cast<WidgetGtk*>(host_->GetWidget())->ReparentChild(
+        host_->native_view());
+  } else {
+    static_cast<WidgetGtk*>(host_->GetWidget())->AddChild(
+        host_->native_view());
+  }
 
-    if (current_parent) {
-      gtk_container_remove(GTK_CONTAINER(current_parent),
-                           host_->native_view());
-    }
-
-    // Adds a mapping between the GtkWidget and us.
-    SetViewForNative(host_->native_view(), host_);
-
-    if (!destroy_signal_id_) {
-      destroy_signal_id_ = g_signal_connect(G_OBJECT(host_->native_view()),
-                                            "destroy", G_CALLBACK(CallDestroy),
-                                            NULL);
-    }
-
-    // Set the parent.
-    static_cast<WidgetGtk*>(host_->GetWidget())->AddChild(host_->native_view());
+  if (!destroy_signal_id_) {
+    destroy_signal_id_ = g_signal_connect(G_OBJECT(host_->native_view()),
+                                          "destroy", G_CALLBACK(CallDestroy),
+                                          this);
   }
 
   // Always layout though.
   host_->Layout();
 
   // TODO(port): figure out focus.
 }
 
 void NativeViewHostGtk::NativeViewDetaching() {
   DCHECK(host_->native_view());
 
   g_signal_handler_disconnect(G_OBJECT(host_->native_view()),
                               destroy_signal_id_);
   destroy_signal_id_ = 0;
 
   // TODO(port): focus.
   // FocusManager::UninstallFocusSubclass(native_view());
   installed_clip_ = false;
 }
 
 void NativeViewHostGtk::AddedToWidget() {
   if (!host_->native_view())
     return;
   WidgetGtk* parent_widget = static_cast<WidgetGtk*>(host_->GetWidget());
   GtkWidget* widget_parent = gtk_widget_get_parent(host_->native_view());
-  GtkWidget* parent_widget_widget = parent_widget->child_widget_parent();
+  GtkWidget* parent_widget_widget = parent_widget->window_contents();
   if (widget_parent != parent_widget_widget) {
     g_object_ref(host_->native_view());
     if (widget_parent)
       gtk_container_remove(GTK_CONTAINER(widget_parent), host_->native_view());
     gtk_container_add(GTK_CONTAINER(parent_widget_widget),
                       host_->native_view());
     g_object_unref(host_->native_view());
   }
   if (host_->IsVisibleInRootView())
     gtk_widget_show(host_->native_view());
   else
     gtk_widget_hide(host_->native_view());
   host_->Layout();
 }
 
 void NativeViewHostGtk::RemovedFromWidget() {
   if (!host_->native_view())
     return;
 
   WidgetGtk* parent_widget = static_cast<WidgetGtk*>(host_->GetWidget());
   gtk_widget_hide(host_->native_view());
   if (parent_widget) {
-    gtk_container_remove(GTK_CONTAINER(parent_widget->child_widget_parent()),
+    gtk_container_remove(GTK_CONTAINER(parent_widget->window_contents()),
                          host_->native_view());
   }
 }
 
 void NativeViewHostGtk::InstallClip(int x, int y, int w, int h) {
   DCHECK(w > 0 && h > 0);
 
   bool has_window =
       (GTK_WIDGET_FLAGS(host_->native_view()) & GTK_NO_WINDOW) == 0;
   if (!has_window) {
@@ skipping 40 matching lines @@
 }
 
 void NativeViewHostGtk::SetFocus() {
   NOTIMPLEMENTED();
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // NativeViewHostGtk, private:
 
 // static
-void NativeViewHostGtk::CallDestroy(GtkObject* object) {
-  View* view = GetViewForNative(GTK_WIDGET(object));
-  if (!view)
-    return;
-
-  return static_cast<NativeViewHost*>(view)->NativeViewDestroyed();
+void NativeViewHostGtk::CallDestroy(GtkObject* object,
+                                    NativeViewHostGtk* host) {
+  return host->host_->NativeViewDestroyed();
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // NativeViewHostWrapper, public:
 
 // static
 NativeViewHostWrapper* NativeViewHostWrapper::CreateWrapper(
     NativeViewHost* host) {
   return new NativeViewHostGtk(host);
 }
 
 }  // namespace views