From 4d7491897922f13118eeff56c346090cfc7817f2 Mon Sep 17 00:00:00 2001
From: "shaver%netscape.com" <shaver%netscape.com>
Date: Fri, 6 Nov 1998 18:26:30 +0000
Subject: [PATCH] improve nsWindow to handle toplevel and embedded windows, and
 use GtkLayout

---
 widget/src/gtk/nsWindow.cpp | 62 ++++++++++++++++++++++++++++---------
 1 file changed, 47 insertions(+), 15 deletions(-)

diff --git a/widget/src/gtk/nsWindow.cpp b/widget/src/gtk/nsWindow.cpp
index db495df0818..5dc21a53b94 100644
--- a/widget/src/gtk/nsWindow.cpp
+++ b/widget/src/gtk/nsWindow.cpp
@@ -240,35 +240,61 @@ nsresult nsWindow::StandardWindowCreate(nsIWidget *aParent,
   mBounds = aRect;
   mAppShell = aAppShell;
 
+#ifdef DEBUG_shaver
+  fprintf(stderr,
+          "StandardWindowCreate: par %p, con %p, app %p, tk %p, init %p,\n"
+          "                      natpar %p\n", aParent, aContext, aAppShell,
+          aToolkit, aInitData, aNativeParent);
+#endif
+
   InitToolkit(aToolkit, aParent);
 
   // save the event callback function
   mEventCallback = aHandleEventFunction;
 
-/*
   if (aParent) {
-     parentWidget = GTK_WIDGET(aParent->GetNativeData(NS_NATIVE_WIDGET));
-  } else if (aAppShell) {
-     parentWidget = GTK_WIDGET(aAppShell->GetNativeData(NS_NATIVE_SHELL));
+    parentWidget = GTK_WIDGET(aParent->GetNativeData(NS_NATIVE_WIDGET));
+#ifdef DEBUG_shaver
+    fprintf(stderr, "StandardCreateWindow: have aParent %p/%p\n",
+            aParent, parentWidget);
+#endif
+  } else if (aNativeParent) {
+    parentWidget = GTK_WIDGET(aNativeParent);
+#ifdef DEBUG_shaver
+    fprintf(stderr, "StandardCreateWindow: have aNativeParent %p\n",
+            aNativeParent);
+#endif
+  } else if(aAppShell) {
+    nsNativeWidget shellWidget = aAppShell->GetNativeData(NS_NATIVE_SHELL);
+    if (shellWidget)
+      parentWidget = GTK_WIDGET(shellWidget);
+#ifdef DEBUG_shaver
+    fprintf(stderr, "StandardCreateWindow: have aAppShell %p/%p\n",
+            aAppShell, parentWidget);
+#endif
   }
-*/
-   // XXX: This is a kludge, need to be able to create multiple top
-   // level windows instead.
-  if (gFirstTopLevelWindow == 0) {
+
+  if (!parentWidget) {
+#ifdef DEBUG_shaver
+    fprintf(stderr, "StandardCreateWindow: creating toplevel\n");
+#endif
     mainWindow = gtk_window_new(GTK_WINDOW_TOPLEVEL);
-    gFirstTopLevelWindow = mainWindow;
+    mWidget = gtk_layout_new(FALSE, FALSE);
+    gtk_container_add(GTK_CONTAINER(mainWindow), mWidget);
   } else {
-    mainWindow = gtk_window_new(GTK_WINDOW_POPUP);
+#ifdef DEBUG_shaver
+    fprintf(stderr, "StandardCreateWindow: creating GtkLayout subarea\n");
+#endif
+    mainWindow = mWidget = gtk_layout_new(FALSE, FALSE);
+    gtk_layout_put(GTK_LAYOUT(parentWidget), mWidget, aRect.x, aRect.y);
   }
 
   gtk_widget_set_usize(mainWindow, aRect.width, aRect.height);
 
-//  mWidget = ::gtk_layout_new(FALSE,FALSE);
-//  mWidget = gtk_label_new("foobar!");
-  mWidget = ::gtk_vbox_new(FALSE, FALSE);
-  gtk_container_add(GTK_CONTAINER(mainWindow), mWidget);
-
   if (aParent) {
+#ifdef DEBUG_shaver
+    fprintf(stderr, "StandardCreateWindow: adding to aParent\n");
+#endif
     aParent->AddChild(this);
   }
 
@@ -495,7 +521,9 @@ void nsWindow::RemoveChild(nsIWidget* aChild)
 //-------------------------------------------------------------------------
 NS_METHOD nsWindow::Show(PRBool bState)
 {
+#ifdef DEBUG_pavlov
   g_print("nsWindow::Show called with %d\n", bState);
+#endif
   mShown = bState;
   if (bState) {
     //gtk_widget_show(mWidget);
@@ -505,6 +533,10 @@ NS_METHOD nsWindow::Show(PRBool bState)
       toplevel = gtk_widget_get_toplevel(mWidget);
       if (toplevel)
         gtk_widget_show_all(toplevel);
+    } else {
+#ifdef DEBUG_shaver
+      g_print("showing a NULL-be-widgeted nsWindow: %p\n", this);
+#endif
     }
   } else {
   // hide it