]> Pileus Git - ~andy/gtk/blobdiff - gtk/gtkcombobox.c
Don't unref model if it is NULL. (#139770)
[~andy/gtk] / gtk / gtkcombobox.c
index 6c349378aaf4332e9eba0a10f7c79025911f261e..79942ee42c0f415e12452bcfb3941534aa0674d5 100644 (file)
@@ -17,6 +17,7 @@
  * Boston, MA 02111-1307, USA.
  */
 
+#include <config.h>
 #include "gtkcombobox.h"
 
 #include "gtkarrow.h"
@@ -98,6 +99,7 @@ struct _GtkComboBoxPrivate
   GSList *cells;
 
   guint popup_in_progress : 1;
+  guint destroying : 1;
 };
 
 /* While debugging this evil code, I have learned that
@@ -198,6 +200,8 @@ static void     gtk_combo_box_button_toggled       (GtkWidget       *widget,
                                                     gpointer         data);
 static void     gtk_combo_box_add                  (GtkContainer    *container,
                                                     GtkWidget       *widget);
+static void     gtk_combo_box_remove               (GtkContainer    *container,
+                                                    GtkWidget       *widget);
 
 static ComboCellInfo *gtk_combo_box_get_cell_info  (GtkComboBox      *combo_box,
                                                     GtkCellRenderer  *cell);
@@ -230,7 +234,6 @@ static gint     gtk_combo_box_calc_requested_width (GtkComboBox      *combo_box,
 static void     gtk_combo_box_remeasure            (GtkComboBox      *combo_box);
 
 static void     gtk_combo_box_unset_model          (GtkComboBox      *combo_box);
-static void     gtk_combo_box_set_model_internal   (GtkComboBox      *combo_box);
 
 static void     gtk_combo_box_size_request         (GtkWidget        *widget,
                                                     GtkRequisition   *requisition);
@@ -246,6 +249,9 @@ static gboolean gtk_combo_box_scroll_event         (GtkWidget        *widget,
                                                     GdkEventScroll   *event);
 static void     gtk_combo_box_set_active_internal  (GtkComboBox      *combo_box,
                                                    gint              index);
+static gboolean gtk_combo_box_key_press            (GtkWidget        *widget,
+                                                   GdkEventKey      *event,
+                                                   gpointer          data);
 
 /* listening to the model */
 static void     gtk_combo_box_model_row_inserted   (GtkTreeModel     *model,
@@ -293,7 +299,7 @@ static void     gtk_combo_box_list_row_changed     (GtkTreeModel     *model,
 
 /* menu */
 static void     gtk_combo_box_menu_setup           (GtkComboBox      *combo_box,
-                                                    gboolean          add_childs);
+                                                    gboolean          add_children);
 static void     gtk_combo_box_menu_fill            (GtkComboBox      *combo_box);
 static void     gtk_combo_box_menu_destroy         (GtkComboBox      *combo_box);
 
@@ -326,6 +332,9 @@ static void     gtk_combo_box_menu_row_changed     (GtkTreeModel     *model,
                                                     GtkTreePath      *path,
                                                     GtkTreeIter      *iter,
                                                     gpointer          data);
+static gboolean gtk_combo_box_menu_key_press       (GtkWidget        *widget,
+                                                   GdkEventKey      *event,
+                                                   gpointer          data);
 
 /* cell layout */
 static void     gtk_combo_box_cell_layout_pack_start         (GtkCellLayout         *layout,
@@ -352,6 +361,8 @@ static void     gtk_combo_box_cell_layout_reorder            (GtkCellLayout
 static gboolean gtk_combo_box_mnemonic_activate              (GtkWidget    *widget,
                                                              gboolean      group_cycling);
 
+static void     cell_view_sync_cells (GtkComboBox *combo_box,
+                                      GtkCellView *cell_view);
 
 GType
 gtk_combo_box_get_type (void)
@@ -408,6 +419,7 @@ gtk_combo_box_class_init (GtkComboBoxClass *klass)
   container_class = (GtkContainerClass *)klass;
   container_class->forall = gtk_combo_box_forall;
   container_class->add = gtk_combo_box_add;
+  container_class->remove = gtk_combo_box_remove;
 
   widget_class = (GtkWidgetClass *)klass;
   widget_class->size_allocate = gtk_combo_box_size_allocate;
@@ -515,7 +527,8 @@ gtk_combo_box_init (GtkComboBox *combo_box)
   combo_box->priv = GTK_COMBO_BOX_GET_PRIVATE (combo_box);
 
   combo_box->priv->cell_view = gtk_cell_view_new ();
-  gtk_container_add (GTK_CONTAINER (combo_box), combo_box->priv->cell_view);
+  gtk_widget_set_parent (combo_box->priv->cell_view, GTK_WIDGET (combo_box));
+  GTK_BIN (combo_box)->child = combo_box->priv->cell_view;
   gtk_widget_show (combo_box->priv->cell_view);
 
   combo_box->priv->width = 0;
@@ -614,26 +627,21 @@ gtk_combo_box_state_changed (GtkWidget    *widget,
 }
 
 static void
-gtk_combo_box_style_set (GtkWidget *widget,
-                         GtkStyle  *previous)
+gtk_combo_box_check_appearance (GtkComboBox *combo_box)
 {
-  gboolean appearance;
-  GtkComboBox *combo_box = GTK_COMBO_BOX (widget);
+  gboolean appears_as_list;
 
-  gtk_widget_queue_resize (widget);
-  
   /* if wrap_width > 0, then we are in grid-mode and forced to use
    * unix style
    */
   if (combo_box->priv->wrap_width)
-    return;
-
-  gtk_widget_style_get (widget,
-                        "appears-as-list", &appearance,
-                        NULL);
+    appears_as_list = FALSE;
+  else
+    gtk_widget_style_get (GTK_WIDGET (combo_box),
+                         "appears-as-list", &appears_as_list,
+                         NULL);
 
-  /* TRUE is windows style */
-  if (appearance)
+  if (appears_as_list)
     {
       /* Destroy all the menu mode widgets, if they exist. */
       if (GTK_IS_MENU (combo_box->priv->popup_widget))
@@ -653,11 +661,19 @@ gtk_combo_box_style_set (GtkWidget *widget,
       if (!GTK_IS_MENU (combo_box->priv->popup_widget))
        gtk_combo_box_menu_setup (combo_box, TRUE);
     }
+}
+
+static void
+gtk_combo_box_style_set (GtkWidget *widget,
+                         GtkStyle  *previous)
+{
+  GtkComboBox *combo_box = GTK_COMBO_BOX (widget);
+
+  gtk_combo_box_check_appearance (combo_box);
 
   if (combo_box->priv->tree_view && combo_box->priv->cell_view)
     gtk_cell_view_set_background_color (GTK_CELL_VIEW (combo_box->priv->cell_view), 
                                        &widget->style->base[GTK_WIDGET_STATE (widget)]);
-
 }
 
 static void
@@ -682,9 +698,14 @@ gtk_combo_box_add (GtkContainer *container,
   GtkComboBox *combo_box = GTK_COMBO_BOX (container);
 
   if (combo_box->priv->cell_view && combo_box->priv->cell_view->parent)
-    gtk_container_remove (container, combo_box->priv->cell_view);
-
-  (* GTK_CONTAINER_CLASS (parent_class)->add) (container, widget);
+    {
+      gtk_widget_unparent (combo_box->priv->cell_view);
+      GTK_BIN (container)->child = NULL;
+      gtk_widget_queue_resize (GTK_WIDGET (container));
+    }
+  
+  gtk_widget_set_parent (widget, GTK_WIDGET (container));
+  GTK_BIN (container)->child = widget;
 
   if (combo_box->priv->cell_view &&
       widget != combo_box->priv->cell_view)
@@ -713,6 +734,56 @@ gtk_combo_box_add (GtkContainer *container,
     }
 }
 
+static void
+gtk_combo_box_remove (GtkContainer *container,
+                     GtkWidget    *widget)
+{
+  GtkComboBox *combo_box = GTK_COMBO_BOX (container);
+  gboolean appears_as_list;
+
+  gtk_widget_unparent (widget);
+  GTK_BIN (container)->child = NULL;
+
+  if (combo_box->priv->destroying)
+    return;
+
+  gtk_widget_queue_resize (GTK_WIDGET (container));
+
+  if (!combo_box->priv->tree_view)
+    appears_as_list = FALSE;
+  else
+    appears_as_list = TRUE;
+  
+  if (appears_as_list)
+    gtk_combo_box_list_destroy (combo_box);
+  else if (GTK_IS_MENU (combo_box->priv->popup_widget))
+    {
+      gtk_combo_box_menu_destroy (combo_box);
+      gtk_menu_detach (GTK_MENU (combo_box->priv->popup_widget));
+      combo_box->priv->popup_widget = NULL;
+    }
+
+  if (!combo_box->priv->cell_view)
+    {
+      combo_box->priv->cell_view = gtk_cell_view_new ();
+      gtk_widget_set_parent (combo_box->priv->cell_view, GTK_WIDGET (container));
+      GTK_BIN (container)->child = combo_box->priv->cell_view;
+      
+      gtk_widget_show (combo_box->priv->cell_view);
+      gtk_cell_view_set_model (GTK_CELL_VIEW (combo_box->priv->cell_view),
+                              combo_box->priv->model);
+      cell_view_sync_cells (combo_box, GTK_CELL_VIEW (combo_box->priv->cell_view));
+    }
+
+
+  if (appears_as_list)
+    gtk_combo_box_list_setup (combo_box);
+  else
+    gtk_combo_box_menu_setup (combo_box, TRUE);
+
+  gtk_combo_box_set_active_internal (combo_box, combo_box->priv->active_item);
+}
+
 static ComboCellInfo *
 gtk_combo_box_get_cell_info (GtkComboBox     *combo_box,
                              GtkCellRenderer *cell)
@@ -854,7 +925,7 @@ gtk_combo_box_menu_position_below (GtkMenu  *menu,
    
    gtk_widget_size_request (GTK_WIDGET (menu), &req);
    
-   if (gtk_widget_get_direction (GTK_WIDGET (combo_box)) == GTK_TEXT_DIR_RTL)
+   if (gtk_widget_get_direction (GTK_WIDGET (combo_box)) == GTK_TEXT_DIR_LTR)
      *x = sx;
    else
      *x = sx + child->allocation.width - req.width;
@@ -949,13 +1020,21 @@ gtk_combo_box_menu_position (GtkMenu  *menu,
                             gpointer  user_data)
 {
   GtkComboBox *combo_box;
+  GtkWidget *menu_item;
 
   combo_box = GTK_COMBO_BOX (user_data);
 
   if (combo_box->priv->wrap_width > 0 || combo_box->priv->cell_view == NULL)   
     gtk_combo_box_menu_position_below (menu, x, y, push_in, user_data);
   else
-    gtk_combo_box_menu_position_over (menu, x, y, push_in, user_data);
+    {
+      menu_item = gtk_menu_get_active (GTK_MENU (combo_box->priv->popup_widget));
+      if (menu_item)
+       gtk_menu_shell_select_item (GTK_MENU_SHELL (combo_box->priv->popup_widget), 
+                                   menu_item);
+
+      gtk_combo_box_menu_position_over (menu, x, y, push_in, user_data);
+    }
 
 }
 
@@ -1025,7 +1104,7 @@ void
 gtk_combo_box_popup (GtkComboBox *combo_box)
 {
   gint x, y, width, height;
-
+  
   g_return_if_fail (GTK_IS_COMBO_BOX (combo_box));
 
   if (GTK_WIDGET_MAPPED (combo_box->priv->popup_widget))
@@ -1191,9 +1270,10 @@ gtk_combo_box_size_request (GtkWidget      *widget,
 
       if (combo_box->priv->cell_view)
         {
-          GtkRequisition sep_req, arrow_req;
+          GtkRequisition button_req, sep_req, arrow_req;
           gint border_width, xthickness, ythickness;
 
+          gtk_widget_size_request (combo_box->priv->button, &button_req);
           border_width = GTK_CONTAINER (combo_box->priv->button)->border_width;
           xthickness = combo_box->priv->button->style->xthickness;
           ythickness = combo_box->priv->button->style->ythickness;
@@ -1208,8 +1288,8 @@ gtk_combo_box_size_request (GtkWidget      *widget,
 
           width = bin_req.width + sep_req.width + arrow_req.width;
 
-          height += border_width + 1 + xthickness * 2 + 4;
-          width += border_width + 1 + ythickness * 2 + 4;
+          height += border_width + 1 + ythickness * 2 + 4;
+          width += border_width + 1 + xthickness * 2 + 4;
 
           requisition->width = width;
           requisition->height = height;
@@ -1227,13 +1307,14 @@ gtk_combo_box_size_request (GtkWidget      *widget,
   else
     {
       /* list mode */
-      GtkRequisition button_req;
+      GtkRequisition button_req, frame_req;
 
       /* sample + frame */
       *requisition = bin_req;
 
       if (combo_box->priv->cell_view_frame)
         {
+         gtk_widget_size_request (combo_box->priv->cell_view_frame, &frame_req);
           requisition->width += 2 *
             (GTK_CONTAINER (combo_box->priv->cell_view_frame)->border_width +
              GTK_WIDGET (combo_box->priv->cell_view_frame)->style->xthickness);
@@ -1281,7 +1362,7 @@ gtk_combo_box_size_allocate (GtkWidget     *widget,
 
           width = allocation->width - (border_width + 1 + xthickness * 2 + 4);
 
-          /* handle the childs */
+          /* handle the children */
           gtk_widget_size_request (combo_box->priv->arrow, &req);
           child.width = req.width;
           child.height = allocation->height - 2 * (child.y - allocation->y);
@@ -1382,29 +1463,16 @@ gtk_combo_box_size_allocate (GtkWidget     *widget,
 static void
 gtk_combo_box_unset_model (GtkComboBox *combo_box)
 {
-  if (combo_box->priv->inserted_id != -1)
+  if (combo_box->priv->model)
     {
       g_signal_handler_disconnect (combo_box->priv->model,
                                   combo_box->priv->inserted_id);
-      combo_box->priv->inserted_id = -1;
-    }
-  if (combo_box->priv->deleted_id != -1)
-    {
       g_signal_handler_disconnect (combo_box->priv->model,
                                   combo_box->priv->deleted_id);
-      combo_box->priv->deleted_id = -1;
-    }
-  if (combo_box->priv->reordered_id != -1)
-    {
       g_signal_handler_disconnect (combo_box->priv->model,
                                   combo_box->priv->reordered_id);
-      combo_box->priv->reordered_id = -1;
-    }
-  if (combo_box->priv->changed_id != -1)
-    {
       g_signal_handler_disconnect (combo_box->priv->model,
                                   combo_box->priv->changed_id);
-      combo_box->priv->changed_id = -1;
     }
 
   /* menu mode */
@@ -1414,33 +1482,11 @@ gtk_combo_box_unset_model (GtkComboBox *combo_box)
         gtk_container_foreach (GTK_CONTAINER (combo_box->priv->popup_widget),
                                (GtkCallback)gtk_widget_destroy, NULL);
     }
-}
 
-static void
-gtk_combo_box_set_model_internal (GtkComboBox *combo_box)
-{
-  combo_box->priv->inserted_id =
-    g_signal_connect (combo_box->priv->model, "row_inserted",
-                     G_CALLBACK (gtk_combo_box_model_row_inserted),
-                     combo_box);
-  combo_box->priv->deleted_id =
-    g_signal_connect (combo_box->priv->model, "row_deleted",
-                     G_CALLBACK (gtk_combo_box_model_row_deleted),
-                     combo_box);
-  combo_box->priv->reordered_id =
-    g_signal_connect (combo_box->priv->model, "rows_reordered",
-                     G_CALLBACK (gtk_combo_box_model_rows_reordered),
-                     combo_box);
-  combo_box->priv->changed_id =
-    g_signal_connect (combo_box->priv->model, "row_changed",
-                     G_CALLBACK (gtk_combo_box_model_row_changed),
-                     combo_box);
-      
-  if (combo_box->priv->tree_view)
+  if (combo_box->priv->model)
     {
-      /* list mode */
-      gtk_tree_view_set_model (GTK_TREE_VIEW (combo_box->priv->tree_view),
-                               combo_box->priv->model);
+      g_object_unref (G_OBJECT (combo_box->priv->model));
+      combo_box->priv->model = NULL;
     }
 }
 
@@ -1568,18 +1614,17 @@ cell_view_sync_cells (GtkComboBox *combo_box,
 
 static void
 gtk_combo_box_menu_setup (GtkComboBox *combo_box,
-                          gboolean     add_childs)
+                          gboolean     add_children)
 {
   GtkWidget *box;
 
-  /* Unset any existing model. */
-  gtk_combo_box_unset_model (combo_box);
-
   if (combo_box->priv->cell_view)
     {
       combo_box->priv->button = gtk_toggle_button_new ();
       g_signal_connect (combo_box->priv->button, "toggled",
                         G_CALLBACK (gtk_combo_box_button_toggled), combo_box);
+      g_signal_connect_after (combo_box->priv->button, "key_press_event",
+                             G_CALLBACK (gtk_combo_box_key_press), combo_box);
       gtk_widget_set_parent (combo_box->priv->button,
                              GTK_BIN (combo_box)->child->parent);
 
@@ -1606,6 +1651,8 @@ gtk_combo_box_menu_setup (GtkComboBox *combo_box,
       combo_box->priv->button = gtk_toggle_button_new ();
       g_signal_connect (combo_box->priv->button, "toggled",
                         G_CALLBACK (gtk_combo_box_button_toggled), combo_box);
+      g_signal_connect_after (combo_box, "key_press_event",
+                             G_CALLBACK (gtk_combo_box_key_press), combo_box);
       gtk_widget_set_parent (combo_box->priv->button,
                              GTK_BIN (combo_box)->child->parent);
 
@@ -1621,13 +1668,12 @@ gtk_combo_box_menu_setup (GtkComboBox *combo_box,
 
   /* create our funky menu */
   box = gtk_menu_new ();
+  g_signal_connect (box, "key_press_event",
+                   G_CALLBACK (gtk_combo_box_menu_key_press), combo_box);
   gtk_combo_box_set_popup_widget (combo_box, box);
 
-  /* set the models */
-  gtk_combo_box_set_model_internal (combo_box);
-
   /* add items */
-  if (add_childs)
+  if (add_children)
     gtk_combo_box_menu_fill (combo_box);
 
 }
@@ -1660,6 +1706,10 @@ gtk_combo_box_menu_fill (GtkComboBox *combo_box)
                             GTK_CELL_VIEW (GTK_BIN (tmp)->child));
 
       gtk_menu_shell_append (GTK_MENU_SHELL (menu), tmp);
+
+      if (combo_box->priv->wrap_width)
+        gtk_combo_box_relayout_item (combo_box, i);
+
       gtk_widget_show (tmp);
 
       gtk_tree_path_free (path);
@@ -1669,9 +1719,6 @@ gtk_combo_box_menu_fill (GtkComboBox *combo_box)
 static void
 gtk_combo_box_menu_destroy (GtkComboBox *combo_box)
 {
-  /* disconnect signal handlers */
-  gtk_combo_box_unset_model (combo_box);
-
   g_signal_handlers_disconnect_matched (combo_box->priv->button,
                                         G_SIGNAL_MATCH_DATA,
                                         0, 0, NULL,
@@ -1698,7 +1745,7 @@ gtk_combo_box_menu_destroy (GtkComboBox *combo_box)
       combo_box->priv->arrow = NULL;
     }
 
-  /* changing the popup window will unref the menu and the childs */
+  /* changing the popup window will unref the menu and the children */
 }
 
 /*
@@ -1822,22 +1869,15 @@ gtk_combo_box_relayout_item (GtkComboBox *combo_box,
 static void
 gtk_combo_box_relayout (GtkComboBox *combo_box)
 {
-  gint i, items;
   GList *list, *j;
   GtkWidget *menu;
 
-  /* ensure we are in menu style */
+  /* do nothing unless we are in menu style */
   if (combo_box->priv->tree_view)
-    gtk_combo_box_list_destroy (combo_box);
+    return;
 
   menu = combo_box->priv->popup_widget;
 
-  if (!GTK_IS_MENU_SHELL (menu))
-    {
-      gtk_combo_box_menu_setup (combo_box, FALSE);
-      menu = combo_box->priv->popup_widget;
-    }
-
   /* get rid of all children */
   g_return_if_fail (GTK_IS_MENU_SHELL (menu));
 
@@ -1849,32 +1889,7 @@ gtk_combo_box_relayout (GtkComboBox *combo_box)
   g_list_free (list);
 
   /* and relayout */
-  items = gtk_tree_model_iter_n_children (combo_box->priv->model, NULL);
-
-  for (i = 0; i < items; i++)
-    {
-      GtkWidget *tmp;
-      GtkTreePath *path;
-
-      path = gtk_tree_path_new_from_indices (i, -1);
-      tmp = gtk_cell_view_menu_item_new_from_model (combo_box->priv->model,
-                                                    path);
-
-      g_signal_connect (tmp, "activate",
-                        G_CALLBACK (gtk_combo_box_menu_item_activate),
-                        combo_box);
-
-      cell_view_sync_cells (combo_box, GTK_CELL_VIEW (GTK_BIN (tmp)->child));
-
-      gtk_menu_shell_insert (GTK_MENU_SHELL (menu), tmp, i);
-
-      if (combo_box->priv->wrap_width)
-        gtk_combo_box_relayout_item (combo_box, i);
-
-      gtk_widget_show (tmp);
-
-      gtk_tree_path_free (path);
-    }
+  gtk_combo_box_menu_fill (combo_box);
 }
 
 /* callbacks */
@@ -1894,6 +1909,7 @@ gtk_combo_box_menu_button_press (GtkWidget      *widget,
       
       gtk_menu_set_active (GTK_MENU (combo_box->priv->popup_widget),
                           combo_box->priv->active_item);
+
       gtk_menu_popup (GTK_MENU (combo_box->priv->popup_widget),
                       NULL, NULL,
                       gtk_combo_box_menu_position, combo_box,
@@ -1929,16 +1945,12 @@ gtk_combo_box_model_row_inserted (GtkTreeModel     *model,
 {
   GtkComboBox *combo_box = GTK_COMBO_BOX (user_data);
   gint index = gtk_tree_path_get_indices (path)[0];
-  gint items = gtk_tree_model_iter_n_children (model, NULL);
 
   if (combo_box->priv->active_item >= index)
     combo_box->priv->active_item++;
-      
+
   if (!combo_box->priv->tree_view)
     gtk_combo_box_menu_row_inserted (model, path, iter, user_data);
-
-  if (items == 1)
-    gtk_combo_box_set_active (combo_box, 0);
 }
 
 static void
@@ -2112,9 +2124,6 @@ gtk_combo_box_list_setup (GtkComboBox *combo_box)
   GSList *i;
   GtkTreeSelection *sel;
 
-  /* Unset any existing model. */
-  gtk_combo_box_unset_model (combo_box);
-
   combo_box->priv->button = gtk_toggle_button_new ();
   gtk_widget_set_parent (combo_box->priv->button,
                          GTK_BIN (combo_box)->child->parent);
@@ -2122,6 +2131,8 @@ gtk_combo_box_list_setup (GtkComboBox *combo_box)
                     G_CALLBACK (gtk_combo_box_list_button_pressed), combo_box);
   g_signal_connect (combo_box->priv->button, "toggled",
                     G_CALLBACK (gtk_combo_box_button_toggled), combo_box);
+  g_signal_connect_after (combo_box, "key_press_event",
+                         G_CALLBACK (gtk_combo_box_key_press), combo_box);
 
   combo_box->priv->arrow = gtk_arrow_new (GTK_ARROW_DOWN, GTK_SHADOW_NONE);
   gtk_container_add (GTK_CONTAINER (combo_box->priv->button),
@@ -2148,7 +2159,10 @@ gtk_combo_box_list_setup (GtkComboBox *combo_box)
   gtk_tree_selection_set_mode (sel, GTK_SELECTION_SINGLE);
   gtk_tree_view_set_headers_visible (GTK_TREE_VIEW (combo_box->priv->tree_view),
                                      FALSE);
-
+  if (combo_box->priv->model)
+    gtk_tree_view_set_model (GTK_TREE_VIEW (combo_box->priv->tree_view),
+                            combo_box->priv->model);
+    
   g_signal_connect (combo_box->priv->tree_view, "button_press_event",
                     G_CALLBACK (gtk_combo_box_list_button_pressed),
                     combo_box);
@@ -2163,9 +2177,6 @@ gtk_combo_box_list_setup (GtkComboBox *combo_box)
   gtk_tree_view_append_column (GTK_TREE_VIEW (combo_box->priv->tree_view),
                                combo_box->priv->column);
 
-  /* set the models */
-  gtk_combo_box_set_model_internal (combo_box);
-
   /* sync up */
   for (i = combo_box->priv->cells; i; i = i->next)
     {
@@ -2211,8 +2222,6 @@ static void
 gtk_combo_box_list_destroy (GtkComboBox *combo_box)
 {
   /* disconnect signals */
-  gtk_combo_box_unset_model (combo_box);
-
   g_signal_handlers_disconnect_matched (combo_box->priv->tree_view,
                                         G_SIGNAL_MATCH_DATA,
                                         0, 0, NULL, NULL, combo_box);
@@ -2345,45 +2354,120 @@ gtk_combo_box_list_button_released (GtkWidget      *widget,
 }
 
 static gboolean
-gtk_combo_box_list_key_press (GtkWidget   *widget,
-                              GdkEventKey *event,
-                              gpointer     data)
+gtk_combo_box_key_press (GtkWidget   *widget,
+                        GdkEventKey *event,
+                        gpointer     data)
 {
   GtkComboBox *combo_box = GTK_COMBO_BOX (data);
+  guint state = event->state & gtk_accelerator_get_default_mod_mask ();
+  gint items = gtk_tree_model_iter_n_children (combo_box->priv->model, NULL);
+  gint index = gtk_combo_box_get_active (combo_box);
+  gint new_index;
 
-  if ((event->keyval == GDK_Return || event->keyval == GDK_KP_Enter ||
-       event->keyval == GDK_space || event->keyval == GDK_KP_Space) ||
-      event->keyval == GDK_Escape)
+  if ((event->keyval == GDK_Down || event->keyval == GDK_KP_Down) && 
+      state == GDK_MOD1_MASK)
     {
-      if (event->keyval != GDK_Escape)
-        {
-          gboolean ret;
-          GtkTreeIter iter;
-          GtkTreeModel *model = NULL;
-          GtkTreeSelection *sel = gtk_tree_view_get_selection (GTK_TREE_VIEW (combo_box->priv->tree_view));
+      gtk_combo_box_popup (combo_box);
 
-          ret = gtk_tree_selection_get_selected (sel, &model, &iter);
-          if (ret)
-            {
-              GtkTreePath *path;
-
-              path = gtk_tree_model_get_path (model, &iter);
-              if (path)
-                {
-                  gtk_combo_box_set_active (combo_box, gtk_tree_path_get_indices (path)[0]);
-                  gtk_tree_path_free (path);
-                }
-            }
-        }
-      else
-        /* reset active item -- this is incredibly lame and ugly */
-        gtk_combo_box_set_active (combo_box,
-                                  gtk_combo_box_get_active (combo_box));
+      return TRUE;
+    }
+
+  switch (event->keyval) 
+    {
+    case GDK_Down:
+    case GDK_KP_Down:
+      new_index = index + 1;
+      break;
+    case GDK_Up:
+    case GDK_KP_Up:
+      new_index = index - 1;
+      break;
+    case GDK_Page_Up:
+    case GDK_KP_Page_Up:
+    case GDK_Home: 
+    case GDK_KP_Home:
+      new_index = 0;
+      break;
+    case GDK_Page_Down:
+    case GDK_KP_Page_Down:
+    case GDK_End: 
+    case GDK_KP_End:
+      new_index = items - 1;
+      break;
+    default:
+      return FALSE;
+    }
+  
+  gtk_combo_box_set_active (combo_box, CLAMP (new_index, 0, items - 1));
+
+  return TRUE;
+}
+
+static gboolean
+gtk_combo_box_menu_key_press (GtkWidget   *widget,
+                             GdkEventKey *event,
+                             gpointer     data)
+{
+  GtkComboBox *combo_box = GTK_COMBO_BOX (data);
+  guint state = event->state & gtk_accelerator_get_default_mod_mask ();
 
+  if ((event->keyval == GDK_Up || event->keyval == GDK_KP_Up) && 
+      state == GDK_MOD1_MASK)
+    {
       gtk_combo_box_popdown (combo_box);
 
       return TRUE;
     }
+  
+  return FALSE;
+}
+
+static gboolean
+gtk_combo_box_list_key_press (GtkWidget   *widget,
+                              GdkEventKey *event,
+                              gpointer     data)
+{
+  GtkComboBox *combo_box = GTK_COMBO_BOX (data);
+  guint state = event->state & gtk_accelerator_get_default_mod_mask ();
+
+  if (event->keyval == GDK_Escape ||
+      ((event->keyval == GDK_Up || event->keyval == GDK_KP_Up) && 
+       state == GDK_MOD1_MASK))
+    {
+      /* reset active item -- this is incredibly lame and ugly */
+      gtk_combo_box_set_active (combo_box,
+                               gtk_combo_box_get_active (combo_box));
+      
+      gtk_combo_box_popdown (combo_box);
+      
+      return TRUE;
+    }
+    
+  if (event->keyval == GDK_Return || event->keyval == GDK_KP_Enter ||
+      event->keyval == GDK_space || event->keyval == GDK_KP_Space) 
+  {
+    gboolean ret;
+    GtkTreeIter iter;
+    GtkTreeModel *model = NULL;
+    GtkTreeSelection *sel = gtk_tree_view_get_selection (GTK_TREE_VIEW (combo_box->priv->tree_view));
+    
+    ret = gtk_tree_selection_get_selected (sel, &model, &iter);
+    if (ret)
+      {
+       GtkTreePath *path;
+       
+       path = gtk_tree_model_get_path (model, &iter);
+       if (path)
+         {
+           gtk_combo_box_set_active (combo_box, gtk_tree_path_get_indices (path)[0]);
+           gtk_tree_path_free (path);
+         }
+      }
+
+    gtk_combo_box_popdown (combo_box);
+    
+    return TRUE;
+  }
 
   return FALSE;
 }
@@ -2857,9 +2941,9 @@ gtk_combo_box_set_wrap_width (GtkComboBox *combo_box,
   if (width != combo_box->priv->wrap_width)
     {
       combo_box->priv->wrap_width = width;
-      
+
+      gtk_combo_box_check_appearance (combo_box);
       gtk_combo_box_relayout (combo_box);
-      gtk_combo_box_style_set (GTK_WIDGET (combo_box), NULL);
       
       g_object_notify (G_OBJECT (combo_box), "wrap_width");
     }
@@ -3096,18 +3180,45 @@ gtk_combo_box_set_model (GtkComboBox  *combo_box,
   g_return_if_fail (GTK_IS_COMBO_BOX (combo_box));
   g_return_if_fail (GTK_IS_TREE_MODEL (model));
 
+  if (model == combo_box->priv->model)
+    return;
+  
   if (combo_box->priv->model)
-    {
-      gtk_combo_box_unset_model (combo_box);
-      g_object_unref (G_OBJECT (combo_box->priv->model));
-    }
+    gtk_combo_box_unset_model (combo_box);
 
   combo_box->priv->model = model;
   g_object_ref (G_OBJECT (combo_box->priv->model));
 
-  gtk_combo_box_set_model_internal (combo_box);
-  if (!combo_box->priv->tree_view && combo_box->priv->popup_widget)
-    gtk_combo_box_menu_fill (combo_box);
+  combo_box->priv->inserted_id =
+    g_signal_connect (combo_box->priv->model, "row_inserted",
+                     G_CALLBACK (gtk_combo_box_model_row_inserted),
+                     combo_box);
+  combo_box->priv->deleted_id =
+    g_signal_connect (combo_box->priv->model, "row_deleted",
+                     G_CALLBACK (gtk_combo_box_model_row_deleted),
+                     combo_box);
+  combo_box->priv->reordered_id =
+    g_signal_connect (combo_box->priv->model, "rows_reordered",
+                     G_CALLBACK (gtk_combo_box_model_rows_reordered),
+                     combo_box);
+  combo_box->priv->changed_id =
+    g_signal_connect (combo_box->priv->model, "row_changed",
+                     G_CALLBACK (gtk_combo_box_model_row_changed),
+                     combo_box);
+      
+  if (combo_box->priv->tree_view)
+    {
+      /* list mode */
+      gtk_tree_view_set_model (GTK_TREE_VIEW (combo_box->priv->tree_view),
+                               combo_box->priv->model);
+    }
+  else
+    {
+      /* menu mode */
+      if (combo_box->priv->popup_widget)
+       gtk_combo_box_menu_fill (combo_box);
+
+    }
 
   if (combo_box->priv->cell_view)
     gtk_cell_view_set_model (GTK_CELL_VIEW (combo_box->priv->cell_view),
@@ -3301,9 +3412,13 @@ gtk_combo_box_destroy (GtkObject *object)
 {
   GtkComboBox *combo_box = GTK_COMBO_BOX (object);
 
+  combo_box->priv->destroying = 1;
+
   GTK_OBJECT_CLASS (parent_class)->destroy (object);
 
   combo_box->priv->cell_view = NULL;
+
+  combo_box->priv->destroying = 0;
 }
 
 static void
@@ -3312,8 +3427,6 @@ gtk_combo_box_finalize (GObject *object)
   GtkComboBox *combo_box = GTK_COMBO_BOX (object);
   GSList *i;
   
-  gtk_combo_box_unset_model (combo_box);
-
   if (GTK_IS_MENU (combo_box->priv->popup_widget))
     gtk_combo_box_menu_destroy (combo_box);
   
@@ -3323,6 +3436,8 @@ gtk_combo_box_finalize (GObject *object)
   if (combo_box->priv->popup_window)
     gtk_widget_destroy (combo_box->priv->popup_window);
 
+  gtk_combo_box_unset_model (combo_box);
+
   if (combo_box->priv->model)
     g_object_unref (combo_box->priv->model);