]> Pileus Git - ~andy/gtk/blobdiff - gtk/gtkcombobox.c
Deprecate widget flag: GTK_WIDGET_MAPPED
[~andy/gtk] / gtk / gtkcombobox.c
index 742de106e5170eb724682989c7a78d469e299e47..22416981e2be7fd5f43a5e52b81031b05fe1ec79 100644 (file)
@@ -17,7 +17,7 @@
  * Boston, MA 02111-1307, USA.
  */
 
-#include <config.h>
+#include "config.h"
 #include "gtkcombobox.h"
 
 #include "gtkarrow.h"
@@ -39,7 +39,6 @@
 #include "gtkvseparator.h"
 #include "gtkwindow.h"
 #include "gtkprivate.h"
-#include "gtkmarshal.h"
 
 #include <gdk/gdkkeysyms.h>
 
@@ -82,6 +81,7 @@ struct _GtkComboBoxPrivate
   gint wrap_width;
   GtkShadowType shadow_type;
 
+  gint active; /* Only temporary */
   GtkTreeRowReference *active_row;
 
   GtkWidget *tree_view;
@@ -104,6 +104,8 @@ struct _GtkComboBoxPrivate
   guint reordered_id;
   guint changed_id;
   guint popup_idle_id;
+  guint activate_button;
+  guint32 activate_time;
   guint scroll_timer;
   guint resize_idle_id;
 
@@ -119,10 +121,11 @@ struct _GtkComboBoxPrivate
   guint editing_canceled : 1;
   guint auto_scroll : 1;
   guint focus_on_click : 1;
+  guint button_sensitivity : 2;
 
   GtkTreeViewRowSeparatorFunc row_separator_func;
   gpointer                    row_separator_data;
-  GtkDestroyNotify            row_separator_destroy;
+  GDestroyNotify              row_separator_destroy;
 
   gchar *tearoff_title;
 };
@@ -200,7 +203,9 @@ enum {
   PROP_TEAROFF_TITLE,
   PROP_HAS_FRAME,
   PROP_FOCUS_ON_CLICK,
-  PROP_POPUP_SHOWN
+  PROP_POPUP_SHOWN,
+  PROP_BUTTON_SENSITIVITY,
+  PROP_EDITING_CANCELED
 };
 
 static guint combo_box_signals[LAST_SIGNAL] = {0,};
@@ -375,6 +380,8 @@ static gboolean gtk_combo_box_menu_button_press    (GtkWidget        *widget,
                                                     gpointer          user_data);
 static void     gtk_combo_box_menu_item_activate   (GtkWidget        *item,
                                                     gpointer          user_data);
+
+static void     gtk_combo_box_update_sensitivity   (GtkComboBox      *combo_box);
 static void     gtk_combo_box_menu_row_inserted    (GtkTreeModel     *model,
                                                     GtkTreePath      *path,
                                                     GtkTreeIter      *iter,
@@ -408,6 +415,7 @@ static void     gtk_combo_box_cell_layout_pack_start         (GtkCellLayout
 static void     gtk_combo_box_cell_layout_pack_end           (GtkCellLayout         *layout,
                                                               GtkCellRenderer       *cell,
                                                               gboolean               expand);
+static GList   *gtk_combo_box_cell_layout_get_cells          (GtkCellLayout         *layout);
 static void     gtk_combo_box_cell_layout_clear              (GtkCellLayout         *layout);
 static void     gtk_combo_box_cell_layout_add_attribute      (GtkCellLayout         *layout,
                                                               GtkCellRenderer       *cell,
@@ -526,34 +534,67 @@ gtk_combo_box_class_init (GtkComboBoxClass *klass)
                   NULL, NULL,
                   g_cclosure_marshal_VOID__VOID,
                   G_TYPE_NONE, 0);
-
+  /**
+   * GtkComboBox::move-active:
+   * @widget: the object that received the signal
+   * @scroll_type: a #GtkScrollType
+   *
+   * The ::move-active signal is a 
+   * <link linkend="keybinding-signals">keybinding signal</link>
+   * which gets emitted to move the active selection.
+   *
+   * Since: 2.12
+   */
   combo_box_signals[MOVE_ACTIVE] =
-    _gtk_binding_signal_new (I_("move-active"),
-                             G_OBJECT_CLASS_TYPE (klass),
-                             G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION,
-                             G_CALLBACK (gtk_combo_box_real_move_active),
-                             NULL, NULL,
-                             g_cclosure_marshal_VOID__ENUM,
-                             G_TYPE_NONE, 1,
-                             GTK_TYPE_SCROLL_TYPE);
+    g_signal_new_class_handler (I_("move-active"),
+                                G_OBJECT_CLASS_TYPE (klass),
+                                G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION,
+                                G_CALLBACK (gtk_combo_box_real_move_active),
+                                NULL, NULL,
+                                g_cclosure_marshal_VOID__ENUM,
+                                G_TYPE_NONE, 1,
+                                GTK_TYPE_SCROLL_TYPE);
 
+  /**
+   * GtkComboBox::popup:
+   * @widget: the object that received the signal
+   *
+   * The ::popup signal is a 
+   * <link linkend="keybinding-signals">keybinding signal</link>
+   * which gets emitted to popup the combo box list.
+   *
+   * The default binding for this signal is Alt+Down.
+   *
+   * Since: 2.12
+   */
   combo_box_signals[POPUP] =
-    _gtk_binding_signal_new (I_("popup"),
-                             G_OBJECT_CLASS_TYPE (klass),
-                             G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION,
-                             G_CALLBACK (gtk_combo_box_real_popup),
-                             NULL, NULL,
-                             g_cclosure_marshal_VOID__VOID,
-                             G_TYPE_NONE, 0);
-
+    g_signal_new_class_handler (I_("popup"),
+                                G_OBJECT_CLASS_TYPE (klass),
+                                G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION,
+                                G_CALLBACK (gtk_combo_box_real_popup),
+                                NULL, NULL,
+                                g_cclosure_marshal_VOID__VOID,
+                                G_TYPE_NONE, 0);
+  /**
+   * GtkComboBox::popdown:
+   * @button: the object which received the signal
+   *
+   * The ::popdown signal is a 
+   * <link linkend="keybinding-signals">keybinding signal</link> 
+   * which gets emitted to popdown the combo box list.
+   *
+   * The default bindings for this signal are Alt+Up and Escape.
+   *
+   * Since: 2.12
+   */
   combo_box_signals[POPDOWN] =
-    _gtk_binding_signal_new (I_("popdown"),
-                             G_OBJECT_CLASS_TYPE (klass),
-                             G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION,
-                             G_CALLBACK (gtk_combo_box_real_popdown),
-                             NULL, NULL,
-                             gtk_marshal_BOOLEAN__VOID,
-                             G_TYPE_BOOLEAN, 0);
+    g_signal_new_class_handler (I_("popdown"),
+                                G_OBJECT_CLASS_TYPE (klass),
+                                G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION,
+                                G_CALLBACK (gtk_combo_box_real_popdown),
+                                NULL, NULL,
+                                _gtk_marshal_BOOLEAN__VOID,
+                                G_TYPE_BOOLEAN, 0);
 
   /* key bindings */
   binding_set = gtk_binding_set_by_class (widget_class);
@@ -609,6 +650,10 @@ gtk_combo_box_class_init (GtkComboBoxClass *klass)
                                GTK_TYPE_SCROLL_TYPE, GTK_SCROLL_END);
 
   /* properties */
+  g_object_class_override_property (object_class,
+                                    PROP_EDITING_CANCELED,
+                                    "editing-canceled");
+
   /**
    * GtkComboBox:model:
    *
@@ -766,7 +811,7 @@ gtk_combo_box_class_init (GtkComboBoxClass *klass)
                                    g_param_spec_string ("tearoff-title",
                                                         P_("Tearoff Title"),
                                                         P_("A title that may be displayed by the window manager when the popup is torn-off"),
-                                                        "",
+                                                        NULL,
                                                         GTK_PARAM_READWRITE));
 
 
@@ -787,6 +832,24 @@ gtk_combo_box_class_init (GtkComboBoxClass *klass)
                                                          FALSE,
                                                          GTK_PARAM_READABLE));
 
+
+   /**
+    * GtkComboBox:button-sensitivity:
+    *
+    * Whether the dropdown button is sensitive when
+    * the model is empty.
+    *
+    * Since: 2.14
+    */
+   g_object_class_install_property (object_class,
+                                    PROP_BUTTON_SENSITIVITY,
+                                    g_param_spec_enum ("button-sensitivity",
+                                                       P_("Button Sensitivity"),
+                                                       P_("Whether the dropdown button is sensitive when the model is empty"),
+                                                       GTK_TYPE_SENSITIVITY_TYPE,
+                                                       GTK_SENSITIVITY_AUTO,
+                                                       GTK_PARAM_READWRITE));
+
   gtk_widget_class_install_style_property (widget_class,
                                            g_param_spec_boolean ("appears-as-list",
                                                                  P_("Appears as list"),
@@ -835,7 +898,7 @@ static void
 gtk_combo_box_buildable_init (GtkBuildableIface *iface)
 {
   parent_buildable_iface = g_type_interface_peek_parent (iface);
-  iface->add = _gtk_cell_layout_buildable_add;
+  iface->add_child = _gtk_cell_layout_buildable_add_child;
   iface->custom_tag_start = gtk_combo_box_buildable_custom_tag_start;
   iface->custom_tag_end = gtk_combo_box_buildable_custom_tag_end;
 }
@@ -845,6 +908,7 @@ gtk_combo_box_cell_layout_init (GtkCellLayoutIface *iface)
 {
   iface->pack_start = gtk_combo_box_cell_layout_pack_start;
   iface->pack_end = gtk_combo_box_cell_layout_pack_end;
+  iface->get_cells = gtk_combo_box_cell_layout_get_cells;
   iface->clear = gtk_combo_box_cell_layout_clear;
   iface->add_attribute = gtk_combo_box_cell_layout_add_attribute;
   iface->set_cell_data_func = gtk_combo_box_cell_layout_set_cell_data_func;
@@ -861,28 +925,32 @@ gtk_combo_box_cell_editable_init (GtkCellEditableIface *iface)
 static void
 gtk_combo_box_init (GtkComboBox *combo_box)
 {
-  combo_box->priv = GTK_COMBO_BOX_GET_PRIVATE (combo_box);
+  GtkComboBoxPrivate *priv = GTK_COMBO_BOX_GET_PRIVATE (combo_box);
+
+  priv->cell_view = gtk_cell_view_new ();
+  gtk_widget_set_parent (priv->cell_view, GTK_WIDGET (combo_box));
+  GTK_BIN (combo_box)->child = priv->cell_view;
+  gtk_widget_show (priv->cell_view);
 
-  combo_box->priv->cell_view = gtk_cell_view_new ();
-  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);
+  priv->width = 0;
+  priv->height = 0;
+  priv->wrap_width = 0;
 
-  combo_box->priv->width = 0;
-  combo_box->priv->height = 0;
-  combo_box->priv->wrap_width = 0;
+  priv->active = -1;
+  priv->active_row = NULL;
+  priv->col_column = -1;
+  priv->row_column = -1;
 
-  combo_box->priv->active_row = NULL;
-  combo_box->priv->col_column = -1;
-  combo_box->priv->row_column = -1;
+  priv->popup_shown = FALSE;
+  priv->add_tearoffs = FALSE;
+  priv->has_frame = TRUE;
+  priv->is_cell_renderer = FALSE;
+  priv->editing_canceled = FALSE;
+  priv->auto_scroll = FALSE;
+  priv->focus_on_click = TRUE;
+  priv->button_sensitivity = GTK_SENSITIVITY_AUTO;
 
-  combo_box->priv->popup_shown = FALSE;
-  combo_box->priv->add_tearoffs = FALSE;
-  combo_box->priv->has_frame = TRUE;
-  combo_box->priv->is_cell_renderer = FALSE;
-  combo_box->priv->editing_canceled = FALSE;
-  combo_box->priv->auto_scroll = FALSE;
-  combo_box->priv->focus_on_click = TRUE;
+  combo_box->priv = priv;
 
   gtk_combo_box_check_appearance (combo_box);
 }
@@ -897,56 +965,62 @@ gtk_combo_box_set_property (GObject      *object,
 
   switch (prop_id)
     {
-      case PROP_MODEL:
-        gtk_combo_box_set_model (combo_box, g_value_get_object (value));
-        break;
+    case PROP_MODEL:
+      gtk_combo_box_set_model (combo_box, g_value_get_object (value));
+      break;
 
-      case PROP_WRAP_WIDTH:
-        gtk_combo_box_set_wrap_width (combo_box, g_value_get_int (value));
-        break;
+    case PROP_WRAP_WIDTH:
+      gtk_combo_box_set_wrap_width (combo_box, g_value_get_int (value));
+      break;
 
-      case PROP_ROW_SPAN_COLUMN:
-        gtk_combo_box_set_row_span_column (combo_box, g_value_get_int (value));
-        break;
+    case PROP_ROW_SPAN_COLUMN:
+      gtk_combo_box_set_row_span_column (combo_box, g_value_get_int (value));
+      break;
 
-      case PROP_COLUMN_SPAN_COLUMN:
-        gtk_combo_box_set_column_span_column (combo_box, g_value_get_int (value));
-        break;
+    case PROP_COLUMN_SPAN_COLUMN:
+      gtk_combo_box_set_column_span_column (combo_box, g_value_get_int (value));
+      break;
 
-      case PROP_ACTIVE:
-        gtk_combo_box_set_active (combo_box, g_value_get_int (value));
-        break;
+    case PROP_ACTIVE:
+      gtk_combo_box_set_active (combo_box, g_value_get_int (value));
+      break;
 
-      case PROP_ADD_TEAROFFS:
-        gtk_combo_box_set_add_tearoffs (combo_box, g_value_get_boolean (value));
-        break;
+    case PROP_ADD_TEAROFFS:
+      gtk_combo_box_set_add_tearoffs (combo_box, g_value_get_boolean (value));
+      break;
 
-      case PROP_HAS_FRAME:
-        combo_box->priv->has_frame = g_value_get_boolean (value);
-        break;
+    case PROP_HAS_FRAME:
+      combo_box->priv->has_frame = g_value_get_boolean (value);
+      break;
 
-      case PROP_FOCUS_ON_CLICK:
-       gtk_combo_box_set_focus_on_click (combo_box, 
-                                         g_value_get_boolean (value));
-        break;
+    case PROP_FOCUS_ON_CLICK:
+      gtk_combo_box_set_focus_on_click (combo_box,
+                                        g_value_get_boolean (value));
+      break;
 
-      case PROP_TEAROFF_TITLE:
-       gtk_combo_box_set_title (combo_box, g_value_get_string (value));
-        break;
+    case PROP_TEAROFF_TITLE:
+      gtk_combo_box_set_title (combo_box, g_value_get_string (value));
+      break;
 
-      case PROP_POPUP_SHOWN:
-        if (g_value_get_boolean (value))
-          {
-            gtk_combo_box_popup (combo_box);
-          }
-        else
-          {
-            gtk_combo_box_popdown (combo_box);
-          }
-        break;
+    case PROP_POPUP_SHOWN:
+      if (g_value_get_boolean (value))
+        gtk_combo_box_popup (combo_box);
+      else
+        gtk_combo_box_popdown (combo_box);
+      break;
 
-      default:
-        break;
+    case PROP_BUTTON_SENSITIVITY:
+      gtk_combo_box_set_button_sensitivity (combo_box,
+                                            g_value_get_enum (value));
+      break;
+
+    case PROP_EDITING_CANCELED:
+      combo_box->priv->editing_canceled = g_value_get_boolean (value);
+      break;
+
+    default:
+      G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
+      break;
     }
 }
 
@@ -957,6 +1031,7 @@ gtk_combo_box_get_property (GObject    *object,
                             GParamSpec *pspec)
 {
   GtkComboBox *combo_box = GTK_COMBO_BOX (object);
+  GtkComboBoxPrivate *priv = GTK_COMBO_BOX_GET_PRIVATE (combo_box);
 
   switch (prop_id)
     {
@@ -1000,6 +1075,14 @@ gtk_combo_box_get_property (GObject    *object,
         g_value_set_boolean (value, combo_box->priv->popup_shown);
         break;
 
+      case PROP_BUTTON_SENSITIVITY:
+        g_value_set_enum (value, combo_box->priv->button_sensitivity);
+        break;
+
+      case PROP_EDITING_CANCELED:
+        g_value_set_boolean (value, priv->editing_canceled);
+        break;
+
       default:
         G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
         break;
@@ -1011,11 +1094,12 @@ gtk_combo_box_state_changed (GtkWidget    *widget,
                             GtkStateType  previous)
 {
   GtkComboBox *combo_box = GTK_COMBO_BOX (widget);
+  GtkComboBoxPrivate *priv = combo_box->priv;
 
   if (GTK_WIDGET_REALIZED (widget))
     {
-      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), 
+      if (priv->tree_view && priv->cell_view)
+       gtk_cell_view_set_background_color (GTK_CELL_VIEW (priv->cell_view), 
                                            &widget->style->base[GTK_WIDGET_STATE (widget)]);
     }
 
@@ -1028,18 +1112,18 @@ gtk_combo_box_button_state_changed (GtkWidget    *widget,
                                    gpointer      data)
 {
   GtkComboBox *combo_box = GTK_COMBO_BOX (data);
+  GtkComboBoxPrivate *priv = combo_box->priv;
 
   if (GTK_WIDGET_REALIZED (widget))
     {
-      if (!combo_box->priv->tree_view && combo_box->priv->cell_view)
+      if (!priv->tree_view && priv->cell_view)
        {
          if ((GTK_WIDGET_STATE (widget) == GTK_STATE_INSENSITIVE) !=
-             (GTK_WIDGET_STATE (combo_box->priv->cell_view) == GTK_STATE_INSENSITIVE))
-           gtk_widget_set_sensitive (combo_box->priv->cell_view, GTK_WIDGET_SENSITIVE (widget));
+             (GTK_WIDGET_STATE (priv->cell_view) == GTK_STATE_INSENSITIVE))
+           gtk_widget_set_sensitive (priv->cell_view, gtk_widget_get_sensitive (widget));
          
-         gtk_widget_set_state (combo_box->priv->cell_view, 
+         gtk_widget_set_state (priv->cell_view, 
                                GTK_WIDGET_STATE (widget));
-         
        }
     }
 
@@ -1049,12 +1133,13 @@ gtk_combo_box_button_state_changed (GtkWidget    *widget,
 static void
 gtk_combo_box_check_appearance (GtkComboBox *combo_box)
 {
+  GtkComboBoxPrivate *priv = combo_box->priv;
   gboolean appears_as_list;
 
   /* if wrap_width > 0, then we are in grid-mode and forced to use
    * unix style
    */
-  if (combo_box->priv->wrap_width)
+  if (priv->wrap_width)
     appears_as_list = FALSE;
   else
     gtk_widget_style_get (GTK_WIDGET (combo_box),
@@ -1064,26 +1149,26 @@ gtk_combo_box_check_appearance (GtkComboBox *combo_box)
   if (appears_as_list)
     {
       /* Destroy all the menu mode widgets, if they exist. */
-      if (GTK_IS_MENU (combo_box->priv->popup_widget))
+      if (GTK_IS_MENU (priv->popup_widget))
        gtk_combo_box_menu_destroy (combo_box);
 
       /* Create the list mode widgets, if they don't already exist. */
-      if (!GTK_IS_TREE_VIEW (combo_box->priv->tree_view))
+      if (!GTK_IS_TREE_VIEW (priv->tree_view))
        gtk_combo_box_list_setup (combo_box);
     }
   else
     {
       /* Destroy all the list mode widgets, if they exist. */
-      if (GTK_IS_TREE_VIEW (combo_box->priv->tree_view))
+      if (GTK_IS_TREE_VIEW (priv->tree_view))
        gtk_combo_box_list_destroy (combo_box);
 
       /* Create the menu mode widgets, if they don't already exist. */
-      if (!GTK_IS_MENU (combo_box->priv->popup_widget))
+      if (!GTK_IS_MENU (priv->popup_widget))
        gtk_combo_box_menu_setup (combo_box, TRUE);
     }
 
   gtk_widget_style_get (GTK_WIDGET (combo_box),
-                       "shadow-type", &combo_box->priv->shadow_type,
+                       "shadow-type", &priv->shadow_type,
                        NULL);
 }
 
@@ -1092,16 +1177,17 @@ gtk_combo_box_style_set (GtkWidget *widget,
                          GtkStyle  *previous)
 {
   GtkComboBox *combo_box = GTK_COMBO_BOX (widget);
+  GtkComboBoxPrivate *priv = combo_box->priv;
 
   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), 
+  if (priv->tree_view && priv->cell_view)
+    gtk_cell_view_set_background_color (GTK_CELL_VIEW (priv->cell_view), 
                                        &widget->style->base[GTK_WIDGET_STATE (widget)]);
 
   if (GTK_IS_ENTRY (GTK_BIN (combo_box)->child))
     g_object_set (GTK_BIN (combo_box)->child, "shadow-type",
-                  GTK_SHADOW_NONE == combo_box->priv->shadow_type ?
+                  GTK_SHADOW_NONE == priv->shadow_type ?
                   GTK_SHADOW_IN : GTK_SHADOW_NONE, NULL);
 }
 
@@ -1125,10 +1211,11 @@ gtk_combo_box_add (GtkContainer *container,
                    GtkWidget    *widget)
 {
   GtkComboBox *combo_box = GTK_COMBO_BOX (container);
+  GtkComboBoxPrivate *priv = combo_box->priv;
 
-  if (combo_box->priv->cell_view && combo_box->priv->cell_view->parent)
+  if (priv->cell_view && priv->cell_view->parent)
     {
-      gtk_widget_unparent (combo_box->priv->cell_view);
+      gtk_widget_unparent (priv->cell_view);
       GTK_BIN (container)->child = NULL;
       gtk_widget_queue_resize (GTK_WIDGET (container));
     }
@@ -1136,25 +1223,25 @@ gtk_combo_box_add (GtkContainer *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)
+  if (priv->cell_view &&
+      widget != priv->cell_view)
     {
       /* since the cell_view was unparented, it's gone now */
-      combo_box->priv->cell_view = NULL;
+      priv->cell_view = NULL;
 
-      if (!combo_box->priv->tree_view && combo_box->priv->separator)
+      if (!priv->tree_view && priv->separator)
         {
-         gtk_container_remove (GTK_CONTAINER (combo_box->priv->separator->parent),
-                               combo_box->priv->separator);
-         combo_box->priv->separator = NULL;
+         gtk_container_remove (GTK_CONTAINER (priv->separator->parent),
+                               priv->separator);
+         priv->separator = NULL;
 
           gtk_widget_queue_resize (GTK_WIDGET (container));
         }
-      else if (combo_box->priv->cell_view_frame)
+      else if (priv->cell_view_frame)
         {
-          gtk_widget_unparent (combo_box->priv->cell_view_frame);
-          combo_box->priv->cell_view_frame = NULL;
-          combo_box->priv->box = NULL;
+          gtk_widget_unparent (priv->cell_view_frame);
+          priv->cell_view_frame = NULL;
+          priv->box = NULL;
         }
     }
 }
@@ -1164,11 +1251,12 @@ gtk_combo_box_remove (GtkContainer *container,
                      GtkWidget    *widget)
 {
   GtkComboBox *combo_box = GTK_COMBO_BOX (container);
+  GtkComboBoxPrivate *priv = combo_box->priv;
   GtkTreePath *path;
   gboolean appears_as_list;
 
-  if (widget == combo_box->priv->cell_view)
-    combo_box->priv->cell_view = NULL;
+  if (widget == priv->cell_view)
+    priv->cell_view = NULL;
 
   gtk_widget_unparent (widget);
   GTK_BIN (container)->child = NULL;
@@ -1178,30 +1266,30 @@ gtk_combo_box_remove (GtkContainer *container,
 
   gtk_widget_queue_resize (GTK_WIDGET (container));
 
-  if (!combo_box->priv->tree_view)
+  if (!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))
+  else if (GTK_IS_MENU (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;
+      gtk_menu_detach (GTK_MENU (priv->popup_widget));
+      priv->popup_widget = NULL;
     }
 
-  if (!combo_box->priv->cell_view)
+  if (!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;
+      priv->cell_view = gtk_cell_view_new ();
+      gtk_widget_set_parent (priv->cell_view, GTK_WIDGET (container));
+      GTK_BIN (container)->child = 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);
-      gtk_combo_box_sync_cells (combo_box, GTK_CELL_LAYOUT (combo_box->priv->cell_view));
+      gtk_widget_show (priv->cell_view);
+      gtk_cell_view_set_model (GTK_CELL_VIEW (priv->cell_view),
+                              priv->model);
+      gtk_combo_box_sync_cells (combo_box, GTK_CELL_LAYOUT (priv->cell_view));
     }
 
 
@@ -1210,9 +1298,9 @@ gtk_combo_box_remove (GtkContainer *container,
   else
     gtk_combo_box_menu_setup (combo_box, TRUE);
 
-  if (gtk_tree_row_reference_valid (combo_box->priv->active_row))
+  if (gtk_tree_row_reference_valid (priv->active_row))
     {
-      path = gtk_tree_row_reference_get_path (combo_box->priv->active_row);
+      path = gtk_tree_row_reference_get_path (priv->active_row);
       gtk_combo_box_set_active_internal (combo_box, path);
       gtk_tree_path_free (path);
     }
@@ -1242,13 +1330,14 @@ gtk_combo_box_menu_show (GtkWidget *menu,
                          gpointer   user_data)
 {
   GtkComboBox *combo_box = GTK_COMBO_BOX (user_data);
+  GtkComboBoxPrivate *priv = combo_box->priv;
 
   gtk_combo_box_child_show (menu, user_data);
 
-  combo_box->priv->popup_in_progress = TRUE;
-  gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (combo_box->priv->button),
+  priv->popup_in_progress = TRUE;
+  gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (priv->button),
                                 TRUE);
-  combo_box->priv->popup_in_progress = FALSE;
+  priv->popup_in_progress = FALSE;
 }
 
 static void
@@ -1257,7 +1346,7 @@ gtk_combo_box_menu_hide (GtkWidget *menu,
 {
   GtkComboBox *combo_box = GTK_COMBO_BOX (user_data);
 
-  gtk_combo_box_child_hide(menu,user_data);
+  gtk_combo_box_child_hide (menu,user_data);
 
   gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (combo_box->priv->button),
                                 FALSE);
@@ -1267,12 +1356,10 @@ static void
 gtk_combo_box_detacher (GtkWidget *widget,
                        GtkMenu   *menu)
 {
-  GtkComboBox *combo_box;
-
-  g_return_if_fail (GTK_IS_COMBO_BOX (widget));
+  GtkComboBox *combo_box = GTK_COMBO_BOX (widget);
+  GtkComboBoxPrivate *priv = combo_box->priv;
 
-  combo_box = GTK_COMBO_BOX (widget);
-  g_return_if_fail (combo_box->priv->popup_widget == (GtkWidget*) menu);
+  g_return_if_fail (priv->popup_widget == (GtkWidget *) menu);
 
   g_signal_handlers_disconnect_by_func (menu->toplevel,
                                        gtk_combo_box_menu_show,
@@ -1281,35 +1368,37 @@ gtk_combo_box_detacher (GtkWidget *widget,
                                        gtk_combo_box_menu_hide,
                                        combo_box);
   
-  combo_box->priv->popup_widget = NULL;
+  priv->popup_widget = NULL;
 }
 
 static void
 gtk_combo_box_set_popup_widget (GtkComboBox *combo_box,
                                 GtkWidget   *popup)
 {
-  if (GTK_IS_MENU (combo_box->priv->popup_widget))
+  GtkComboBoxPrivate *priv = combo_box->priv;
+
+  if (GTK_IS_MENU (priv->popup_widget))
     {
-      gtk_menu_detach (GTK_MENU (combo_box->priv->popup_widget));
-      combo_box->priv->popup_widget = NULL;
+      gtk_menu_detach (GTK_MENU (priv->popup_widget));
+      priv->popup_widget = NULL;
     }
-  else if (combo_box->priv->popup_widget)
+  else if (priv->popup_widget)
     {
-      gtk_container_remove (GTK_CONTAINER (combo_box->priv->scrolled_window),
-                            combo_box->priv->popup_widget);
-      g_object_unref (combo_box->priv->popup_widget);
-      combo_box->priv->popup_widget = NULL;
+      gtk_container_remove (GTK_CONTAINER (priv->scrolled_window),
+                            priv->popup_widget);
+      g_object_unref (priv->popup_widget);
+      priv->popup_widget = NULL;
     }
 
   if (GTK_IS_MENU (popup))
     {
-      if (combo_box->priv->popup_window)
+      if (priv->popup_window)
         {
-          gtk_widget_destroy (combo_box->priv->popup_window);
-          combo_box->priv->popup_window = NULL;
+          gtk_widget_destroy (priv->popup_window);
+          priv->popup_window = NULL;
         }
 
-      combo_box->priv->popup_widget = popup;
+      priv->popup_widget = popup;
 
       /* 
        * Note that we connect to show/hide on the toplevel, not the
@@ -1327,20 +1416,20 @@ gtk_combo_box_set_popup_widget (GtkComboBox *combo_box,
     }
   else
     {
-      if (!combo_box->priv->popup_window)
+      if (!priv->popup_window)
         {
          GtkWidget *toplevel;
          
-          combo_box->priv->popup_window = gtk_window_new (GTK_WINDOW_POPUP);
-          gtk_widget_set_name (combo_box->priv->popup_window, "gtk-combobox-popup-window");
+          priv->popup_window = gtk_window_new (GTK_WINDOW_POPUP);
+          gtk_widget_set_name (priv->popup_window, "gtk-combobox-popup-window");
 
-         gtk_window_set_type_hint (GTK_WINDOW (combo_box->priv->popup_window),
+         gtk_window_set_type_hint (GTK_WINDOW (priv->popup_window),
                                    GDK_WINDOW_TYPE_HINT_COMBO);
 
-         g_signal_connect (GTK_WINDOW(combo_box->priv->popup_window),"show",
+         g_signal_connect (GTK_WINDOW (priv->popup_window),"show",
                            G_CALLBACK (gtk_combo_box_child_show),
                            combo_box);
-         g_signal_connect (GTK_WINDOW(combo_box->priv->popup_window),"hide",
+         g_signal_connect (GTK_WINDOW (priv->popup_window),"hide",
                            G_CALLBACK (gtk_combo_box_child_hide),
                            combo_box);
          
@@ -1348,35 +1437,35 @@ gtk_combo_box_set_popup_widget (GtkComboBox *combo_box,
          if (GTK_IS_WINDOW (toplevel))
            {
              gtk_window_group_add_window (gtk_window_get_group (GTK_WINDOW (toplevel)), 
-                                          GTK_WINDOW (combo_box->priv->popup_window));
-             gtk_window_set_transient_for (GTK_WINDOW (combo_box->priv->popup_window),
+                                          GTK_WINDOW (priv->popup_window));
+             gtk_window_set_transient_for (GTK_WINDOW (priv->popup_window),
                                            GTK_WINDOW (toplevel));
            }
 
-         gtk_window_set_resizable (GTK_WINDOW (combo_box->priv->popup_window), FALSE);
-          gtk_window_set_screen (GTK_WINDOW (combo_box->priv->popup_window),
+         gtk_window_set_resizable (GTK_WINDOW (priv->popup_window), FALSE);
+          gtk_window_set_screen (GTK_WINDOW (priv->popup_window),
                                  gtk_widget_get_screen (GTK_WIDGET (combo_box)));
 
-         combo_box->priv->scrolled_window = gtk_scrolled_window_new (NULL, NULL);
+         priv->scrolled_window = gtk_scrolled_window_new (NULL, NULL);
          
-         gtk_scrolled_window_set_policy (GTK_SCROLLED_WINDOW (combo_box->priv->scrolled_window),
+         gtk_scrolled_window_set_policy (GTK_SCROLLED_WINDOW (priv->scrolled_window),
                                          GTK_POLICY_NEVER,
                                          GTK_POLICY_NEVER);
-         gtk_scrolled_window_set_shadow_type (GTK_SCROLLED_WINDOW (combo_box->priv->scrolled_window),
+         gtk_scrolled_window_set_shadow_type (GTK_SCROLLED_WINDOW (priv->scrolled_window),
                                               GTK_SHADOW_IN);
 
-          gtk_widget_show (combo_box->priv->scrolled_window);
+          gtk_widget_show (priv->scrolled_window);
          
-         gtk_container_add (GTK_CONTAINER (combo_box->priv->popup_window),
-                            combo_box->priv->scrolled_window);
+         gtk_container_add (GTK_CONTAINER (priv->popup_window),
+                            priv->scrolled_window);
         }
 
-      gtk_container_add (GTK_CONTAINER (combo_box->priv->scrolled_window),
+      gtk_container_add (GTK_CONTAINER (priv->scrolled_window),
                          popup);
 
       gtk_widget_show (popup);
       g_object_ref (popup);
-      combo_box->priv->popup_widget = popup;
+      priv->popup_widget = popup;
     }
 }
 
@@ -1396,26 +1485,28 @@ gtk_combo_box_menu_position_below (GtkMenu  *menu,
   GdkRectangle monitor;
   
   /* FIXME: is using the size request here broken? */
-   child = GTK_BIN (combo_box)->child;
-   
-   gdk_window_get_origin (child->window, &sx, &sy);
-   
-   if (GTK_WIDGET_NO_WINDOW (child))
-      {
-       sx += child->allocation.x;
-       sy += child->allocation.y;
-      }
+  child = GTK_BIN (combo_box)->child;
 
-   if (GTK_SHADOW_NONE != combo_box->priv->shadow_type)
-     sx -= GTK_WIDGET (combo_box)->style->xthickness;
+  sx = sy = 0;
 
-   gtk_widget_size_request (GTK_WIDGET (menu), &req);
+  if (!gtk_widget_get_has_window (child))
+    {
+      sx += child->allocation.x;
+      sy += child->allocation.y;
+    }
+
+  gdk_window_get_root_coords (child->window, sx, sy, &sx, &sy);
+
+  if (GTK_SHADOW_NONE != combo_box->priv->shadow_type)
+    sx -= GTK_WIDGET (combo_box)->style->xthickness;
 
-   if (gtk_widget_get_direction (GTK_WIDGET (combo_box)) == GTK_TEXT_DIR_LTR)
-     *x = sx;
-   else
-     *x = sx + child->allocation.width - req.width;
-   *y = sy;
+  gtk_widget_size_request (GTK_WIDGET (menu), &req);
+
+  if (gtk_widget_get_direction (GTK_WIDGET (combo_box)) == GTK_TEXT_DIR_LTR)
+    *x = sx;
+  else
+    *x = sx + child->allocation.width - req.width;
+  *y = sy;
 
   screen = gtk_widget_get_screen (GTK_WIDGET (combo_box));
   monitor_num = gdk_screen_get_monitor_at_window (screen, 
@@ -1457,8 +1548,6 @@ gtk_combo_box_menu_position_over (GtkMenu  *menu,
   gint menu_ypos;
   gint menu_width;
 
-  g_return_if_fail (GTK_IS_COMBO_BOX (user_data));
-  
   combo_box = GTK_COMBO_BOX (user_data);
   widget = GTK_WIDGET (combo_box);
 
@@ -1466,10 +1555,9 @@ gtk_combo_box_menu_position_over (GtkMenu  *menu,
   menu_width = requisition.width;
 
   active = gtk_menu_get_active (GTK_MENU (combo_box->priv->popup_widget));
-  gdk_window_get_origin (widget->window, &menu_xpos, &menu_ypos);
 
-  menu_xpos += widget->allocation.x;
-  menu_ypos += widget->allocation.y + widget->allocation.height / 2 - 2;
+  menu_xpos = widget->allocation.x;
+  menu_ypos = widget->allocation.y + widget->allocation.height / 2 - 2;
 
   if (active != NULL)
     {
@@ -1485,7 +1573,7 @@ gtk_combo_box_menu_position_over (GtkMenu  *menu,
       if (active == child)
        break;
 
-      if (GTK_WIDGET_VISIBLE (child))
+      if (gtk_widget_get_visible (child))
        {
          gtk_widget_get_child_requisition (child, &requisition);
          menu_ypos -= requisition.height;
@@ -1497,6 +1585,9 @@ gtk_combo_box_menu_position_over (GtkMenu  *menu,
   if (gtk_widget_get_direction (widget) == GTK_TEXT_DIR_RTL)
     menu_xpos = menu_xpos + widget->allocation.width - menu_width;
 
+  gdk_window_get_root_coords (widget->window, menu_xpos, menu_ypos,
+                             &menu_xpos, &menu_ypos);
+
   /* Clamp the position on screen */
   screen_width = gdk_screen_get_width (gtk_widget_get_screen (widget));
   
@@ -1518,24 +1609,27 @@ gtk_combo_box_menu_position (GtkMenu  *menu,
                             gint     *push_in,
                             gpointer  user_data)
 {
-  GtkComboBox *combo_box;
+  GtkComboBox *combo_box = GTK_COMBO_BOX (user_data);
+  GtkComboBoxPrivate *priv = combo_box->priv;
   GtkWidget *menu_item;
 
-  combo_box = GTK_COMBO_BOX (user_data);
 
-  if (combo_box->priv->wrap_width > 0 || combo_box->priv->cell_view == NULL)   
+  if (priv->wrap_width > 0 || priv->cell_view == NULL) 
     gtk_combo_box_menu_position_below (menu, x, y, push_in, user_data);
   else
     {
       /* FIXME handle nested menus better */
-      menu_item = gtk_menu_get_active (GTK_MENU (combo_box->priv->popup_widget));
+      menu_item = gtk_menu_get_active (GTK_MENU (priv->popup_widget));
       if (menu_item)
-       gtk_menu_shell_select_item (GTK_MENU_SHELL (combo_box->priv->popup_widget), 
+       gtk_menu_shell_select_item (GTK_MENU_SHELL (priv->popup_widget), 
                                    menu_item);
 
       gtk_combo_box_menu_position_over (menu, x, y, push_in, user_data);
     }
 
+  if (!gtk_widget_get_visible (GTK_MENU (priv->popup_widget)->toplevel))
+    gtk_window_set_type_hint (GTK_WINDOW (GTK_MENU (priv->popup_widget)->toplevel),
+                              GDK_WINDOW_TYPE_HINT_COMBO);
 }
 
 static void
@@ -1545,6 +1639,7 @@ gtk_combo_box_list_position (GtkComboBox *combo_box,
                             gint        *width,
                             gint        *height)
 {
+  GtkComboBoxPrivate *priv = combo_box->priv;
   GdkScreen *screen;
   gint monitor_num;
   GdkRectangle monitor;
@@ -1555,27 +1650,29 @@ gtk_combo_box_list_position (GtkComboBox *combo_box,
      see bug #340204 */
   GtkWidget *sample = GTK_WIDGET (combo_box);
 
-  gdk_window_get_origin (sample->window, x, y);
+  *x = *y = 0;
 
-  if (GTK_WIDGET_NO_WINDOW (sample))
+  if (!gtk_widget_get_has_window (sample))
     {
       *x += sample->allocation.x;
       *y += sample->allocation.y;
     }
   
+  gdk_window_get_root_coords (sample->window, *x, *y, x, y);
+
   *width = sample->allocation.width;
 
   hpolicy = vpolicy = GTK_POLICY_NEVER;
-  gtk_scrolled_window_set_policy (GTK_SCROLLED_WINDOW (combo_box->priv->scrolled_window),
+  gtk_scrolled_window_set_policy (GTK_SCROLLED_WINDOW (priv->scrolled_window),
                                  hpolicy, vpolicy);
-  gtk_widget_size_request (combo_box->priv->scrolled_window, &popup_req);
+  gtk_widget_size_request (priv->scrolled_window, &popup_req);
 
   if (popup_req.width > *width)
     {
       hpolicy = GTK_POLICY_ALWAYS;
-      gtk_scrolled_window_set_policy (GTK_SCROLLED_WINDOW (combo_box->priv->scrolled_window),
+      gtk_scrolled_window_set_policy (GTK_SCROLLED_WINDOW (priv->scrolled_window),
                                      hpolicy, vpolicy);
-      gtk_widget_size_request (combo_box->priv->scrolled_window, &popup_req);
+      gtk_widget_size_request (priv->scrolled_window, &popup_req);
     }
 
   *height = popup_req.height;
@@ -1609,7 +1706,7 @@ gtk_combo_box_list_position (GtkComboBox *combo_box,
     {
       vpolicy = GTK_POLICY_ALWAYS;
       
-      gtk_scrolled_window_set_policy (GTK_SCROLLED_WINDOW (combo_box->priv->scrolled_window),
+      gtk_scrolled_window_set_policy (GTK_SCROLLED_WINDOW (priv->scrolled_window),
                                      hpolicy, vpolicy);
     }
 } 
@@ -1620,18 +1717,15 @@ cell_view_is_sensitive (GtkCellView *cell_view)
   GList *cells, *list;
   gboolean sensitive;
   
-  cells = gtk_cell_view_get_cell_renderers (cell_view);
+  cells = gtk_cell_layout_get_cells (GTK_CELL_LAYOUT (cell_view));
 
   sensitive = FALSE;
-  list = cells;
-  while (list)
+  for (list = cells; list; list = list->next)
     {
       g_object_get (list->data, "sensitive", &sensitive, NULL);
       
       if (sensitive)
        break;
-
-      list = list->next;
     }
   g_list_free (cells);
 
@@ -1642,35 +1736,33 @@ static gboolean
 tree_column_row_is_sensitive (GtkComboBox *combo_box,
                              GtkTreeIter *iter)
 {
+  GtkComboBoxPrivate *priv = combo_box->priv;
   GList *cells, *list;
   gboolean sensitive;
 
-  if (!combo_box->priv->column)
+  if (!priv->column)
     return TRUE;
 
-  if (combo_box->priv->row_separator_func)
+  if (priv->row_separator_func)
     {
-      if ((*combo_box->priv->row_separator_func) (combo_box->priv->model, iter,
-                                                 combo_box->priv->row_separator_data))
+      if (priv->row_separator_func (priv->model, iter,
+                                    priv->row_separator_data))
        return FALSE;
     }
 
-  gtk_tree_view_column_cell_set_cell_data (combo_box->priv->column,
-                                          combo_box->priv->model,
+  gtk_tree_view_column_cell_set_cell_data (priv->column,
+                                          priv->model,
                                           iter, FALSE, FALSE);
 
-  cells = gtk_tree_view_column_get_cell_renderers (combo_box->priv->column);
+  cells = gtk_cell_layout_get_cells (GTK_CELL_LAYOUT (priv->column));
 
   sensitive = FALSE;
-  list = cells;
-  while (list)
+  for (list = cells; list; list = list->next)
     {
       g_object_get (list->data, "sensitive", &sensitive, NULL);
       
       if (sensitive)
        break;
-
-      list = list->next;
     }
   g_list_free (cells);
 
@@ -1681,12 +1773,13 @@ static void
 update_menu_sensitivity (GtkComboBox *combo_box,
                         GtkWidget   *menu)
 {
+  GtkComboBoxPrivate *priv = combo_box->priv;
   GList *children, *child;
   GtkWidget *item, *submenu, *separator;
   GtkWidget *cell_view;
   gboolean sensitive;
 
-  if (!combo_box->priv->model)
+  if (!priv->model)
     return;
 
   children = gtk_container_get_children (GTK_CONTAINER (menu));
@@ -1709,7 +1802,7 @@ update_menu_sensitivity (GtkComboBox *combo_box,
        {
          sensitive = cell_view_is_sensitive (GTK_CELL_VIEW (cell_view));
 
-         if (menu != combo_box->priv->popup_widget && child == children)
+         if (menu != priv->popup_widget && child == children)
            {
              separator = GTK_WIDGET (child->next->data);
              g_object_set (item, "visible", sensitive, NULL);
@@ -1728,38 +1821,39 @@ gtk_combo_box_menu_popup (GtkComboBox *combo_box,
                          guint        button, 
                          guint32      activate_time)
 {
+  GtkComboBoxPrivate *priv = combo_box->priv;
   GtkTreePath *path;
   gint active_item;
   GtkRequisition requisition;
   gint width;
   
-  update_menu_sensitivity (combo_box, combo_box->priv->popup_widget);
+  update_menu_sensitivity (combo_box, priv->popup_widget);
 
   active_item = -1;
-  if (gtk_tree_row_reference_valid (combo_box->priv->active_row))
+  if (gtk_tree_row_reference_valid (priv->active_row))
     {
-      path = gtk_tree_row_reference_get_path (combo_box->priv->active_row);
+      path = gtk_tree_row_reference_get_path (priv->active_row);
       active_item = gtk_tree_path_get_indices (path)[0];
       gtk_tree_path_free (path);
       
-      if (combo_box->priv->add_tearoffs)
+      if (priv->add_tearoffs)
        active_item++;
     }
 
   /* FIXME handle nested menus better */
-  gtk_menu_set_active (GTK_MENU (combo_box->priv->popup_widget), active_item);
+  gtk_menu_set_active (GTK_MENU (priv->popup_widget), active_item);
   
-  if (combo_box->priv->wrap_width == 0)
+  if (priv->wrap_width == 0)
     {
       width = GTK_WIDGET (combo_box)->allocation.width;
-      gtk_widget_set_size_request (combo_box->priv->popup_widget, -1, -1);
-      gtk_widget_size_request (combo_box->priv->popup_widget, &requisition);
+      gtk_widget_set_size_request (priv->popup_widget, -1, -1);
+      gtk_widget_size_request (priv->popup_widget, &requisition);
       
-      gtk_widget_set_size_request (combo_box->priv->popup_widget,
+      gtk_widget_set_size_request (priv->popup_widget,
                                   MAX (width, requisition.width), -1);
     }
   
-  gtk_menu_popup (GTK_MENU (combo_box->priv->popup_widget),
+  gtk_menu_popup (GTK_MENU (priv->popup_widget),
                  NULL, NULL,
                  gtk_combo_box_menu_position, combo_box,
                  button, activate_time);
@@ -1812,6 +1906,7 @@ gtk_combo_box_popup (GtkComboBox *combo_box)
 static void
 gtk_combo_box_real_popup (GtkComboBox *combo_box)
 {
+  GtkComboBoxPrivate *priv = combo_box->priv;
   gint x, y, width, height;
   GtkTreePath *path = NULL, *ppath;
   GtkWidget *toplevel;
@@ -1819,63 +1914,65 @@ gtk_combo_box_real_popup (GtkComboBox *combo_box)
   if (!GTK_WIDGET_REALIZED (combo_box))
     return;
 
-  if (GTK_WIDGET_MAPPED (combo_box->priv->popup_widget))
+  if (gtk_widget_get_mapped (priv->popup_widget))
     return;
 
-  if (GTK_IS_MENU (combo_box->priv->popup_widget))
+  if (GTK_IS_MENU (priv->popup_widget))
     {
-      gtk_combo_box_menu_popup (combo_box, 0, 0);
+      gtk_combo_box_menu_popup (combo_box, 
+                                priv->activate_button,
+                                priv->activate_time);
       return;
     }
 
   toplevel = gtk_widget_get_toplevel (GTK_WIDGET (combo_box));
   if (GTK_IS_WINDOW (toplevel))
     gtk_window_group_add_window (gtk_window_get_group (GTK_WINDOW (toplevel)), 
-                                GTK_WINDOW (combo_box->priv->popup_window));
+                                GTK_WINDOW (priv->popup_window));
 
-  gtk_widget_show_all (combo_box->priv->scrolled_window);
+  gtk_widget_show_all (priv->scrolled_window);
   gtk_combo_box_list_position (combo_box, &x, &y, &width, &height);
   
-  gtk_widget_set_size_request (combo_box->priv->popup_window, width, height);  
-  gtk_window_move (GTK_WINDOW (combo_box->priv->popup_window), x, y);
+  gtk_widget_set_size_request (priv->popup_window, width, height);  
+  gtk_window_move (GTK_WINDOW (priv->popup_window), x, y);
 
-  if (gtk_tree_row_reference_valid (combo_box->priv->active_row))
+  if (gtk_tree_row_reference_valid (priv->active_row))
     {
-      path = gtk_tree_row_reference_get_path (combo_box->priv->active_row);
+      path = gtk_tree_row_reference_get_path (priv->active_row);
       ppath = gtk_tree_path_copy (path);
       if (gtk_tree_path_up (ppath))
-       gtk_tree_view_expand_to_path (GTK_TREE_VIEW (combo_box->priv->tree_view),
+       gtk_tree_view_expand_to_path (GTK_TREE_VIEW (priv->tree_view),
                                      ppath);
       gtk_tree_path_free (ppath);
     }
-  gtk_tree_view_set_hover_expand (GTK_TREE_VIEW (combo_box->priv->tree_view), 
+  gtk_tree_view_set_hover_expand (GTK_TREE_VIEW (priv->tree_view), 
                                  TRUE);
   
   /* popup */
-  gtk_widget_show (combo_box->priv->popup_window);
+  gtk_widget_show (priv->popup_window);
 
   if (path)
     {
-      gtk_tree_view_set_cursor (GTK_TREE_VIEW (combo_box->priv->tree_view),
+      gtk_tree_view_set_cursor (GTK_TREE_VIEW (priv->tree_view),
                                path, NULL, FALSE);
       gtk_tree_path_free (path);
     }
 
-  gtk_widget_grab_focus (combo_box->priv->popup_window);
-  gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (combo_box->priv->button),
+  gtk_widget_grab_focus (priv->popup_window);
+  gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (priv->button),
                                 TRUE);
 
-  if (!GTK_WIDGET_HAS_FOCUS (combo_box->priv->tree_view))
-    gtk_widget_grab_focus (combo_box->priv->tree_view);
+  if (!gtk_widget_has_focus (priv->tree_view))
+    gtk_widget_grab_focus (priv->tree_view);
 
-  if (!popup_grab_on_window (combo_box->priv->popup_window->window,
+  if (!popup_grab_on_window (priv->popup_window->window,
                             GDK_CURRENT_TIME, TRUE))
     {
-      gtk_widget_hide (combo_box->priv->popup_window);
+      gtk_widget_hide (priv->popup_window);
       return;
     }
 
-  gtk_grab_add (combo_box->priv->popup_window);
+  gtk_grab_add (priv->popup_window);
 }
 
 static gboolean
@@ -1904,20 +2001,22 @@ gtk_combo_box_real_popdown (GtkComboBox *combo_box)
 void
 gtk_combo_box_popdown (GtkComboBox *combo_box)
 {
+  GtkComboBoxPrivate *priv = combo_box->priv;
+
   g_return_if_fail (GTK_IS_COMBO_BOX (combo_box));
 
-  if (GTK_IS_MENU (combo_box->priv->popup_widget))
+  if (GTK_IS_MENU (priv->popup_widget))
     {
-      gtk_menu_popdown (GTK_MENU (combo_box->priv->popup_widget));
+      gtk_menu_popdown (GTK_MENU (priv->popup_widget));
       return;
     }
 
   if (!GTK_WIDGET_REALIZED (GTK_WIDGET (combo_box)))
     return;
 
-  gtk_grab_remove (combo_box->priv->popup_window);
-  gtk_widget_hide_all (combo_box->priv->popup_window);
-  gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (combo_box->priv->button),
+  gtk_grab_remove (priv->popup_window);
+  gtk_widget_hide_all (priv->popup_window);
+  gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (priv->button),
                                 FALSE);
 }
 
@@ -1925,11 +2024,12 @@ static gint
 gtk_combo_box_calc_requested_width (GtkComboBox *combo_box,
                                     GtkTreePath *path)
 {
+  GtkComboBoxPrivate *priv = combo_box->priv;
   gint padding;
   GtkRequisition req;
 
-  if (combo_box->priv->cell_view)
-    gtk_widget_style_get (combo_box->priv->cell_view,
+  if (priv->cell_view)
+    gtk_widget_style_get (priv->cell_view,
                           "focus-line-width", &padding,
                           NULL);
   else
@@ -1938,8 +2038,8 @@ gtk_combo_box_calc_requested_width (GtkComboBox *combo_box,
   /* add some pixels for good measure */
   padding += BONUS_PADDING;
 
-  if (combo_box->priv->cell_view)
-    gtk_cell_view_get_size_of_row (GTK_CELL_VIEW (combo_box->priv->cell_view),
+  if (priv->cell_view)
+    gtk_cell_view_get_size_of_row (GTK_CELL_VIEW (priv->cell_view),
                                    path, &req);
   else
     req.width = 0;
@@ -1950,15 +2050,16 @@ gtk_combo_box_calc_requested_width (GtkComboBox *combo_box,
 static void
 gtk_combo_box_remeasure (GtkComboBox *combo_box)
 {
+  GtkComboBoxPrivate *priv = combo_box->priv;
   GtkTreeIter iter;
   GtkTreePath *path;
 
-  if (!combo_box->priv->model ||
-      !gtk_tree_model_get_iter_first (combo_box->priv->model, &iter))
+  if (!priv->model ||
+      !gtk_tree_model_get_iter_first (priv->model, &iter))
     return;
 
-  combo_box->priv->width = 0;
-  combo_box->priv->height = 0;
+  priv->width = 0;
+  priv->height = 0;
 
   path = gtk_tree_path_new_from_indices (0, -1);
 
@@ -1966,8 +2067,8 @@ gtk_combo_box_remeasure (GtkComboBox *combo_box)
     {
       GtkRequisition req;
 
-      if (combo_box->priv->cell_view)
-       gtk_cell_view_get_size_of_row (GTK_CELL_VIEW (combo_box->priv->cell_view), 
+      if (priv->cell_view)
+       gtk_cell_view_get_size_of_row (GTK_CELL_VIEW (priv->cell_view), 
                                        path, &req);
       else
         {
@@ -1975,12 +2076,12 @@ gtk_combo_box_remeasure (GtkComboBox *combo_box)
           req.height = 0;
         }
 
-      combo_box->priv->width = MAX (combo_box->priv->width, req.width);
-      combo_box->priv->height = MAX (combo_box->priv->height, req.height);
+      priv->width = MAX (priv->width, req.width);
+      priv->height = MAX (priv->height, req.height);
 
       gtk_tree_path_next (path);
     }
-  while (gtk_tree_model_iter_next (combo_box->priv->model, &iter));
+  while (gtk_tree_model_iter_next (priv->model, &iter));
 
   gtk_tree_path_free (path);
 }
@@ -1999,12 +2100,13 @@ gtk_combo_box_size_request (GtkWidget      *widget,
   PangoFontDescription *font_desc;
 
   GtkComboBox *combo_box = GTK_COMBO_BOX (widget);
+  GtkComboBoxPrivate *priv = combo_box->priv;
  
   /* common */
   gtk_widget_size_request (GTK_BIN (widget)->child, &bin_req);
   gtk_combo_box_remeasure (combo_box);
-  bin_req.width = MAX (bin_req.width, combo_box->priv->width);
-  bin_req.height = MAX (bin_req.height, combo_box->priv->height);
+  bin_req.width = MAX (bin_req.width, priv->width);
+  bin_req.height = MAX (bin_req.height, priv->height);
 
   gtk_widget_style_get (GTK_WIDGET (widget),
                        "focus-line-width", &focus_width,
@@ -2022,27 +2124,27 @@ gtk_combo_box_size_request (GtkWidget      *widget,
 
   arrow_size = MAX (arrow_size, font_size);
 
-  gtk_widget_set_size_request (combo_box->priv->arrow, arrow_size, arrow_size);
+  gtk_widget_set_size_request (priv->arrow, arrow_size, arrow_size);
 
-  if (!combo_box->priv->tree_view)
+  if (!priv->tree_view)
     {
       /* menu mode */
 
-      if (combo_box->priv->cell_view)
+      if (priv->cell_view)
         {
           GtkRequisition button_req, sep_req, arrow_req;
           gint border_width, xthickness, ythickness;
 
-          gtk_widget_size_request (combo_box->priv->button, &button_req);
+          gtk_widget_size_request (priv->button, &button_req);
          border_width = GTK_CONTAINER (combo_box)->border_width;
-          xthickness = combo_box->priv->button->style->xthickness;
-          ythickness = combo_box->priv->button->style->ythickness;
+          xthickness = priv->button->style->xthickness;
+          ythickness = priv->button->style->ythickness;
 
-          bin_req.width = MAX (bin_req.width, combo_box->priv->width);
-          bin_req.height = MAX (bin_req.height, combo_box->priv->height);
+          bin_req.width = MAX (bin_req.width, priv->width);
+          bin_req.height = MAX (bin_req.height, priv->height);
 
-          gtk_widget_size_request (combo_box->priv->separator, &sep_req);
-          gtk_widget_size_request (combo_box->priv->arrow, &arrow_req);
+          gtk_widget_size_request (priv->separator, &sep_req);
+          gtk_widget_size_request (priv->arrow, &arrow_req);
 
           height = MAX (sep_req.height, arrow_req.height);
           height = MAX (height, bin_req.height);
@@ -2059,7 +2161,7 @@ gtk_combo_box_size_request (GtkWidget      *widget,
         {
           GtkRequisition but_req;
 
-          gtk_widget_size_request (combo_box->priv->button, &but_req);
+          gtk_widget_size_request (priv->button, &but_req);
 
           requisition->width = bin_req.width + but_req.width;
           requisition->height = MAX (bin_req.height, but_req.height);
@@ -2075,28 +2177,28 @@ gtk_combo_box_size_request (GtkWidget      *widget,
 
       requisition->width += 2 * focus_width;
       
-      if (combo_box->priv->cell_view_frame)
+      if (priv->cell_view_frame)
         {
-         gtk_widget_size_request (combo_box->priv->cell_view_frame, &frame_req);
-         if (combo_box->priv->has_frame)
+         gtk_widget_size_request (priv->cell_view_frame, &frame_req);
+         if (priv->has_frame)
            {
              requisition->width += 2 *
-               (GTK_CONTAINER (combo_box->priv->cell_view_frame)->border_width +
-                GTK_WIDGET (combo_box->priv->cell_view_frame)->style->xthickness);
+               (GTK_CONTAINER (priv->cell_view_frame)->border_width +
+                GTK_WIDGET (priv->cell_view_frame)->style->xthickness);
              requisition->height += 2 *
-               (GTK_CONTAINER (combo_box->priv->cell_view_frame)->border_width +
-                GTK_WIDGET (combo_box->priv->cell_view_frame)->style->ythickness);
+               (GTK_CONTAINER (priv->cell_view_frame)->border_width +
+                GTK_WIDGET (priv->cell_view_frame)->style->ythickness);
            }
         }
 
       /* the button */
-      gtk_widget_size_request (combo_box->priv->button, &button_req);
+      gtk_widget_size_request (priv->button, &button_req);
 
       requisition->height = MAX (requisition->height, button_req.height);
       requisition->width += button_req.width;
     }
 
-  if (GTK_SHADOW_NONE != combo_box->priv->shadow_type)
+  if (GTK_SHADOW_NONE != priv->shadow_type)
     {
       requisition->height += 2 * widget->style->ythickness;
       requisition->width += 2 * widget->style->xthickness;
@@ -2124,6 +2226,7 @@ gtk_combo_box_size_allocate (GtkWidget     *widget,
                              GtkAllocation *allocation)
 {
   GtkComboBox *combo_box = GTK_COMBO_BOX (widget);
+  GtkComboBoxPrivate *priv = combo_box->priv;
   gint shadow_width, shadow_height;
   gint focus_width, focus_pad;
   GtkAllocation child;
@@ -2137,7 +2240,7 @@ gtk_combo_box_size_allocate (GtkWidget     *widget,
                        "focus-padding", &focus_pad,
                        NULL);
 
-  if (GTK_SHADOW_NONE != combo_box->priv->shadow_type)
+  if (GTK_SHADOW_NONE != priv->shadow_type)
     {
       shadow_width = widget->style->xthickness;
       shadow_height = widget->style->ythickness;
@@ -2148,9 +2251,9 @@ gtk_combo_box_size_allocate (GtkWidget     *widget,
       shadow_height = 0;
     }
 
-  if (!combo_box->priv->tree_view)
+  if (!priv->tree_view)
     {
-      if (combo_box->priv->cell_view)
+      if (priv->cell_view)
         {
           gint border_width, xthickness, ythickness;
           gint width;
@@ -2161,19 +2264,19 @@ gtk_combo_box_size_allocate (GtkWidget     *widget,
           allocation->width -= 2 * shadow_width;
           allocation->height -= 2 * shadow_height;
 
-          gtk_widget_size_allocate (combo_box->priv->button, allocation);
+          gtk_widget_size_allocate (priv->button, allocation);
 
           /* set some things ready */
-          border_width = GTK_CONTAINER (combo_box->priv->button)->border_width;
-          xthickness = combo_box->priv->button->style->xthickness;
-          ythickness = combo_box->priv->button->style->ythickness;
+          border_width = GTK_CONTAINER (priv->button)->border_width;
+          xthickness = priv->button->style->xthickness;
+          ythickness = priv->button->style->ythickness;
 
           child.x = allocation->x;
           child.y = allocation->y;
          width = allocation->width;
          child.height = allocation->height;
 
-         if (!combo_box->priv->is_cell_renderer)
+         if (!priv->is_cell_renderer)
            {
              child.x += border_width + xthickness + focus_width + focus_pad;
              child.y += border_width + ythickness + focus_width + focus_pad;
@@ -2183,22 +2286,22 @@ gtk_combo_box_size_allocate (GtkWidget     *widget,
 
 
           /* handle the children */
-          gtk_widget_size_request (combo_box->priv->arrow, &req);
+          gtk_widget_size_request (priv->arrow, &req);
           child.width = req.width;
           if (!is_rtl)
             child.x += width - req.width;
          child.width = MAX (1, child.width);
          child.height = MAX (1, child.height);
-          gtk_widget_size_allocate (combo_box->priv->arrow, &child);
+          gtk_widget_size_allocate (priv->arrow, &child);
           if (is_rtl)
             child.x += req.width;
-          gtk_widget_size_request (combo_box->priv->separator, &req);
+          gtk_widget_size_request (priv->separator, &req);
           child.width = req.width;
           if (!is_rtl)
             child.x -= req.width;
          child.width = MAX (1, child.width);
          child.height = MAX (1, child.height);
-          gtk_widget_size_allocate (combo_box->priv->separator, &child);
+          gtk_widget_size_allocate (priv->separator, &child);
 
           if (is_rtl)
             {
@@ -2215,6 +2318,23 @@ gtk_combo_box_size_allocate (GtkWidget     *widget,
               child.width -= child.x;
             }
 
+          if (gtk_widget_get_visible (priv->popup_widget))
+            {
+              gint width;
+              GtkRequisition requisition;
+
+              /* Warning here, without the check in the position func */
+              gtk_menu_reposition (GTK_MENU (priv->popup_widget));
+              if (priv->wrap_width == 0)
+                {
+                  width = GTK_WIDGET (combo_box)->allocation.width;
+                  gtk_widget_set_size_request (priv->popup_widget, -1, -1);
+                  gtk_widget_size_request (priv->popup_widget, &requisition);
+                  gtk_widget_set_size_request (priv->popup_widget,
+                    MAX (width, requisition.width), -1);
+               }
+            }
+
          child.width = MAX (1, child.width);
          child.height = MAX (1, child.height);
           gtk_widget_size_allocate (GTK_BIN (widget)->child, &child);
@@ -2238,6 +2358,10 @@ gtk_combo_box_size_allocate (GtkWidget     *widget,
     {
       /* list mode */
 
+      /* Combobox thickness + border-width */
+      int delta_x = shadow_width + GTK_CONTAINER (widget)->border_width;
+      int delta_y = shadow_height + GTK_CONTAINER (widget)->border_width;
+
       /* button */
       GTK_COMBO_BOX_SIZE_ALLOCATE_BUTTON
 
@@ -2246,36 +2370,53 @@ gtk_combo_box_size_allocate (GtkWidget     *widget,
         child.x = allocation->x + req.width;
       else
         child.x = allocation->x;
+
       child.y = allocation->y;
       child.width = allocation->width - req.width;
       child.height = allocation->height;
 
-      if (combo_box->priv->cell_view_frame)
+      if (priv->cell_view_frame)
         {
-         child.width = MAX (1, child.width);
-         child.height = MAX (1, child.height);
-          gtk_widget_size_allocate (combo_box->priv->cell_view_frame, &child);
+          child.x += delta_x;
+          child.y += delta_y;
+          child.width = MAX (1, child.width - delta_x * 2);
+          child.height = MAX (1, child.height - delta_y * 2);
+          gtk_widget_size_allocate (priv->cell_view_frame, &child);
 
           /* the sample */
-         if (combo_box->priv->has_frame)
-           {
-             child.x +=
-               GTK_CONTAINER (combo_box->priv->cell_view_frame)->border_width +
-               GTK_WIDGET (combo_box->priv->cell_view_frame)->style->xthickness;
-             child.y +=
-               GTK_CONTAINER (combo_box->priv->cell_view_frame)->border_width +
-               GTK_WIDGET (combo_box->priv->cell_view_frame)->style->ythickness;
-             child.width -= 2 * (
-                                 GTK_CONTAINER (combo_box->priv->cell_view_frame)->border_width +
-                                 GTK_WIDGET (combo_box->priv->cell_view_frame)->style->xthickness);
-             child.height -= 2 * (
-                                  GTK_CONTAINER (combo_box->priv->cell_view_frame)->border_width +
-                                  GTK_WIDGET (combo_box->priv->cell_view_frame)->style->ythickness);
-           }
+          if (priv->has_frame)
+            {
+              delta_x = GTK_CONTAINER (priv->cell_view_frame)->border_width +
+                GTK_WIDGET (priv->cell_view_frame)->style->xthickness;
+              delta_y = GTK_CONTAINER (priv->cell_view_frame)->border_width +
+                GTK_WIDGET (priv->cell_view_frame)->style->ythickness;
+
+              child.x += delta_x;
+              child.y += delta_y;
+              child.width -= delta_x * 2;
+              child.height -= delta_y * 2;
+            }
+        }
+      else
+        {
+          child.x += delta_x;
+          child.y += delta_y;
+          child.width -= delta_x * 2;
+          child.height -= delta_y * 2;
         }
+
+      if (gtk_widget_get_visible (priv->popup_window))
+        {
+          gint x, y, width, height;
+          gtk_combo_box_list_position (combo_box, &x, &y, &width, &height);
+          gtk_window_move (GTK_WINDOW (priv->popup_window), x, y);
+          gtk_widget_set_size_request (priv->popup_window, width, height);
+        }
+
       
       child.width = MAX (1, child.width);
       child.height = MAX (1, child.height);
+      
       gtk_widget_size_allocate (GTK_BIN (combo_box)->child, &child);
     }
 }
@@ -2285,40 +2426,42 @@ gtk_combo_box_size_allocate (GtkWidget     *widget,
 static void
 gtk_combo_box_unset_model (GtkComboBox *combo_box)
 {
-  if (combo_box->priv->model)
+  GtkComboBoxPrivate *priv = combo_box->priv;
+
+  if (priv->model)
     {
-      g_signal_handler_disconnect (combo_box->priv->model,
-                                  combo_box->priv->inserted_id);
-      g_signal_handler_disconnect (combo_box->priv->model,
-                                  combo_box->priv->deleted_id);
-      g_signal_handler_disconnect (combo_box->priv->model,
-                                  combo_box->priv->reordered_id);
-      g_signal_handler_disconnect (combo_box->priv->model,
-                                  combo_box->priv->changed_id);
+      g_signal_handler_disconnect (priv->model,
+                                  priv->inserted_id);
+      g_signal_handler_disconnect (priv->model,
+                                  priv->deleted_id);
+      g_signal_handler_disconnect (priv->model,
+                                  priv->reordered_id);
+      g_signal_handler_disconnect (priv->model,
+                                  priv->changed_id);
     }
 
   /* menu mode */
-  if (!combo_box->priv->tree_view)
+  if (!priv->tree_view)
     {
-      if (combo_box->priv->popup_widget)
-        gtk_container_foreach (GTK_CONTAINER (combo_box->priv->popup_widget),
+      if (priv->popup_widget)
+        gtk_container_foreach (GTK_CONTAINER (priv->popup_widget),
                                (GtkCallback)gtk_widget_destroy, NULL);
     }
 
-  if (combo_box->priv->model)
+  if (priv->model)
     {
-      g_object_unref (combo_box->priv->model);
-      combo_box->priv->model = NULL;
+      g_object_unref (priv->model);
+      priv->model = NULL;
     }
 
-  if (combo_box->priv->active_row)
+  if (priv->active_row)
     {
-      gtk_tree_row_reference_free (combo_box->priv->active_row);
-      combo_box->priv->active_row = NULL;
+      gtk_tree_row_reference_free (priv->active_row);
+      priv->active_row = NULL;
     }
 
-  if (combo_box->priv->cell_view)
-    gtk_cell_view_set_model (GTK_CELL_VIEW (combo_box->priv->cell_view), NULL);
+  if (priv->cell_view)
+    gtk_cell_view_set_model (GTK_CELL_VIEW (priv->cell_view), NULL);
 }
 
 static void
@@ -2328,13 +2471,14 @@ gtk_combo_box_forall (GtkContainer *container,
                       gpointer      callback_data)
 {
   GtkComboBox *combo_box = GTK_COMBO_BOX (container);
+  GtkComboBoxPrivate *priv = combo_box->priv;
 
   if (include_internals)
     {
-      if (combo_box->priv->button)
-       (* callback) (combo_box->priv->button, callback_data);
-      if (combo_box->priv->cell_view_frame)
-       (* callback) (combo_box->priv->cell_view_frame, callback_data);
+      if (priv->button)
+       (* callback) (priv->button, callback_data);
+      if (priv->cell_view_frame)
+       (* callback) (priv->cell_view_frame, callback_data);
     }
 
   if (GTK_BIN (container)->child)
@@ -2366,25 +2510,25 @@ gtk_combo_box_expose_event (GtkWidget      *widget,
                             GdkEventExpose *event)
 {
   GtkComboBox *combo_box = GTK_COMBO_BOX (widget);
+  GtkComboBoxPrivate *priv = combo_box->priv;
 
-  if (GTK_WIDGET_DRAWABLE (widget) &&
-      GTK_SHADOW_NONE != combo_box->priv->shadow_type)
+  if (gtk_widget_is_drawable (widget) &&
+      GTK_SHADOW_NONE != priv->shadow_type)
     {
       gtk_paint_shadow (widget->style, widget->window,
-                        GTK_STATE_NORMAL, combo_box->priv->shadow_type,
+                        GTK_STATE_NORMAL, priv->shadow_type,
                         NULL, widget, "combobox",
                         widget->allocation.x, widget->allocation.y,
                         widget->allocation.width, widget->allocation.height);
     }
 
   gtk_container_propagate_expose (GTK_CONTAINER (widget),
-                                 combo_box->priv->button, event);
+                                 priv->button, event);
 
-  if (combo_box->priv->tree_view &&
-      combo_box->priv->cell_view_frame)
+  if (priv->tree_view && priv->cell_view_frame)
     {
       gtk_container_propagate_expose (GTK_CONTAINER (widget),
-                                     combo_box->priv->cell_view_frame, event);
+                                     priv->cell_view_frame, event);
     }
 
   gtk_container_propagate_expose (GTK_CONTAINER (widget),
@@ -2634,9 +2778,10 @@ static void
 gtk_combo_box_sync_cells (GtkComboBox   *combo_box,
                          GtkCellLayout *cell_layout)
 {
+  GtkComboBoxPrivate *priv = combo_box->priv;
   GSList *k;
 
-  for (k = combo_box->priv->cells; k; k = k->next)
+  for (k = priv->cells; k; k = k->next)
     {
       GSList *j;
       ComboCellInfo *info = (ComboCellInfo *)k->data;
@@ -2666,62 +2811,60 @@ static void
 gtk_combo_box_menu_setup (GtkComboBox *combo_box,
                           gboolean     add_children)
 {
+  GtkComboBoxPrivate *priv = combo_box->priv;
   GtkWidget *menu;
 
-  if (combo_box->priv->cell_view)
+  if (priv->cell_view)
     {
-      combo_box->priv->button = gtk_toggle_button_new ();
-      gtk_button_set_focus_on_click (GTK_BUTTON (combo_box->priv->button),
-                                    combo_box->priv->focus_on_click);
+      priv->button = gtk_toggle_button_new ();
+      gtk_button_set_focus_on_click (GTK_BUTTON (priv->button),
+                                    priv->focus_on_click);
 
-      g_signal_connect (combo_box->priv->button, "toggled",
+      g_signal_connect (priv->button, "toggled",
                         G_CALLBACK (gtk_combo_box_button_toggled), combo_box);
-      gtk_widget_set_parent (combo_box->priv->button,
+      gtk_widget_set_parent (priv->button,
                              GTK_BIN (combo_box)->child->parent);
 
-      combo_box->priv->box = gtk_hbox_new (FALSE, 0);
-      gtk_container_add (GTK_CONTAINER (combo_box->priv->button), 
-                        combo_box->priv->box);
+      priv->box = gtk_hbox_new (FALSE, 0);
+      gtk_container_add (GTK_CONTAINER (priv->button), priv->box);
 
-      combo_box->priv->separator = gtk_vseparator_new ();
-      gtk_container_add (GTK_CONTAINER (combo_box->priv->box), 
-                        combo_box->priv->separator);
+      priv->separator = gtk_vseparator_new ();
+      gtk_container_add (GTK_CONTAINER (priv->box), priv->separator);
 
-      combo_box->priv->arrow = gtk_arrow_new (GTK_ARROW_DOWN, GTK_SHADOW_NONE);
-      gtk_container_add (GTK_CONTAINER (combo_box->priv->box), 
-                        combo_box->priv->arrow);
+      priv->arrow = gtk_arrow_new (GTK_ARROW_DOWN, GTK_SHADOW_NONE);
+      gtk_container_add (GTK_CONTAINER (priv->box), priv->arrow);
 
-      gtk_widget_show_all (combo_box->priv->button);
+      gtk_widget_show_all (priv->button);
     }
   else
     {
-      combo_box->priv->button = gtk_toggle_button_new ();
-      gtk_button_set_focus_on_click (GTK_BUTTON (combo_box->priv->button),
-                                    combo_box->priv->focus_on_click);
+      priv->button = gtk_toggle_button_new ();
+      gtk_button_set_focus_on_click (GTK_BUTTON (priv->button),
+                                    priv->focus_on_click);
 
-      g_signal_connect (combo_box->priv->button, "toggled",
+      g_signal_connect (priv->button, "toggled",
                         G_CALLBACK (gtk_combo_box_button_toggled), combo_box);
-      gtk_widget_set_parent (combo_box->priv->button,
+      gtk_widget_set_parent (priv->button,
                              GTK_BIN (combo_box)->child->parent);
 
-      combo_box->priv->arrow = gtk_arrow_new (GTK_ARROW_DOWN, GTK_SHADOW_NONE);
-      gtk_container_add (GTK_CONTAINER (combo_box->priv->button),
-                         combo_box->priv->arrow);
-      gtk_widget_show_all (combo_box->priv->button);
+      priv->arrow = gtk_arrow_new (GTK_ARROW_DOWN, GTK_SHADOW_NONE);
+      gtk_container_add (GTK_CONTAINER (priv->button), priv->arrow);
+      gtk_widget_show_all (priv->button);
     }
 
-  g_signal_connect (combo_box->priv->button, "button_press_event",
+  g_signal_connect (priv->button, "button-press-event",
                     G_CALLBACK (gtk_combo_box_menu_button_press),
                     combo_box);
-  g_signal_connect (combo_box->priv->button, "state_changed",
+  g_signal_connect (priv->button, "state-changed",
                    G_CALLBACK (gtk_combo_box_button_state_changed), 
                    combo_box);
 
   /* create our funky menu */
   menu = gtk_menu_new ();
   gtk_widget_set_name (menu, "gtk-combobox-popup-menu");
+  gtk_menu_set_reserve_toggle_size (GTK_MENU (menu), FALSE);
   
-  g_signal_connect (menu, "key_press_event",
+  g_signal_connect (menu, "key-press-event",
                    G_CALLBACK (gtk_combo_box_menu_key_press), combo_box);
   gtk_combo_box_set_popup_widget (combo_box, menu);
 
@@ -2730,33 +2873,33 @@ gtk_combo_box_menu_setup (GtkComboBox *combo_box,
     gtk_combo_box_menu_fill (combo_box);
 
   /* the column is needed in tree_column_row_is_sensitive() */
-  combo_box->priv->column = gtk_tree_view_column_new ();
-  g_object_ref_sink (combo_box->priv->column);
-  gtk_combo_box_sync_cells (combo_box, 
-                           GTK_CELL_LAYOUT (combo_box->priv->column));
+  priv->column = gtk_tree_view_column_new ();
+  g_object_ref_sink (priv->column);
+  gtk_combo_box_sync_cells (combo_box, GTK_CELL_LAYOUT (priv->column));
 
   gtk_combo_box_update_title (combo_box);
+  gtk_combo_box_update_sensitivity (combo_box);
 }
 
 static void
 gtk_combo_box_menu_fill (GtkComboBox *combo_box)
 {
+  GtkComboBoxPrivate *priv = combo_box->priv;
   GtkWidget *menu;
 
-  if (!combo_box->priv->model)
+  if (!priv->model)
     return;
 
-  menu = combo_box->priv->popup_widget;
+  menu = priv->popup_widget;
 
-  if (combo_box->priv->add_tearoffs)
+  if (priv->add_tearoffs)
     {
       GtkWidget *tearoff = gtk_tearoff_menu_item_new ();
 
       gtk_widget_show (tearoff);
       
-      if (combo_box->priv->wrap_width)
-       gtk_menu_attach (GTK_MENU (menu), tearoff, 
-                        0, combo_box->priv->wrap_width, 0, 1);
+      if (priv->wrap_width)
+       gtk_menu_attach (GTK_MENU (menu), tearoff, 0, priv->wrap_width, 0, 1);
       else
        gtk_menu_shell_append (GTK_MENU_SHELL (menu), tearoff);
     }
@@ -2795,7 +2938,8 @@ gtk_combo_box_menu_fill_level (GtkComboBox *combo_box,
                               GtkWidget   *menu,
                               GtkTreeIter *parent)
 {
-  GtkTreeModel *model = combo_box->priv->model;
+  GtkComboBoxPrivate *priv = combo_box->priv;
+  GtkTreeModel *model = priv->model;
   GtkWidget *item, *submenu, *subitem, *separator;
   GtkTreeIter iter;
   gboolean is_separator;
@@ -2810,9 +2954,9 @@ gtk_combo_box_menu_fill_level (GtkComboBox *combo_box,
     {
       gtk_tree_model_iter_nth_child (model, &iter, parent, i);
 
-      if (combo_box->priv->row_separator_func)
-       is_separator = (*combo_box->priv->row_separator_func) (combo_box->priv->model, &iter,
-                                                              combo_box->priv->row_separator_data);
+      if (priv->row_separator_func)
+       is_separator = priv->row_separator_func (priv->model, &iter,
+                                                 priv->row_separator_data);
       else
        is_separator = FALSE;
       
@@ -2832,6 +2976,7 @@ gtk_combo_box_menu_fill_level (GtkComboBox *combo_box,
          if (gtk_tree_model_iter_has_child (model, &iter))
            {
              submenu = gtk_menu_new ();
+              gtk_menu_set_reserve_toggle_size (GTK_MENU (submenu), FALSE);
              gtk_widget_show (submenu);
              gtk_menu_item_set_submenu (GTK_MENU_ITEM (item), submenu);
              
@@ -2856,7 +3001,7 @@ gtk_combo_box_menu_fill_level (GtkComboBox *combo_box,
        }
       
       gtk_menu_shell_append (GTK_MENU_SHELL (menu), item);
-      if (combo_box->priv->wrap_width && menu == combo_box->priv->popup_widget)
+      if (priv->wrap_width && menu == priv->popup_widget)
         gtk_combo_box_relayout_item (combo_box, item, &iter, last);
       gtk_widget_show (item);
       
@@ -2867,25 +3012,27 @@ gtk_combo_box_menu_fill_level (GtkComboBox *combo_box,
 static void
 gtk_combo_box_menu_destroy (GtkComboBox *combo_box)
 {
-  g_signal_handlers_disconnect_matched (combo_box->priv->button,
+  GtkComboBoxPrivate *priv = combo_box->priv;
+
+  g_signal_handlers_disconnect_matched (priv->button,
                                         G_SIGNAL_MATCH_DATA,
                                         0, 0, NULL,
                                         gtk_combo_box_menu_button_press, NULL);
-  g_signal_handlers_disconnect_matched (combo_box->priv->button,
+  g_signal_handlers_disconnect_matched (priv->button,
                                         G_SIGNAL_MATCH_DATA,
                                         0, 0, NULL,
                                         gtk_combo_box_button_state_changed, combo_box);
 
   /* unparent will remove our latest ref */
-  gtk_widget_unparent (combo_box->priv->button);
+  gtk_widget_unparent (priv->button);
   
-  combo_box->priv->box = NULL;
-  combo_box->priv->button = NULL;
-  combo_box->priv->arrow = NULL;
-  combo_box->priv->separator = NULL;
+  priv->box = NULL;
+  priv->button = NULL;
+  priv->arrow = NULL;
+  priv->separator = NULL;
 
-  g_object_unref (combo_box->priv->column);
-  combo_box->priv->column = NULL;
+  g_object_unref (priv->column);
+  priv->column = NULL;
 
   /* changing the popup window will unref the menu and the children */
 }
@@ -2929,15 +3076,16 @@ gtk_combo_box_relayout_item (GtkComboBox *combo_box,
                              GtkTreeIter *iter,
                             GtkWidget   *last)
 {
+  GtkComboBoxPrivate *priv = combo_box->priv;
   gint current_col = 0, current_row = 0;
   gint rows = 1, cols = 1;
-  GtkWidget *menu = combo_box->priv->popup_widget;
+  GtkWidget *menu = priv->popup_widget;
 
   if (!GTK_IS_MENU_SHELL (menu))
     return;
   
-  if (combo_box->priv->col_column == -1 &&
-      combo_box->priv->row_column == -1 &&
+  if (priv->col_column == -1 &&
+      priv->row_column == -1 &&
       last)
     {
       gtk_container_child_get (GTK_CONTAINER (menu), 
@@ -2945,7 +3093,7 @@ gtk_combo_box_relayout_item (GtkComboBox *combo_box,
                               "right-attach", &current_col,
                               "top-attach", &current_row,
                               NULL);
-      if (current_col + cols > combo_box->priv->wrap_width)
+      if (current_col + cols > priv->wrap_width)
        {
          current_col = 0;
          current_row++;
@@ -2953,18 +3101,18 @@ gtk_combo_box_relayout_item (GtkComboBox *combo_box,
     }
   else
     {
-      if (combo_box->priv->col_column != -1)
-       gtk_tree_model_get (combo_box->priv->model, iter,
-                           combo_box->priv->col_column, &cols,
+      if (priv->col_column != -1)
+       gtk_tree_model_get (priv->model, iter,
+                           priv->col_column, &cols,
                            -1);
-      if (combo_box->priv->row_column != -1)
-       gtk_tree_model_get (combo_box->priv->model, iter,
-                           combo_box->priv->row_column, &rows,
+      if (priv->row_column != -1)
+       gtk_tree_model_get (priv->model, iter,
+                           priv->row_column, &rows,
                            -1);
 
       while (1)
        {
-         if (current_col + cols > combo_box->priv->wrap_width)
+         if (current_col + cols > priv->wrap_width)
            {
              current_col = 0;
              current_row++;
@@ -3014,13 +3162,14 @@ gtk_combo_box_menu_button_press (GtkWidget      *widget,
                                  gpointer        user_data)
 {
   GtkComboBox *combo_box = GTK_COMBO_BOX (user_data);
+  GtkComboBoxPrivate *priv = combo_box->priv;
 
-  if (GTK_IS_MENU (combo_box->priv->popup_widget) &&
+  if (GTK_IS_MENU (priv->popup_widget) &&
       event->type == GDK_BUTTON_PRESS && event->button == 1)
     {
-      if (combo_box->priv->focus_on_click && 
-         !GTK_WIDGET_HAS_FOCUS (combo_box->priv->button))
-       gtk_widget_grab_focus (combo_box->priv->button);
+      if (priv->focus_on_click && 
+         !gtk_widget_has_focus (priv->button))
+       gtk_widget_grab_focus (priv->button);
 
       gtk_combo_box_menu_popup (combo_box, event->button, event->time);
 
@@ -3053,7 +3202,43 @@ gtk_combo_box_menu_item_activate (GtkWidget *item,
 
   gtk_tree_path_free (path);
 
-  combo_box->priv->editing_canceled = FALSE;
+  g_object_set (combo_box,
+                "editing-canceled", FALSE,
+                NULL);
+}
+
+static void
+gtk_combo_box_update_sensitivity (GtkComboBox *combo_box)
+{
+  GtkTreeIter iter;
+  gboolean sensitive = TRUE; /* fool code checkers */
+
+  if (!combo_box->priv->button)
+    return;
+
+  switch (combo_box->priv->button_sensitivity)
+    {
+      case GTK_SENSITIVITY_ON:
+        sensitive = TRUE;
+        break;
+      case GTK_SENSITIVITY_OFF:
+        sensitive = FALSE;
+        break;
+      case GTK_SENSITIVITY_AUTO:
+        sensitive = combo_box->priv->model &&
+                    gtk_tree_model_get_iter_first (combo_box->priv->model, &iter);
+        break;
+      default:
+        g_assert_not_reached ();
+        break;
+    }
+
+  gtk_widget_set_sensitive (combo_box->priv->button, sensitive);
+
+  /* In list-mode, we also need to update sensitivity of the event box */
+  if (GTK_IS_TREE_VIEW (combo_box->priv->tree_view)
+      && combo_box->priv->cell_view)
+    gtk_widget_set_sensitive (combo_box->priv->box, sensitive);
 }
 
 static void
@@ -3068,6 +3253,8 @@ gtk_combo_box_model_row_inserted (GtkTreeModel     *model,
     gtk_combo_box_list_popup_resize (combo_box);
   else
     gtk_combo_box_menu_row_inserted (model, path, iter, user_data);
+
+  gtk_combo_box_update_sensitivity (combo_box);
 }
 
 static void
@@ -3076,17 +3263,21 @@ gtk_combo_box_model_row_deleted (GtkTreeModel     *model,
                                 gpointer          user_data)
 {
   GtkComboBox *combo_box = GTK_COMBO_BOX (user_data);
+  GtkComboBoxPrivate *priv = combo_box->priv;
 
-  if (combo_box->priv->cell_view)
+  if (!gtk_tree_row_reference_valid (priv->active_row))
     {
-      if (!gtk_tree_row_reference_valid (combo_box->priv->active_row))
-       gtk_cell_view_set_displayed_row (GTK_CELL_VIEW (combo_box->priv->cell_view), NULL);
+      if (priv->cell_view)
+       gtk_cell_view_set_displayed_row (GTK_CELL_VIEW (priv->cell_view), NULL);
+      g_signal_emit (combo_box, combo_box_signals[CHANGED], 0);
     }
   
-  if (combo_box->priv->tree_view)
+  if (priv->tree_view)
     gtk_combo_box_list_popup_resize (combo_box);
   else
     gtk_combo_box_menu_row_deleted (model, path, user_data);  
+
+  gtk_combo_box_update_sensitivity (combo_box);
 }
 
 static void
@@ -3111,19 +3302,20 @@ gtk_combo_box_model_row_changed (GtkTreeModel     *model,
                                 gpointer          user_data)
 {
   GtkComboBox *combo_box = GTK_COMBO_BOX (user_data);
+  GtkComboBoxPrivate *priv = combo_box->priv;
   GtkTreePath *active_path;
 
   /* FIXME this belongs to GtkCellView */
-  if (gtk_tree_row_reference_valid (combo_box->priv->active_row))
+  if (gtk_tree_row_reference_valid (priv->active_row))
     {
-      active_path = gtk_tree_row_reference_get_path (combo_box->priv->active_row);
+      active_path = gtk_tree_row_reference_get_path (priv->active_row);
       if (gtk_tree_path_compare (path, active_path) == 0 &&
-         combo_box->priv->cell_view)
-       gtk_widget_queue_resize (GTK_WIDGET (combo_box->priv->cell_view));
+         priv->cell_view)
+       gtk_widget_queue_resize (GTK_WIDGET (priv->cell_view));
       gtk_tree_path_free (active_path);
     }
       
-  if (combo_box->priv->tree_view)
+  if (priv->tree_view)
     gtk_combo_box_list_row_changed (model, path, iter, user_data);
   else
     gtk_combo_box_menu_row_changed (model, path, iter, user_data);
@@ -3132,21 +3324,19 @@ gtk_combo_box_model_row_changed (GtkTreeModel     *model,
 static gboolean
 list_popup_resize_idle (gpointer user_data)
 {
-  GtkComboBox *combo_box;
+  GtkComboBox *combo_box = GTK_COMBO_BOX (user_data);
+  GtkComboBoxPrivate *priv = combo_box->priv;
   gint x, y, width, height;
 
-  combo_box = GTK_COMBO_BOX (user_data);
-
-  if (combo_box->priv->tree_view &&
-      GTK_WIDGET_MAPPED (combo_box->priv->popup_window))
+  if (priv->tree_view && gtk_widget_get_mapped (priv->popup_window))
     {
       gtk_combo_box_list_position (combo_box, &x, &y, &width, &height);
   
-      gtk_widget_set_size_request (combo_box->priv->popup_window, width, height);
-      gtk_window_move (GTK_WINDOW (combo_box->priv->popup_window), x, y);
+      gtk_widget_set_size_request (priv->popup_window, width, height);
+      gtk_window_move (GTK_WINDOW (priv->popup_window), x, y);
     }
 
-  combo_box->priv->resize_idle_id = 0;
+  priv->resize_idle_id = 0;
 
   return FALSE;
 }
@@ -3154,8 +3344,10 @@ list_popup_resize_idle (gpointer user_data)
 static void
 gtk_combo_box_list_popup_resize (GtkComboBox *combo_box)
 {
-  if (!combo_box->priv->resize_idle_id)
-    combo_box->priv->resize_idle_id = 
+  GtkComboBoxPrivate *priv = combo_box->priv;
+
+  if (!priv->resize_idle_id)
+    priv->resize_idle_id = 
       gdk_threads_add_idle (list_popup_resize_idle, combo_box);
 }
 
@@ -3277,15 +3469,16 @@ gtk_combo_box_menu_row_inserted (GtkTreeModel *model,
                                  GtkTreeIter  *iter,
                                  gpointer      user_data)
 {
+  GtkComboBox *combo_box = GTK_COMBO_BOX (user_data);
+  GtkComboBoxPrivate *priv = combo_box->priv;
   GtkWidget *parent;
   GtkWidget *item, *menu, *separator;
-  GtkComboBox *combo_box = GTK_COMBO_BOX (user_data);
   GtkTreePath *ppath;
   GtkTreeIter piter;
   gint depth, pos;
   gboolean is_separator;
 
-  if (!combo_box->priv->popup_widget)
+  if (!priv->popup_widget)
     return;
 
   depth = gtk_tree_path_get_depth (path);
@@ -3294,13 +3487,14 @@ gtk_combo_box_menu_row_inserted (GtkTreeModel *model,
     {
       ppath = gtk_tree_path_copy (path);
       gtk_tree_path_up (ppath);
-      parent = find_menu_by_path (combo_box->priv->popup_widget, ppath, FALSE);
+      parent = find_menu_by_path (priv->popup_widget, ppath, FALSE);
       gtk_tree_path_free (ppath);
 
       menu = gtk_menu_item_get_submenu (GTK_MENU_ITEM (parent));
       if (!menu)
        {
          menu = gtk_menu_new ();
+          gtk_menu_set_reserve_toggle_size (GTK_MENU (menu), FALSE);
          gtk_widget_show (menu);
          gtk_menu_item_set_submenu (GTK_MENU_ITEM (parent), menu);
          
@@ -3325,14 +3519,14 @@ gtk_combo_box_menu_row_inserted (GtkTreeModel *model,
     }
   else
     {
-      menu = combo_box->priv->popup_widget;
-      if (combo_box->priv->add_tearoffs)
+      menu = priv->popup_widget;
+      if (priv->add_tearoffs)
        pos += 1;
     }
   
-  if (combo_box->priv->row_separator_func)
-    is_separator = (*combo_box->priv->row_separator_func) (model, iter,
-                                                          combo_box->priv->row_separator_data);
+  if (priv->row_separator_func)
+    is_separator = priv->row_separator_func (model, iter,
+                                             priv->row_separator_data);
   else
     is_separator = FALSE;
 
@@ -3363,13 +3557,14 @@ gtk_combo_box_menu_row_deleted (GtkTreeModel *model,
                                 gpointer      user_data)
 {
   GtkComboBox *combo_box = GTK_COMBO_BOX (user_data);
+  GtkComboBoxPrivate *priv = combo_box->priv;
   GtkWidget *menu;
   GtkWidget *item;
 
-  if (!combo_box->priv->popup_widget)
+  if (!priv->popup_widget)
     return;
 
-  item = find_menu_by_path (combo_box->priv->popup_widget, path, FALSE);
+  item = find_menu_by_path (priv->popup_widget, path, FALSE);
   menu = gtk_widget_get_parent (item);
   gtk_container_remove (GTK_CONTAINER (menu), item);
 
@@ -3385,7 +3580,7 @@ gtk_combo_box_menu_row_deleted (GtkTreeModel *model,
 
       if (!gtk_tree_model_iter_has_child (model, &iter))
        {
-         parent = find_menu_by_path (combo_box->priv->popup_widget, 
+         parent = find_menu_by_path (priv->popup_widget, 
                                      parent_path, FALSE);
          gtk_menu_item_set_submenu (GTK_MENU_ITEM (parent), NULL);
        }
@@ -3411,18 +3606,19 @@ gtk_combo_box_menu_row_changed (GtkTreeModel *model,
                                 gpointer      user_data)
 {
   GtkComboBox *combo_box = GTK_COMBO_BOX (user_data);
+  GtkComboBoxPrivate *priv = combo_box->priv;
   GtkWidget *item;
   gint width;
   gboolean is_separator;
 
-  if (!combo_box->priv->popup_widget)
+  if (!priv->popup_widget)
     return;
 
-  item = find_menu_by_path (combo_box->priv->popup_widget, path, FALSE);
+  item = find_menu_by_path (priv->popup_widget, path, FALSE);
 
-  if (combo_box->priv->row_separator_func)
-    is_separator = (*combo_box->priv->row_separator_func) (model, iter,
-                                                          combo_box->priv->row_separator_data);
+  if (priv->row_separator_func)
+    is_separator = priv->row_separator_func (model, iter,
+                                             priv->row_separator_data);
   else
     is_separator = FALSE;
 
@@ -3432,8 +3628,7 @@ gtk_combo_box_menu_row_changed (GtkTreeModel *model,
       gtk_combo_box_menu_row_inserted (model, path, iter, combo_box);
     }
 
-  if (combo_box->priv->wrap_width
-      && item->parent == combo_box->priv->popup_widget)
+  if (priv->wrap_width && item->parent == priv->popup_widget)
     {
       GtkWidget *pitem = NULL;
       GtkTreePath *prev;
@@ -3441,13 +3636,13 @@ gtk_combo_box_menu_row_changed (GtkTreeModel *model,
       prev = gtk_tree_path_copy (path);
 
       if (gtk_tree_path_prev (prev))
-        pitem = find_menu_by_path (combo_box->priv->popup_widget, prev, FALSE);
+        pitem = find_menu_by_path (priv->popup_widget, prev, FALSE);
 
       gtk_tree_path_free (prev);
 
       /* unattach item so gtk_combo_box_relayout_item() won't spuriously
          move it */
-      gtk_container_child_set (GTK_CONTAINER (combo_box->priv->popup_widget),
+      gtk_container_child_set (GTK_CONTAINER (priv->popup_widget),
                                item, 
                               "left-attach", -1, 
                               "right-attach", -1,
@@ -3460,14 +3655,14 @@ gtk_combo_box_menu_row_changed (GtkTreeModel *model,
 
   width = gtk_combo_box_calc_requested_width (combo_box, path);
 
-  if (width > combo_box->priv->width)
+  if (width > priv->width)
     {
-      if (combo_box->priv->cell_view)
+      if (priv->cell_view)
        {
-         gtk_widget_set_size_request (combo_box->priv->cell_view, width, -1);
-         gtk_widget_queue_resize (combo_box->priv->cell_view);
+         gtk_widget_set_size_request (priv->cell_view, width, -1);
+         gtk_widget_queue_resize (priv->cell_view);
        }
-      combo_box->priv->width = width;
+      priv->width = width;
     }
 }
 
@@ -3478,35 +3673,35 @@ gtk_combo_box_menu_row_changed (GtkTreeModel *model,
 static void
 gtk_combo_box_list_setup (GtkComboBox *combo_box)
 {
+  GtkComboBoxPrivate *priv = combo_box->priv;
   GtkTreeSelection *sel;
 
-  combo_box->priv->button = gtk_toggle_button_new ();
-  gtk_widget_set_parent (combo_box->priv->button,
+  priv->button = gtk_toggle_button_new ();
+  gtk_widget_set_parent (priv->button,
                          GTK_BIN (combo_box)->child->parent);
-  g_signal_connect (combo_box->priv->button, "button_press_event",
+  g_signal_connect (priv->button, "button-press-event",
                     G_CALLBACK (gtk_combo_box_list_button_pressed), combo_box);
-  g_signal_connect (combo_box->priv->button, "toggled",
+  g_signal_connect (priv->button, "toggled",
                     G_CALLBACK (gtk_combo_box_button_toggled), combo_box);
 
-  combo_box->priv->arrow = gtk_arrow_new (GTK_ARROW_DOWN, GTK_SHADOW_NONE);
-  gtk_container_add (GTK_CONTAINER (combo_box->priv->button),
-                     combo_box->priv->arrow);
-  combo_box->priv->separator = NULL;
-  gtk_widget_show_all (combo_box->priv->button);
+  priv->arrow = gtk_arrow_new (GTK_ARROW_DOWN, GTK_SHADOW_NONE);
+  gtk_container_add (GTK_CONTAINER (priv->button), priv->arrow);
+  priv->separator = NULL;
+  gtk_widget_show_all (priv->button);
 
-  if (combo_box->priv->cell_view)
+  if (priv->cell_view)
     {
-      gtk_cell_view_set_background_color (GTK_CELL_VIEW (combo_box->priv->cell_view), 
+      gtk_cell_view_set_background_color (GTK_CELL_VIEW (priv->cell_view), 
                                          &GTK_WIDGET (combo_box)->style->base[GTK_WIDGET_STATE (combo_box)]);
 
-      combo_box->priv->box = gtk_event_box_new ();
-      gtk_event_box_set_visible_window (GTK_EVENT_BOX (combo_box->priv->box), 
+      priv->box = gtk_event_box_new ();
+      gtk_event_box_set_visible_window (GTK_EVENT_BOX (priv->box), 
                                        FALSE);
 
-      if (combo_box->priv->has_frame)
+      if (priv->has_frame)
        {
-         combo_box->priv->cell_view_frame = gtk_frame_new (NULL);
-         gtk_frame_set_shadow_type (GTK_FRAME (combo_box->priv->cell_view_frame),
+         priv->cell_view_frame = gtk_frame_new (NULL);
+         gtk_frame_set_shadow_type (GTK_FRAME (priv->cell_view_frame),
                                     GTK_SHADOW_IN);
        }
       else 
@@ -3516,116 +3711,117 @@ gtk_combo_box_list_setup (GtkComboBox *combo_box)
                                            FALSE);
        }
       
-      gtk_widget_set_parent (combo_box->priv->cell_view_frame,
+      gtk_widget_set_parent (priv->cell_view_frame,
                             GTK_BIN (combo_box)->child->parent);
-      gtk_container_add (GTK_CONTAINER (combo_box->priv->cell_view_frame),
-                        combo_box->priv->box);
-      gtk_widget_show_all (combo_box->priv->cell_view_frame);
+      gtk_container_add (GTK_CONTAINER (priv->cell_view_frame), priv->box);
+      gtk_widget_show_all (priv->cell_view_frame);
 
-      g_signal_connect (combo_box->priv->box, "button_press_event",
+      g_signal_connect (priv->box, "button-press-event",
                        G_CALLBACK (gtk_combo_box_list_button_pressed), 
                        combo_box);
     }
 
-  combo_box->priv->tree_view = gtk_tree_view_new ();
-  sel = gtk_tree_view_get_selection (GTK_TREE_VIEW (combo_box->priv->tree_view));
+  priv->tree_view = gtk_tree_view_new ();
+  sel = gtk_tree_view_get_selection (GTK_TREE_VIEW (priv->tree_view));
   gtk_tree_selection_set_mode (sel, GTK_SELECTION_BROWSE);
   gtk_tree_selection_set_select_function (sel,
                                          gtk_combo_box_list_select_func,
                                          NULL, NULL);
-  gtk_tree_view_set_headers_visible (GTK_TREE_VIEW (combo_box->priv->tree_view),
+  gtk_tree_view_set_headers_visible (GTK_TREE_VIEW (priv->tree_view),
                                      FALSE);
-  gtk_tree_view_set_hover_selection (GTK_TREE_VIEW (combo_box->priv->tree_view),
+  gtk_tree_view_set_hover_selection (GTK_TREE_VIEW (priv->tree_view),
                                     TRUE);
-  if (combo_box->priv->row_separator_func)
-    gtk_tree_view_set_row_separator_func (GTK_TREE_VIEW (combo_box->priv->tree_view), 
-                                         combo_box->priv->row_separator_func, 
-                                         combo_box->priv->row_separator_data, 
+  if (priv->row_separator_func)
+    gtk_tree_view_set_row_separator_func (GTK_TREE_VIEW (priv->tree_view), 
+                                         priv->row_separator_func, 
+                                         priv->row_separator_data, 
                                          NULL);
-  if (combo_box->priv->model)
-    gtk_tree_view_set_model (GTK_TREE_VIEW (combo_box->priv->tree_view),
-                            combo_box->priv->model);
+  if (priv->model)
+    gtk_tree_view_set_model (GTK_TREE_VIEW (priv->tree_view), priv->model);
     
-  combo_box->priv->column = gtk_tree_view_column_new ();
-  gtk_tree_view_append_column (GTK_TREE_VIEW (combo_box->priv->tree_view),
-                               combo_box->priv->column);
+  priv->column = gtk_tree_view_column_new ();
+  gtk_tree_view_append_column (GTK_TREE_VIEW (priv->tree_view), priv->column);
 
   /* sync up */
   gtk_combo_box_sync_cells (combo_box, 
-                           GTK_CELL_LAYOUT (combo_box->priv->column));
+                           GTK_CELL_LAYOUT (priv->column));
 
-  if (gtk_tree_row_reference_valid (combo_box->priv->active_row))
+  if (gtk_tree_row_reference_valid (priv->active_row))
     {
       GtkTreePath *path;
 
-      path = gtk_tree_row_reference_get_path (combo_box->priv->active_row);
-      gtk_tree_view_set_cursor (GTK_TREE_VIEW (combo_box->priv->tree_view),
+      path = gtk_tree_row_reference_get_path (priv->active_row);
+      gtk_tree_view_set_cursor (GTK_TREE_VIEW (priv->tree_view),
                                path, NULL, FALSE);
       gtk_tree_path_free (path);
     }
 
   /* set sample/popup widgets */
-  gtk_combo_box_set_popup_widget (combo_box, combo_box->priv->tree_view);
+  gtk_combo_box_set_popup_widget (combo_box, priv->tree_view);
 
-  g_signal_connect (combo_box->priv->tree_view, "key_press_event",
+  g_signal_connect (priv->tree_view, "key-press-event",
                     G_CALLBACK (gtk_combo_box_list_key_press),
                     combo_box);
-  g_signal_connect (combo_box->priv->tree_view, "enter_notify_event",
+  g_signal_connect (priv->tree_view, "enter-notify-event",
                     G_CALLBACK (gtk_combo_box_list_enter_notify),
                     combo_box);
-  g_signal_connect (combo_box->priv->tree_view, "row_expanded",
+  g_signal_connect (priv->tree_view, "row-expanded",
                    G_CALLBACK (gtk_combo_box_model_row_expanded),
                    combo_box);
-  g_signal_connect (combo_box->priv->tree_view, "row_collapsed",
+  g_signal_connect (priv->tree_view, "row-collapsed",
                    G_CALLBACK (gtk_combo_box_model_row_expanded),
                    combo_box);
-  g_signal_connect (combo_box->priv->popup_window, "button_press_event",
+  g_signal_connect (priv->popup_window, "button-press-event",
                     G_CALLBACK (gtk_combo_box_list_button_pressed),
                     combo_box);
-  g_signal_connect (combo_box->priv->popup_window, "button_release_event",
+  g_signal_connect (priv->popup_window, "button-release-event",
                     G_CALLBACK (gtk_combo_box_list_button_released),
                     combo_box);
 
-  gtk_widget_show (combo_box->priv->tree_view);
+  gtk_widget_show (priv->tree_view);
+
+  gtk_combo_box_update_sensitivity (combo_box);
 }
 
 static void
 gtk_combo_box_list_destroy (GtkComboBox *combo_box)
 {
+  GtkComboBoxPrivate *priv = combo_box->priv;
+
   /* disconnect signals */
-  g_signal_handlers_disconnect_matched (combo_box->priv->tree_view,
+  g_signal_handlers_disconnect_matched (priv->tree_view,
                                         G_SIGNAL_MATCH_DATA,
                                         0, 0, NULL, NULL, combo_box);
-  g_signal_handlers_disconnect_matched (combo_box->priv->button,
+  g_signal_handlers_disconnect_matched (priv->button,
                                         G_SIGNAL_MATCH_DATA,
                                         0, 0, NULL,
                                         gtk_combo_box_list_button_pressed,
                                         NULL);
-  g_signal_handlers_disconnect_matched (combo_box->priv->popup_window,
+  g_signal_handlers_disconnect_matched (priv->popup_window,
                                         G_SIGNAL_MATCH_DATA,
                                         0, 0, NULL,
                                         gtk_combo_box_list_button_pressed,
                                         NULL);
-  g_signal_handlers_disconnect_matched (combo_box->priv->popup_window,
+  g_signal_handlers_disconnect_matched (priv->popup_window,
                                         G_SIGNAL_MATCH_DATA,
                                         0, 0, NULL,
                                         gtk_combo_box_list_button_released,
                                         NULL);
 
-  g_signal_handlers_disconnect_matched (combo_box->priv->popup_window,
+  g_signal_handlers_disconnect_matched (priv->popup_window,
                                        G_SIGNAL_MATCH_DATA,
                                        0, 0, NULL, 
                                        gtk_combo_box_child_show,
                                        NULL);
 
-  g_signal_handlers_disconnect_matched (combo_box->priv->popup_window,
+  g_signal_handlers_disconnect_matched (priv->popup_window,
                                        G_SIGNAL_MATCH_DATA,
                                        0, 0, NULL, 
                                        gtk_combo_box_child_hide,
                                        NULL);
   
-  if (combo_box->priv->box)
-    g_signal_handlers_disconnect_matched (combo_box->priv->box,
+  if (priv->box)
+    g_signal_handlers_disconnect_matched (priv->box,
                                          G_SIGNAL_MATCH_DATA,
                                          0, 0, NULL,
                                          gtk_combo_box_list_button_pressed,
@@ -3634,43 +3830,43 @@ gtk_combo_box_list_destroy (GtkComboBox *combo_box)
   /* destroy things (unparent will kill the latest ref from us)
    * last unref on button will destroy the arrow
    */
-  gtk_widget_unparent (combo_box->priv->button);
-  combo_box->priv->button = NULL;
-  combo_box->priv->arrow = NULL;
+  gtk_widget_unparent (priv->button);
+  priv->button = NULL;
+  priv->arrow = NULL;
 
-  if (combo_box->priv->cell_view)
+  if (priv->cell_view)
     {
-      g_object_set (combo_box->priv->cell_view,
+      g_object_set (priv->cell_view,
                     "background-set", FALSE,
                     NULL);
     }
 
-  if (combo_box->priv->cell_view_frame)
+  if (priv->cell_view_frame)
     {
-      gtk_widget_unparent (combo_box->priv->cell_view_frame);
-      combo_box->priv->cell_view_frame = NULL;
-      combo_box->priv->box = NULL;
+      gtk_widget_unparent (priv->cell_view_frame);
+      priv->cell_view_frame = NULL;
+      priv->box = NULL;
     }
 
-  if (combo_box->priv->scroll_timer)
+  if (priv->scroll_timer)
     {
-      g_source_remove (combo_box->priv->scroll_timer);
-      combo_box->priv->scroll_timer = 0;
+      g_source_remove (priv->scroll_timer);
+      priv->scroll_timer = 0;
     }
 
-  if (combo_box->priv->resize_idle_id)
+  if (priv->resize_idle_id)
     {
-      g_source_remove (combo_box->priv->resize_idle_id);
-      combo_box->priv->resize_idle_id = 0;
+      g_source_remove (priv->resize_idle_id);
+      priv->resize_idle_id = 0;
     }
 
-  gtk_widget_destroy (combo_box->priv->tree_view);
+  gtk_widget_destroy (priv->tree_view);
 
-  combo_box->priv->tree_view = NULL;
-  if (combo_box->priv->popup_widget)
+  priv->tree_view = NULL;
+  if (priv->popup_widget)
     {
-      g_object_unref (combo_box->priv->popup_widget);
-      combo_box->priv->popup_widget = NULL;
+      g_object_unref (priv->popup_widget);
+      priv->popup_widget = NULL;
     }
 }
 
@@ -3682,32 +3878,32 @@ gtk_combo_box_list_button_pressed (GtkWidget      *widget,
                                    gpointer        data)
 {
   GtkComboBox *combo_box = GTK_COMBO_BOX (data);
+  GtkComboBoxPrivate *priv = combo_box->priv;
 
   GtkWidget *ewidget = gtk_get_event_widget ((GdkEvent *)event);
 
-  if (ewidget == combo_box->priv->popup_window)
+  if (ewidget == priv->popup_window)
     return TRUE;
 
-  if ((ewidget != combo_box->priv->button && ewidget != combo_box->priv->box) ||
-      gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (combo_box->priv->button)))
+  if ((ewidget != priv->button && ewidget != priv->box) ||
+      gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (priv->button)))
     return FALSE;
 
-  if (combo_box->priv->focus_on_click && 
-      !GTK_WIDGET_HAS_FOCUS (combo_box->priv->button))
-    gtk_widget_grab_focus (combo_box->priv->button);
+  if (priv->focus_on_click && 
+      !gtk_widget_has_focus (priv->button))
+    gtk_widget_grab_focus (priv->button);
 
   gtk_combo_box_popup (combo_box);
 
-  gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (combo_box->priv->button),
-                                TRUE);
+  gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (priv->button), TRUE);
 
-  combo_box->priv->auto_scroll = FALSE;
-  if (combo_box->priv->scroll_timer == 0)
-    combo_box->priv->scroll_timer = gdk_threads_add_timeout (SCROLL_TIME, 
-                                                  (GSourceFunc) gtk_combo_box_list_scroll_timeout, 
+  priv->auto_scroll = FALSE;
+  if (priv->scroll_timer == 0)
+    priv->scroll_timer = gdk_threads_add_timeout (SCROLL_TIME, 
+                                                 (GSourceFunc) gtk_combo_box_list_scroll_timeout, 
                                                   combo_box);
 
-  combo_box->priv->popup_in_progress = TRUE;
+  priv->popup_in_progress = TRUE;
 
   return TRUE;
 }
@@ -3722,39 +3918,39 @@ gtk_combo_box_list_button_released (GtkWidget      *widget,
   GtkTreeIter iter;
 
   GtkComboBox *combo_box = GTK_COMBO_BOX (data);
+  GtkComboBoxPrivate *priv = combo_box->priv;
 
   gboolean popup_in_progress = FALSE;
 
   GtkWidget *ewidget = gtk_get_event_widget ((GdkEvent *)event);
 
-  if (combo_box->priv->popup_in_progress)
+  if (priv->popup_in_progress)
     {
       popup_in_progress = TRUE;
-      combo_box->priv->popup_in_progress = FALSE;
+      priv->popup_in_progress = FALSE;
     }
 
-  gtk_tree_view_set_hover_expand (GTK_TREE_VIEW (combo_box->priv->tree_view), 
+  gtk_tree_view_set_hover_expand (GTK_TREE_VIEW (priv->tree_view), 
                                  FALSE);
-  if (combo_box->priv->scroll_timer)
+  if (priv->scroll_timer)
     {
-      g_source_remove (combo_box->priv->scroll_timer);
-      combo_box->priv->scroll_timer = 0;
+      g_source_remove (priv->scroll_timer);
+      priv->scroll_timer = 0;
     }
 
-  if (ewidget != combo_box->priv->tree_view)
+  if (ewidget != priv->tree_view)
     {
-      if ((ewidget == combo_box->priv->button || 
-          ewidget == combo_box->priv->box) &&
+      if ((ewidget == priv->button || 
+          ewidget == priv->box) &&
           !popup_in_progress &&
-          gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (combo_box->priv->button)))
+          gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (priv->button)))
         {
           gtk_combo_box_popdown (combo_box);
           return TRUE;
         }
 
       /* released outside treeview */
-      if (ewidget != combo_box->priv->button && 
-         ewidget != combo_box->priv->box)
+      if (ewidget != priv->button && ewidget != priv->box)
         {
           gtk_combo_box_popdown (combo_box);
 
@@ -3765,7 +3961,7 @@ gtk_combo_box_list_button_released (GtkWidget      *widget,
     }
 
   /* select something cool */
-  ret = gtk_tree_view_get_path_at_pos (GTK_TREE_VIEW (combo_box->priv->tree_view),
+  ret = gtk_tree_view_get_path_at_pos (GTK_TREE_VIEW (priv->tree_view),
                                        event->x, event->y,
                                        &path,
                                        NULL, NULL, NULL);
@@ -3773,7 +3969,7 @@ gtk_combo_box_list_button_released (GtkWidget      *widget,
   if (!ret)
     return TRUE; /* clicked outside window? */
 
-  gtk_tree_model_get_iter (combo_box->priv->model, &iter, path);
+  gtk_tree_model_get_iter (priv->model, &iter, path);
   gtk_tree_path_free (path);
 
   gtk_combo_box_popdown (combo_box);
@@ -3810,7 +4006,7 @@ gtk_combo_box_list_key_press (GtkWidget   *widget,
   GtkComboBox *combo_box = GTK_COMBO_BOX (data);
   GtkTreeIter iter;
 
-  if (event->keyval == GDK_Return || event->keyval == GDK_KP_Enter ||
+  if (event->keyval == GDK_Return || event->keyval == GDK_ISO_Enter || event->keyval == GDK_KP_Enter ||
       event->keyval == GDK_space || event->keyval == GDK_KP_Space) 
   {
     GtkTreeModel *model = NULL;
@@ -3888,12 +4084,12 @@ gtk_combo_box_list_auto_scroll (GtkComboBox *combo_box,
 static gboolean
 gtk_combo_box_list_scroll_timeout (GtkComboBox *combo_box)
 {
+  GtkComboBoxPrivate *priv = combo_box->priv;
   gint x, y;
 
-  if (combo_box->priv->auto_scroll)
+  if (priv->auto_scroll)
     {
-      gdk_window_get_pointer (combo_box->priv->tree_view->window, 
-                             &x, &y, NULL);
+      gdk_window_get_pointer (priv->tree_view->window, &x, &y, NULL);
       gtk_combo_box_list_auto_scroll (combo_box, x, y);
     }
 
@@ -3936,7 +4132,7 @@ gtk_combo_box_list_select_func (GtkTreeSelection *selection,
       gtk_tree_view_column_cell_set_cell_data (column, model, &iter,
                                               FALSE, FALSE);
 
-      cell = cells = gtk_tree_view_column_get_cell_renderers (column);
+      cell = cells = gtk_cell_layout_get_cells (GTK_CELL_LAYOUT (column));
       while (cell)
         {
          g_object_get (cell->data,
@@ -3964,18 +4160,19 @@ gtk_combo_box_list_row_changed (GtkTreeModel *model,
                                 gpointer      data)
 {
   GtkComboBox *combo_box = GTK_COMBO_BOX (data);
+  GtkComboBoxPrivate *priv = combo_box->priv;
   gint width;
 
   width = gtk_combo_box_calc_requested_width (combo_box, path);
 
-  if (width > combo_box->priv->width)
+  if (width > priv->width)
     {
-      if (combo_box->priv->cell_view) 
+      if (priv->cell_view) 
        {
-         gtk_widget_set_size_request (combo_box->priv->cell_view, width, -1);
-         gtk_widget_queue_resize (combo_box->priv->cell_view);
+         gtk_widget_set_size_request (priv->cell_view, width, -1);
+         gtk_widget_queue_resize (priv->cell_view);
        }
-      combo_box->priv->width = width;
+      priv->width = width;
     }
 }
 
@@ -4011,32 +4208,30 @@ gtk_combo_box_cell_layout_pack_start (GtkCellLayout   *layout,
                                       GtkCellRenderer *cell,
                                       gboolean         expand)
 {
+  GtkComboBox *combo_box = GTK_COMBO_BOX (layout);
   ComboCellInfo *info;
-  GtkComboBox *combo_box;
-
-  g_return_if_fail (GTK_IS_COMBO_BOX (layout));
-  g_return_if_fail (GTK_IS_CELL_RENDERER (cell));
+  GtkComboBoxPrivate *priv;
 
-  combo_box = GTK_COMBO_BOX (layout);
+  priv = combo_box->priv;
 
   g_object_ref_sink (cell);
 
-  info = g_new0 (ComboCellInfo, 1);
+  info = g_slice_new0 (ComboCellInfo);
   info->cell = cell;
   info->expand = expand;
   info->pack = GTK_PACK_START;
 
-  combo_box->priv->cells = g_slist_append (combo_box->priv->cells, info);
+  priv->cells = g_slist_append (priv->cells, info);
 
-  if (combo_box->priv->cell_view)
-    gtk_cell_layout_pack_start (GTK_CELL_LAYOUT (combo_box->priv->cell_view),
+  if (priv->cell_view)
+    gtk_cell_layout_pack_start (GTK_CELL_LAYOUT (priv->cell_view),
                                 cell, expand);
 
-  if (combo_box->priv->column)
-    gtk_tree_view_column_pack_start (combo_box->priv->column, cell, expand);
+  if (priv->column)
+    gtk_tree_view_column_pack_start (priv->column, cell, expand);
 
-  if (GTK_IS_MENU (combo_box->priv->popup_widget))
-    pack_start_recurse (combo_box->priv->popup_widget, cell, expand);
+  if (GTK_IS_MENU (priv->popup_widget))
+    pack_start_recurse (priv->popup_widget, cell, expand);
 }
 
 static void
@@ -4067,32 +4262,47 @@ gtk_combo_box_cell_layout_pack_end (GtkCellLayout   *layout,
                                     GtkCellRenderer *cell,
                                     gboolean         expand)
 {
+  GtkComboBox *combo_box = GTK_COMBO_BOX (layout);
   ComboCellInfo *info;
-  GtkComboBox *combo_box;
-
-  g_return_if_fail (GTK_IS_COMBO_BOX (layout));
-  g_return_if_fail (GTK_IS_CELL_RENDERER (cell));
+  GtkComboBoxPrivate *priv;
 
-  combo_box = GTK_COMBO_BOX (layout);
+  priv = combo_box->priv;
 
   g_object_ref_sink (cell);
 
-  info = g_new0 (ComboCellInfo, 1);
+  info = g_slice_new0 (ComboCellInfo);
   info->cell = cell;
   info->expand = expand;
   info->pack = GTK_PACK_END;
 
-  combo_box->priv->cells = g_slist_append (combo_box->priv->cells, info);
+  priv->cells = g_slist_append (priv->cells, info);
 
-  if (combo_box->priv->cell_view)
-    gtk_cell_layout_pack_end (GTK_CELL_LAYOUT (combo_box->priv->cell_view),
+  if (priv->cell_view)
+    gtk_cell_layout_pack_end (GTK_CELL_LAYOUT (priv->cell_view),
                               cell, expand);
 
-  if (combo_box->priv->column)
-    gtk_tree_view_column_pack_end (combo_box->priv->column, cell, expand);
+  if (priv->column)
+    gtk_tree_view_column_pack_end (priv->column, cell, expand);
 
-  if (GTK_IS_MENU (combo_box->priv->popup_widget))
-    pack_end_recurse (combo_box->priv->popup_widget, cell, expand);
+  if (GTK_IS_MENU (priv->popup_widget))
+    pack_end_recurse (priv->popup_widget, cell, expand);
+}
+
+static GList *
+gtk_combo_box_cell_layout_get_cells (GtkCellLayout *layout)
+{
+  GtkComboBox *combo_box = GTK_COMBO_BOX (layout);
+  GSList *list;
+  GList *retval = NULL;
+
+  for (list = combo_box->priv->cells; list; list = list->next)
+    {
+      ComboCellInfo *info = (ComboCellInfo *)list->data;
+
+      retval = g_list_prepend (retval, info->cell);
+    }
+
+  return g_list_reverse (retval);
 }
 
 static void
@@ -4118,33 +4328,30 @@ clear_recurse (GtkWidget *menu)
 static void
 gtk_combo_box_cell_layout_clear (GtkCellLayout *layout)
 {
-  GtkComboBox *combo_box;
+  GtkComboBox *combo_box = GTK_COMBO_BOX (layout);
+  GtkComboBoxPrivate *priv = combo_box->priv;
   GSList *i;
   
-  g_return_if_fail (GTK_IS_COMBO_BOX (layout));
-
-  combo_box = GTK_COMBO_BOX (layout);
-  if (combo_box->priv->cell_view)
-    gtk_cell_layout_clear (GTK_CELL_LAYOUT (combo_box->priv->cell_view));
+  if (priv->cell_view)
+    gtk_cell_layout_clear (GTK_CELL_LAYOUT (priv->cell_view));
 
-  if (combo_box->priv->column)
-    gtk_tree_view_column_clear (combo_box->priv->column);
+  if (priv->column)
+    gtk_tree_view_column_clear (priv->column);
 
-  for (i = combo_box->priv->cells; i; i = i->next)
+  for (i = priv->cells; i; i = i->next)
     {
-     ComboCellInfo *info = (ComboCellInfo *)i->data;
+      ComboCellInfo *info = (ComboCellInfo *)i->data;
 
       gtk_combo_box_cell_layout_clear_attributes (layout, info->cell);
       g_object_unref (info->cell);
-      g_free (info);
+      g_slice_free (ComboCellInfo, info);
       i->data = NULL;
     }
-  g_slist_free (combo_box->priv->cells);
-  combo_box->priv->cells = NULL;
+  g_slist_free (priv->cells);
+  priv->cells = NULL;
 
-  if (GTK_IS_MENU (combo_box->priv->popup_widget))
-    clear_recurse (combo_box->priv->popup_widget);
+  if (GTK_IS_MENU (priv->popup_widget))
+    clear_recurse (priv->popup_widget);
 }
 
 static void
@@ -4177,15 +4384,11 @@ gtk_combo_box_cell_layout_add_attribute (GtkCellLayout   *layout,
                                          const gchar     *attribute,
                                          gint             column)
 {
+  GtkComboBox *combo_box = GTK_COMBO_BOX (layout);
   ComboCellInfo *info;
-  GtkComboBox *combo_box;
-
-  g_return_if_fail (GTK_IS_COMBO_BOX (layout));
-  g_return_if_fail (GTK_IS_CELL_RENDERER (cell));
-
-  combo_box = GTK_COMBO_BOX (layout);
 
   info = gtk_combo_box_get_cell_info (combo_box, cell);
+  g_return_if_fail (info != NULL);
 
   info->attributes = g_slist_prepend (info->attributes,
                                       GINT_TO_POINTER (column));
@@ -4218,8 +4421,8 @@ combo_cell_data_func (GtkCellLayout   *cell_layout,
   if (!info->func)
     return;
 
-  (*info->func) (cell_layout, cell, tree_model, iter, info->func_data);
-  
+  info->func (cell_layout, cell, tree_model, iter, info->func_data);
+
   if (GTK_IS_WIDGET (cell_layout))
     parent = gtk_widget_get_parent (GTK_WIDGET (cell_layout));
   
@@ -4230,9 +4433,9 @@ combo_cell_data_func (GtkCellLayout   *cell_layout,
 
 
 static void 
-set_cell_data_func_recurse (GtkWidget             *menu,
-                           GtkCellRenderer       *cell,
-                           ComboCellInfo         *info)
+set_cell_data_func_recurse (GtkWidget       *menu,
+                           GtkCellRenderer *cell,
+                           ComboCellInfo   *info)
 {
   GList *i, *list;
   GtkWidget *submenu;    
@@ -4267,12 +4470,9 @@ gtk_combo_box_cell_layout_set_cell_data_func (GtkCellLayout         *layout,
                                               gpointer               func_data,
                                               GDestroyNotify         destroy)
 {
+  GtkComboBox *combo_box = GTK_COMBO_BOX (layout);
+  GtkComboBoxPrivate *priv = combo_box->priv;
   ComboCellInfo *info;
-  GtkComboBox *combo_box;
-
-  g_return_if_fail (GTK_IS_COMBO_BOX (layout));
-
-  combo_box = GTK_COMBO_BOX (layout);
 
   info = gtk_combo_box_get_cell_info (combo_box, cell);
   g_return_if_fail (info != NULL);
@@ -4289,21 +4489,21 @@ gtk_combo_box_cell_layout_set_cell_data_func (GtkCellLayout         *layout,
   info->func_data = func_data;
   info->destroy = destroy;
 
-  if (combo_box->priv->cell_view)
-    gtk_cell_layout_set_cell_data_func (GTK_CELL_LAYOUT (combo_box->priv->cell_view), cell, func, func_data, NULL);
+  if (priv->cell_view)
+    gtk_cell_layout_set_cell_data_func (GTK_CELL_LAYOUT (priv->cell_view), cell, func, func_data, NULL);
 
-  if (combo_box->priv->column)
-    gtk_cell_layout_set_cell_data_func (GTK_CELL_LAYOUT (combo_box->priv->column), cell, func, func_data, NULL);
+  if (priv->column)
+    gtk_cell_layout_set_cell_data_func (GTK_CELL_LAYOUT (priv->column), cell, func, func_data, NULL);
 
-  if (GTK_IS_MENU (combo_box->priv->popup_widget))
-    set_cell_data_func_recurse (combo_box->priv->popup_widget, cell, info);
+  if (GTK_IS_MENU (priv->popup_widget))
+    set_cell_data_func_recurse (priv->popup_widget, cell, info);
 
   gtk_widget_queue_resize (GTK_WIDGET (combo_box));
 }
 
 static void 
-clear_attributes_recurse (GtkWidget             *menu,
-                         GtkCellRenderer       *cell)
+clear_attributes_recurse (GtkWidget       *menu,
+                         GtkCellRenderer *cell)
 {
   GList *i, *list;
   GtkWidget *submenu;    
@@ -4327,14 +4527,12 @@ static void
 gtk_combo_box_cell_layout_clear_attributes (GtkCellLayout   *layout,
                                             GtkCellRenderer *cell)
 {
+  GtkComboBox *combo_box = GTK_COMBO_BOX (layout);
+  GtkComboBoxPrivate *priv;
   ComboCellInfo *info;
-  GtkComboBox *combo_box;
   GSList *list;
 
-  g_return_if_fail (GTK_IS_COMBO_BOX (layout));
-  g_return_if_fail (GTK_IS_CELL_RENDERER (cell));
-
-  combo_box = GTK_COMBO_BOX (layout);
+  priv = combo_box->priv;
 
   info = gtk_combo_box_get_cell_info (combo_box, cell);
   g_return_if_fail (info != NULL);
@@ -4348,14 +4546,14 @@ gtk_combo_box_cell_layout_clear_attributes (GtkCellLayout   *layout,
   g_slist_free (info->attributes);
   info->attributes = NULL;
 
-  if (combo_box->priv->cell_view)
-    gtk_cell_layout_clear_attributes (GTK_CELL_LAYOUT (combo_box->priv->cell_view), cell);
+  if (priv->cell_view)
+    gtk_cell_layout_clear_attributes (GTK_CELL_LAYOUT (priv->cell_view), cell);
 
-  if (combo_box->priv->column)
-    gtk_cell_layout_clear_attributes (GTK_CELL_LAYOUT (combo_box->priv->column), cell);
+  if (priv->column)
+    gtk_cell_layout_clear_attributes (GTK_CELL_LAYOUT (priv->column), cell);
 
-  if (GTK_IS_MENU (combo_box->priv->popup_widget))
-    clear_attributes_recurse (combo_box->priv->popup_widget, cell);
+  if (GTK_IS_MENU (priv->popup_widget))
+    clear_attributes_recurse (priv->popup_widget, cell);
 
   gtk_widget_queue_resize (GTK_WIDGET (combo_box));
 }
@@ -4388,38 +4586,35 @@ gtk_combo_box_cell_layout_reorder (GtkCellLayout   *layout,
                                    GtkCellRenderer *cell,
                                    gint             position)
 {
+  GtkComboBox *combo_box = GTK_COMBO_BOX (layout);
+  GtkComboBoxPrivate *priv;
   ComboCellInfo *info;
-  GtkComboBox *combo_box;
   GSList *link;
 
-  g_return_if_fail (GTK_IS_COMBO_BOX (layout));
-  g_return_if_fail (GTK_IS_CELL_RENDERER (cell));
-
-  combo_box = GTK_COMBO_BOX (layout);
+  priv = combo_box->priv;
 
   info = gtk_combo_box_get_cell_info (combo_box, cell);
 
   g_return_if_fail (info != NULL);
   g_return_if_fail (position >= 0);
 
-  link = g_slist_find (combo_box->priv->cells, info);
+  link = g_slist_find (priv->cells, info);
 
   g_return_if_fail (link != NULL);
 
-  combo_box->priv->cells = g_slist_delete_link (combo_box->priv->cells, link);
-  combo_box->priv->cells = g_slist_insert (combo_box->priv->cells, info,
-                                           position);
+  priv->cells = g_slist_delete_link (priv->cells, link);
+  priv->cells = g_slist_insert (priv->cells, info, position);
 
-  if (combo_box->priv->cell_view)
-    gtk_cell_layout_reorder (GTK_CELL_LAYOUT (combo_box->priv->cell_view),
+  if (priv->cell_view)
+    gtk_cell_layout_reorder (GTK_CELL_LAYOUT (priv->cell_view),
                              cell, position);
 
-  if (combo_box->priv->column)
-    gtk_cell_layout_reorder (GTK_CELL_LAYOUT (combo_box->priv->column),
+  if (priv->column)
+    gtk_cell_layout_reorder (GTK_CELL_LAYOUT (priv->column),
                              cell, position);
 
-  if (GTK_IS_MENU (combo_box->priv->popup_widget))
-    reorder_recurse (combo_box->priv->popup_widget, cell, position);
+  if (GTK_IS_MENU (priv->popup_widget))
+    reorder_recurse (priv->popup_widget, cell, position);
 
   gtk_widget_queue_draw (GTK_WIDGET (combo_box));
 }
@@ -4500,12 +4695,16 @@ void
 gtk_combo_box_set_wrap_width (GtkComboBox *combo_box,
                               gint         width)
 {
+  GtkComboBoxPrivate *priv;
+
   g_return_if_fail (GTK_IS_COMBO_BOX (combo_box));
   g_return_if_fail (width >= 0);
 
-  if (width != combo_box->priv->wrap_width)
+  priv = combo_box->priv;
+
+  if (width != priv->wrap_width)
     {
-      combo_box->priv->wrap_width = width;
+      priv->wrap_width = width;
 
       gtk_combo_box_check_appearance (combo_box);
       gtk_combo_box_relayout (combo_box);
@@ -4547,16 +4746,19 @@ void
 gtk_combo_box_set_row_span_column (GtkComboBox *combo_box,
                                    gint         row_span)
 {
+  GtkComboBoxPrivate *priv;
   gint col;
 
   g_return_if_fail (GTK_IS_COMBO_BOX (combo_box));
 
-  col = gtk_tree_model_get_n_columns (combo_box->priv->model);
+  priv = combo_box->priv;
+
+  col = gtk_tree_model_get_n_columns (priv->model);
   g_return_if_fail (row_span >= -1 && row_span < col);
 
-  if (row_span != combo_box->priv->row_column)
+  if (row_span != priv->row_column)
     {
-      combo_box->priv->row_column = row_span;
+      priv->row_column = row_span;
       
       gtk_combo_box_relayout (combo_box);
  
@@ -4597,16 +4799,19 @@ void
 gtk_combo_box_set_column_span_column (GtkComboBox *combo_box,
                                       gint         column_span)
 {
+  GtkComboBoxPrivate *priv;
   gint col;
 
   g_return_if_fail (GTK_IS_COMBO_BOX (combo_box));
 
-  col = gtk_tree_model_get_n_columns (combo_box->priv->model);
+  priv = combo_box->priv;
+
+  col = gtk_tree_model_get_n_columns (priv->model);
   g_return_if_fail (column_span >= -1 && column_span < col);
 
-  if (column_span != combo_box->priv->col_column)
+  if (column_span != priv->col_column)
     {
-      combo_box->priv->col_column = column_span;
+      priv->col_column = column_span;
       
       gtk_combo_box_relayout (combo_box);
 
@@ -4632,14 +4837,18 @@ gtk_combo_box_set_column_span_column (GtkComboBox *combo_box,
 gint
 gtk_combo_box_get_active (GtkComboBox *combo_box)
 {
+  GtkComboBoxPrivate *priv;
   gint result;
+
   g_return_val_if_fail (GTK_IS_COMBO_BOX (combo_box), 0);
 
-  if (gtk_tree_row_reference_valid (combo_box->priv->active_row))
+  priv = combo_box->priv;
+
+  if (gtk_tree_row_reference_valid (priv->active_row))
     {
       GtkTreePath *path;
 
-      path = gtk_tree_row_reference_get_path (combo_box->priv->active_row);      
+      path = gtk_tree_row_reference_get_path (priv->active_row);      
       result = gtk_tree_path_get_indices (path)[0];
       gtk_tree_path_free (path);
     }
@@ -4667,6 +4876,14 @@ gtk_combo_box_set_active (GtkComboBox *combo_box,
   g_return_if_fail (GTK_IS_COMBO_BOX (combo_box));
   g_return_if_fail (index_ >= -1);
 
+  if (combo_box->priv->model == NULL)
+    {
+      /* Save index, in case the model is set after the index */
+      combo_box->priv->active = index_;
+      if (index_ != -1)
+        return;
+    }
+
   if (index_ != -1)
     path = gtk_tree_path_new_from_indices (index_, -1);
    
@@ -4680,58 +4897,69 @@ static void
 gtk_combo_box_set_active_internal (GtkComboBox *combo_box,
                                   GtkTreePath *path)
 {
+  GtkComboBoxPrivate *priv = combo_box->priv;
   GtkTreePath *active_path;
   gint path_cmp;
 
-  if (path && gtk_tree_row_reference_valid (combo_box->priv->active_row))
+  /* Remember whether the initially active row is valid. */
+  gboolean is_valid_row_reference = gtk_tree_row_reference_valid (priv->active_row);
+
+  if (path && is_valid_row_reference)
     {
-      active_path = gtk_tree_row_reference_get_path (combo_box->priv->active_row);
+      active_path = gtk_tree_row_reference_get_path (priv->active_row);
       path_cmp = gtk_tree_path_compare (path, active_path);
       gtk_tree_path_free (active_path);
       if (path_cmp == 0)
        return;
     }
 
-  if (combo_box->priv->active_row)
+  if (priv->active_row)
     {
-      gtk_tree_row_reference_free (combo_box->priv->active_row);
-      combo_box->priv->active_row = NULL;
+      gtk_tree_row_reference_free (priv->active_row);
+      priv->active_row = NULL;
     }
   
   if (!path)
     {
-      if (combo_box->priv->tree_view)
-        gtk_tree_selection_unselect_all (gtk_tree_view_get_selection (GTK_TREE_VIEW (combo_box->priv->tree_view)));
+      if (priv->tree_view)
+        gtk_tree_selection_unselect_all (gtk_tree_view_get_selection (GTK_TREE_VIEW (priv->tree_view)));
       else
         {
-          GtkMenu *menu = GTK_MENU (combo_box->priv->popup_widget);
+          GtkMenu *menu = GTK_MENU (priv->popup_widget);
 
           if (GTK_IS_MENU (menu))
             gtk_menu_set_active (menu, -1);
         }
 
-      if (combo_box->priv->cell_view)
-        gtk_cell_view_set_displayed_row (GTK_CELL_VIEW (combo_box->priv->cell_view), NULL);
+      if (priv->cell_view)
+        gtk_cell_view_set_displayed_row (GTK_CELL_VIEW (priv->cell_view), NULL);
+
+      /*
+       *  Do not emit a "changed" signal when an already invalid selection was
+       *  now set to invalid.
+       */
+      if (!is_valid_row_reference)
+        return;
     }
   else
     {
-      combo_box->priv->active_row = 
-       gtk_tree_row_reference_new (combo_box->priv->model, path);
+      priv->active_row = 
+       gtk_tree_row_reference_new (priv->model, path);
 
-      if (combo_box->priv->tree_view)
+      if (priv->tree_view)
        {
-         gtk_tree_view_set_cursor (GTK_TREE_VIEW (combo_box->priv->tree_view), 
+         gtk_tree_view_set_cursor (GTK_TREE_VIEW (priv->tree_view), 
                                    path, NULL, FALSE);
        }
-      else if (GTK_IS_MENU (combo_box->priv->popup_widget))
+      else if (GTK_IS_MENU (priv->popup_widget))
         {
          /* FIXME handle nested menus better */
-         gtk_menu_set_active (GTK_MENU (combo_box->priv->popup_widget), 
+         gtk_menu_set_active (GTK_MENU (priv->popup_widget), 
                               gtk_tree_path_get_indices (path)[0]);
         }
 
-      if (combo_box->priv->cell_view)
-       gtk_cell_view_set_displayed_row (GTK_CELL_VIEW (combo_box->priv->cell_view), 
+      if (priv->cell_view)
+       gtk_cell_view_set_displayed_row (GTK_CELL_VIEW (priv->cell_view), 
                                         path);
     }
 
@@ -4773,10 +5001,12 @@ gtk_combo_box_get_active_iter (GtkComboBox     *combo_box,
 /**
  * gtk_combo_box_set_active_iter:
  * @combo_box: A #GtkComboBox
- * @iter: The #GtkTreeIter
+ * @iter: (allow-none): The #GtkTreeIter, or %NULL
  * 
- * Sets the current active item to be the one referenced by @iter. 
- * @iter must correspond to a path of depth one.
+ * Sets the current active item to be the one referenced by @iter, or
+ * unsets the active item if @iter is %NULL.
+ *
+ * @iter must correspond to a path of depth one, or be %NULL.
  * 
  * Since: 2.4
  */
@@ -4784,11 +5014,13 @@ void
 gtk_combo_box_set_active_iter (GtkComboBox     *combo_box,
                                GtkTreeIter     *iter)
 {
-  GtkTreePath *path;
+  GtkTreePath *path = NULL;
 
   g_return_if_fail (GTK_IS_COMBO_BOX (combo_box));
 
-  path = gtk_tree_model_get_path (gtk_combo_box_get_model (combo_box), iter);
+  if (iter)
+    path = gtk_tree_model_get_path (gtk_combo_box_get_model (combo_box), iter);
+
   gtk_combo_box_set_active_internal (combo_box, path);
   gtk_tree_path_free (path);
 }
@@ -4796,9 +5028,9 @@ gtk_combo_box_set_active_iter (GtkComboBox     *combo_box,
 /**
  * gtk_combo_box_set_model:
  * @combo_box: A #GtkComboBox
- * @model: A #GtkTreeModel
+ * @model: (allow-none): A #GtkTreeModel
  *
- * Sets the model used by @combo_box to be @model. Will unset a previously set 
+ * Sets the model used by @combo_box to be @model. Will unset a previously set
  * model (if applicable). If model is %NULL, then it will unset the model.
  *
  * Note that this function does not clear the cell renderers, you have to 
@@ -4820,25 +5052,25 @@ gtk_combo_box_set_model (GtkComboBox  *combo_box,
   gtk_combo_box_unset_model (combo_box);
 
   if (model == NULL)
-    return;
+    goto out;
 
   combo_box->priv->model = model;
   g_object_ref (combo_box->priv->model);
 
   combo_box->priv->inserted_id =
-    g_signal_connect (combo_box->priv->model, "row_inserted",
+    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_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_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_signal_connect (combo_box->priv->model, "row-changed",
                      G_CALLBACK (gtk_combo_box_model_row_changed),
                      combo_box);
       
@@ -4860,6 +5092,18 @@ gtk_combo_box_set_model (GtkComboBox  *combo_box,
   if (combo_box->priv->cell_view)
     gtk_cell_view_set_model (GTK_CELL_VIEW (combo_box->priv->cell_view),
                              combo_box->priv->model);
+
+  if (combo_box->priv->active != -1)
+    {
+      /* If an index was set in advance, apply it now */
+      gtk_combo_box_set_active (combo_box, combo_box->priv->active);
+      combo_box->priv->active = -1;
+    }
+
+out:
+  gtk_combo_box_update_sensitivity (combo_box);
+
+  g_object_notify (G_OBJECT (combo_box), "model");
 }
 
 /**
@@ -4868,7 +5112,7 @@ gtk_combo_box_set_model (GtkComboBox  *combo_box,
  *
  * Returns the #GtkTreeModel which is acting as data source for @combo_box.
  *
- * Return value: A #GtkTreeModel which was passed during construction.
+ * Return value: (transfer none): A #GtkTreeModel which was passed during construction.
  *
  * Since: 2.4
  */
@@ -4893,7 +5137,7 @@ gtk_combo_box_get_model (GtkComboBox *combo_box)
  * gtk_combo_box_insert_text(), gtk_combo_box_prepend_text() and
  * gtk_combo_box_remove_text().
  *
- * Return value: A new text combo box.
+ * Return value: (transfer none): A new text combo box.
  *
  * Since: 2.4
  */
@@ -4937,6 +5181,8 @@ gtk_combo_box_append_text (GtkComboBox *combo_box,
 
   g_return_if_fail (GTK_IS_COMBO_BOX (combo_box));
   g_return_if_fail (GTK_IS_LIST_STORE (combo_box->priv->model));
+  g_return_if_fail (gtk_tree_model_get_column_type (combo_box->priv->model, 0)
+                   == G_TYPE_STRING);
   g_return_if_fail (text != NULL);
 
   store = GTK_LIST_STORE (combo_box->priv->model);
@@ -4968,6 +5214,8 @@ gtk_combo_box_insert_text (GtkComboBox *combo_box,
   g_return_if_fail (GTK_IS_COMBO_BOX (combo_box));
   g_return_if_fail (GTK_IS_LIST_STORE (combo_box->priv->model));
   g_return_if_fail (position >= 0);
+  g_return_if_fail (gtk_tree_model_get_column_type (combo_box->priv->model, 0)
+                   == G_TYPE_STRING);
   g_return_if_fail (text != NULL);
 
   store = GTK_LIST_STORE (combo_box->priv->model);
@@ -4996,6 +5244,8 @@ gtk_combo_box_prepend_text (GtkComboBox *combo_box,
 
   g_return_if_fail (GTK_IS_COMBO_BOX (combo_box));
   g_return_if_fail (GTK_IS_LIST_STORE (combo_box->priv->model));
+  g_return_if_fail (gtk_tree_model_get_column_type (combo_box->priv->model, 0)
+                   == G_TYPE_STRING);
   g_return_if_fail (text != NULL);
 
   store = GTK_LIST_STORE (combo_box->priv->model);
@@ -5023,6 +5273,8 @@ gtk_combo_box_remove_text (GtkComboBox *combo_box,
 
   g_return_if_fail (GTK_IS_COMBO_BOX (combo_box));
   g_return_if_fail (GTK_IS_LIST_STORE (combo_box->priv->model));
+  g_return_if_fail (gtk_tree_model_get_column_type (combo_box->priv->model, 0)
+                   == G_TYPE_STRING);
   g_return_if_fail (position >= 0);
 
   store = GTK_LIST_STORE (combo_box->priv->model);
@@ -5038,10 +5290,11 @@ gtk_combo_box_remove_text (GtkComboBox *combo_box,
  *
  * Returns the currently active string in @combo_box or %NULL if none
  * is selected.  Note that you can only use this function with combo
- * boxes constructed with gtk_combo_box_new_text() and with 
+ * boxes constructed with gtk_combo_box_new_text() and with
  * #GtkComboBoxEntry<!-- -->s.
  *
  * Returns: a newly allocated string containing the currently active text.
+ *     Must be freed with g_free().
  *
  * Since: 2.6
  */
@@ -5055,7 +5308,7 @@ gtk_combo_box_get_active_text (GtkComboBox *combo_box)
   class = GTK_COMBO_BOX_GET_CLASS (combo_box);
 
   if (class->get_active_text)
-    return (* class->get_active_text) (combo_box);
+    return class->get_active_text (combo_box);
 
   return NULL;
 }
@@ -5067,6 +5320,8 @@ gtk_combo_box_real_get_active_text (GtkComboBox *combo_box)
   gchar *text = NULL;
 
   g_return_val_if_fail (GTK_IS_LIST_STORE (combo_box->priv->model), NULL);
+  g_return_val_if_fail (gtk_tree_model_get_column_type (combo_box->priv->model, 0)
+                        == G_TYPE_STRING, NULL);
 
   if (gtk_combo_box_get_active_iter (combo_box, &iter))
     gtk_tree_model_get (combo_box->priv->model, &iter, 
@@ -5192,7 +5447,7 @@ gtk_combo_box_destroy (GtkObject *object)
   gtk_combo_box_popdown (combo_box);
 
   if (combo_box->priv->row_separator_destroy)
-    (* combo_box->priv->row_separator_destroy) (combo_box->priv->row_separator_data);
+    combo_box->priv->row_separator_destroy (combo_box->priv->row_separator_data);
 
   combo_box->priv->row_separator_func = NULL;
   combo_box->priv->row_separator_data = NULL;
@@ -5247,7 +5502,7 @@ gtk_combo_box_finalize (GObject *object)
       g_slist_free (info->attributes);
 
       g_object_unref (info->cell);
-      g_free (info);
+      g_slice_free (ComboCellInfo, info);
     }
    g_slist_free (combo_box->priv->cells);
 
@@ -5265,14 +5520,17 @@ gtk_cell_editable_key_press (GtkWidget   *widget,
 
   if (event->keyval == GDK_Escape)
     {
-      combo_box->priv->editing_canceled = TRUE;
-
+      g_object_set (combo_box,
+                    "editing-canceled", TRUE,
+                    NULL);
       gtk_cell_editable_editing_done (GTK_CELL_EDITABLE (combo_box));
       gtk_cell_editable_remove_widget (GTK_CELL_EDITABLE (combo_box));
       
       return TRUE;
     }
-  else if (event->keyval == GDK_Return)
+  else if (event->keyval == GDK_Return ||
+           event->keyval == GDK_ISO_Enter ||
+           event->keyval == GDK_KP_Enter)
     {
       gtk_cell_editable_editing_done (GTK_CELL_EDITABLE (combo_box));
       gtk_cell_editable_remove_widget (GTK_CELL_EDITABLE (combo_box));
@@ -5319,10 +5577,14 @@ popup_idle (gpointer data)
                             combo_box, 0);
   
   /* we unset this if a menu item is activated */
-  combo_box->priv->editing_canceled = TRUE;
+  g_object_set (combo_box,
+                "editing-canceled", TRUE,
+                NULL);
   gtk_combo_box_popup (combo_box);
 
   combo_box->priv->popup_idle_id = 0;
+  combo_box->priv->activate_button = 0;
+  combo_box->priv->activate_time = 0;
 
   return FALSE;
 }
@@ -5337,7 +5599,7 @@ gtk_combo_box_start_editing (GtkCellEditable *cell_editable,
 
   if (combo_box->priv->cell_view)
     {
-      g_signal_connect_object (combo_box->priv->button, "key_press_event",
+      g_signal_connect_object (combo_box->priv->button, "key-press-event",
                               G_CALLBACK (gtk_cell_editable_key_press), 
                               cell_editable, 0);  
 
@@ -5345,12 +5607,12 @@ gtk_combo_box_start_editing (GtkCellEditable *cell_editable,
     }
   else
     {
-      g_signal_connect_object (GTK_BIN (combo_box)->child, "key_press_event",
+      g_signal_connect_object (GTK_BIN (combo_box)->child, "key-press-event",
                               G_CALLBACK (gtk_cell_editable_key_press), 
                               cell_editable, 0);  
 
       gtk_widget_grab_focus (GTK_WIDGET (GTK_BIN (combo_box)->child));
-      GTK_WIDGET_UNSET_FLAGS (combo_box->priv->button, GTK_CAN_FOCUS);
+      gtk_widget_set_can_focus (combo_box->priv->button, FALSE);
     }
 
   /* we do the immediate popup only for the optionmenu-like 
@@ -5358,7 +5620,18 @@ gtk_combo_box_start_editing (GtkCellEditable *cell_editable,
    */  
   if (combo_box->priv->is_cell_renderer && 
       combo_box->priv->cell_view && !combo_box->priv->tree_view)
-    combo_box->priv->popup_idle_id = gdk_threads_add_idle (popup_idle, combo_box);
+    {
+      if (event && event->type == GDK_BUTTON_PRESS)
+        {
+          GdkEventButton *event_button = (GdkEventButton *)event;
+
+          combo_box->priv->activate_button = event_button->button;
+          combo_box->priv->activate_time = event_button->time;
+        }
+
+      combo_box->priv->popup_idle_id = 
+          gdk_threads_add_idle (popup_idle, combo_box);
+    }
 }
 
 
@@ -5449,13 +5722,17 @@ void
 gtk_combo_box_set_title (GtkComboBox *combo_box,
                         const gchar *title)
 {
+  GtkComboBoxPrivate *priv;
+
   g_return_if_fail (GTK_IS_COMBO_BOX (combo_box));
 
+  priv = combo_box->priv;
+
   if (strcmp (title ? title : "", 
-             combo_box->priv->tearoff_title ? combo_box->priv->tearoff_title : "") != 0)
+             priv->tearoff_title ? priv->tearoff_title : "") != 0)
     {
-      g_free (combo_box->priv->tearoff_title);
-      combo_box->priv->tearoff_title = g_strdup (title);
+      g_free (priv->tearoff_title);
+      priv->tearoff_title = g_strdup (title);
 
       gtk_combo_box_update_title (combo_box);
 
@@ -5463,14 +5740,6 @@ gtk_combo_box_set_title (GtkComboBox *combo_box,
     }
 }
 
-gboolean
-_gtk_combo_box_editing_canceled (GtkComboBox *combo_box)
-{
-  g_return_val_if_fail (GTK_IS_COMBO_BOX (combo_box), TRUE);
-
-  return combo_box->priv->editing_canceled;
-}
-
 /**
  * gtk_combo_box_get_popup_accessible:
  * @combo_box: a #GtkComboBox
@@ -5522,8 +5791,8 @@ gtk_combo_box_get_row_separator_func (GtkComboBox *combo_box)
  * gtk_combo_box_set_row_separator_func:
  * @combo_box: a #GtkComboBox
  * @func: a #GtkTreeViewRowSeparatorFunc
- * @data: user data to pass to @func, or %NULL
- * @destroy: destroy notifier for @data, or %NULL
+ * @data: (allow-none): user data to pass to @func, or %NULL
+ * @destroy: (allow-none): destroy notifier for @data, or %NULL
  * 
  * Sets the row separator function, which is used to determine
  * whether a row should be drawn as a separator. If the row separator
@@ -5535,12 +5804,12 @@ void
 gtk_combo_box_set_row_separator_func (GtkComboBox                 *combo_box,
                                      GtkTreeViewRowSeparatorFunc  func,
                                      gpointer                     data,
-                                     GtkDestroyNotify             destroy)
+                                     GDestroyNotify               destroy)
 {
   g_return_if_fail (GTK_IS_COMBO_BOX (combo_box));
 
   if (combo_box->priv->row_separator_destroy)
-    (* combo_box->priv->row_separator_destroy) (combo_box->priv->row_separator_data);
+    combo_box->priv->row_separator_destroy (combo_box->priv->row_separator_data);
 
   combo_box->priv->row_separator_func = func;
   combo_box->priv->row_separator_data = data;
@@ -5555,6 +5824,55 @@ gtk_combo_box_set_row_separator_func (GtkComboBox                 *combo_box,
   gtk_widget_queue_draw (GTK_WIDGET (combo_box));
 }
 
+/**
+ * gtk_combo_box_set_button_sensitivity:
+ * @combo_box: a #GtkComboBox
+ * @sensitivity: specify the sensitivity of the dropdown button
+ *
+ * Sets whether the dropdown button of the combo box should be
+ * always sensitive (%GTK_SENSITIVITY_ON), never sensitive (%GTK_SENSITIVITY_OFF)
+ * or only if there is at least one item to display (%GTK_SENSITIVITY_AUTO).
+ *
+ * Since: 2.14
+ **/
+void
+gtk_combo_box_set_button_sensitivity (GtkComboBox        *combo_box,
+                                      GtkSensitivityType  sensitivity)
+{
+  g_return_if_fail (GTK_IS_COMBO_BOX (combo_box));
+
+  if (combo_box->priv->button_sensitivity != sensitivity)
+    {
+      combo_box->priv->button_sensitivity = sensitivity;
+      gtk_combo_box_update_sensitivity (combo_box);
+
+      g_object_notify (G_OBJECT (combo_box), "button-sensitivity");
+    }
+}
+
+/**
+ * gtk_combo_box_get_button_sensitivity:
+ * @combo_box: a #GtkComboBox
+ *
+ * Returns whether the combo box sets the dropdown button
+ * sensitive or not when there are no items in the model.
+ *
+ * Return Value: %GTK_SENSITIVITY_ON if the dropdown button
+ *    is sensitive when the model is empty, %GTK_SENSITIVITY_OFF
+ *    if the button is always insensitive or
+ *    %GTK_SENSITIVITY_AUTO if it is only sensitive as long as
+ *    the model has one item to be selected.
+ *
+ * Since: 2.14
+ **/
+GtkSensitivityType
+gtk_combo_box_get_button_sensitivity (GtkComboBox *combo_box)
+{
+  g_return_val_if_fail (GTK_IS_COMBO_BOX (combo_box), FALSE);
+
+  return combo_box->priv->button_sensitivity;
+}
+
 
 /**
  * gtk_combo_box_set_focus_on_click: