]> Pileus Git - ~andy/gtk/blobdiff - gtk/gtkprogressbar.c
draw text twice, once with the prelight state and clipped to the progress
[~andy/gtk] / gtk / gtkprogressbar.c
index a0061b99aeec5534542125bf3a324c36f40eaf0d..94aefc1049d243242023fcfb42aaa95f22fdbbbc 100644 (file)
@@ -35,7 +35,7 @@
 #endif
 
 #include "gtkprogressbar.h"
-#include "gtksignal.h"
+#include "gtkintl.h"
 
 
 #define MIN_HORIZONTAL_BAR_WIDTH   150
 #define TEXT_SPACING               2
 
 enum {
-  ARG_0,
-  ARG_ADJUSTMENT,
-  ARG_ORIENTATION,
-  ARG_BAR_STYLE,
-  ARG_ACTIVITY_STEP,
-  ARG_ACTIVITY_BLOCKS,
-  ARG_DISCRETE_BLOCKS,
-  ARG_FRACTION,
-  ARG_PULSE_STEP
+  PROP_0,
+
+  /* Supported args */
+  PROP_FRACTION,
+  PROP_PULSE_STEP,
+  PROP_ORIENTATION,
+  PROP_TEXT,
+  
+  /* Deprecated args */
+  PROP_ADJUSTMENT,
+  PROP_BAR_STYLE,
+  PROP_ACTIVITY_STEP,
+  PROP_ACTIVITY_BLOCKS,
+  PROP_DISCRETE_BLOCKS
 };
 
 static void gtk_progress_bar_class_init    (GtkProgressBarClass *klass);
 static void gtk_progress_bar_init          (GtkProgressBar      *pbar);
-static void gtk_progress_bar_set_arg       (GtkObject           *object,
-                                           GtkArg              *arg,
-                                           guint                arg_id);
-static void gtk_progress_bar_get_arg       (GtkObject           *object,
-                                           GtkArg              *arg,
-                                           guint                arg_id);
+static void gtk_progress_bar_set_property  (GObject             *object,
+                                           guint                prop_id,
+                                           const GValue        *value,
+                                           GParamSpec          *pspec);
+static void gtk_progress_bar_get_property  (GObject             *object,
+                                           guint                prop_id,
+                                           GValue              *value,
+                                           GParamSpec          *pspec);
 static void gtk_progress_bar_size_request  (GtkWidget           *widget,
                                            GtkRequisition      *requisition);
 static void gtk_progress_bar_real_update   (GtkProgress         *progress);
 static void gtk_progress_bar_paint         (GtkProgress         *progress);
 static void gtk_progress_bar_act_mode_enter (GtkProgress        *progress);
 
+static void gtk_progress_bar_set_bar_style_internal       (GtkProgressBar *pbar,
+                                                          GtkProgressBarStyle style);
+static void gtk_progress_bar_set_discrete_blocks_internal (GtkProgressBar *pbar,
+                                                          guint           blocks);
+static void gtk_progress_bar_set_activity_step_internal   (GtkProgressBar *pbar,
+                                                          guint           step);
+static void gtk_progress_bar_set_activity_blocks_internal (GtkProgressBar *pbar,
+                                                          guint           blocks);
 
-GtkType
+
+GType
 gtk_progress_bar_get_type (void)
 {
-  static GtkType progress_bar_type = 0;
+  static GType progress_bar_type = 0;
 
   if (!progress_bar_type)
     {
-      static const GtkTypeInfo progress_bar_info =
+      static const GTypeInfo progress_bar_info =
       {
-       "GtkProgressBar",
-       sizeof (GtkProgressBar),
        sizeof (GtkProgressBarClass),
-       (GtkClassInitFunc) gtk_progress_bar_class_init,
-       (GtkObjectInitFunc) gtk_progress_bar_init,
-        /* reserved_1 */ NULL,
-        /* reserved_2 */ NULL,
-        (GtkClassInitFunc) NULL
+       NULL,           /* base_init */
+       NULL,           /* base_finalize */
+       (GClassInitFunc) gtk_progress_bar_class_init,
+       NULL,           /* class_finalize */
+       NULL,           /* class_data */
+       sizeof (GtkProgressBar),
+       0,              /* n_preallocs */
+       (GInstanceInitFunc) gtk_progress_bar_init,
       };
 
-      progress_bar_type = gtk_type_unique (GTK_TYPE_PROGRESS, &progress_bar_info);
+      progress_bar_type =
+       g_type_register_static (GTK_TYPE_PROGRESS, "GtkProgressBar",
+                               &progress_bar_info, 0);
     }
 
   return progress_bar_type;
@@ -100,55 +119,107 @@ gtk_progress_bar_get_type (void)
 static void
 gtk_progress_bar_class_init (GtkProgressBarClass *class)
 {
-  GtkObjectClass *object_class;
+  GObjectClass *gobject_class;
   GtkWidgetClass *widget_class;
   GtkProgressClass *progress_class;
   
-  object_class = (GtkObjectClass *) class;
+  gobject_class = G_OBJECT_CLASS (class);
   widget_class = (GtkWidgetClass *) class;
   progress_class = (GtkProgressClass *) class;
-  
-  gtk_object_add_arg_type ("GtkProgressBar::adjustment",
-                          GTK_TYPE_ADJUSTMENT,
-                          GTK_ARG_READWRITE,
-                          ARG_ADJUSTMENT);
-  gtk_object_add_arg_type ("GtkProgressBar::orientation",
-                          GTK_TYPE_PROGRESS_BAR_ORIENTATION,
-                          GTK_ARG_READWRITE,
-                          ARG_ORIENTATION);
-  gtk_object_add_arg_type ("GtkProgressBar::bar_style",
-                          GTK_TYPE_PROGRESS_BAR_STYLE,
-                          GTK_ARG_READWRITE,
-                          ARG_BAR_STYLE);
-  gtk_object_add_arg_type ("GtkProgressBar::activity_step",
-                          GTK_TYPE_UINT,
-                          GTK_ARG_READWRITE,
-                          ARG_ACTIVITY_STEP);
-  gtk_object_add_arg_type ("GtkProgressBar::activity_blocks",
-                          GTK_TYPE_UINT,
-                          GTK_ARG_READWRITE,
-                          ARG_ACTIVITY_BLOCKS);
-  gtk_object_add_arg_type ("GtkProgressBar::discrete_blocks",
-                          GTK_TYPE_UINT,
-                          GTK_ARG_READWRITE,
-                          ARG_DISCRETE_BLOCKS);
-  gtk_object_add_arg_type ("GtkProgressBar::fraction",
-                          GTK_TYPE_FLOAT,
-                          GTK_ARG_READWRITE,
-                          ARG_FRACTION);
-  gtk_object_add_arg_type ("GtkProgressBar::pulse_step",
-                          GTK_TYPE_FLOAT,
-                          GTK_ARG_READWRITE,
-                          ARG_PULSE_STEP);
-  
-  object_class->set_arg = gtk_progress_bar_set_arg;
-  object_class->get_arg = gtk_progress_bar_get_arg;
 
+  gobject_class->set_property = gtk_progress_bar_set_property;
+  gobject_class->get_property = gtk_progress_bar_get_property;
+  
   widget_class->size_request = gtk_progress_bar_size_request;
 
   progress_class->paint = gtk_progress_bar_paint;
   progress_class->update = gtk_progress_bar_real_update;
   progress_class->act_mode_enter = gtk_progress_bar_act_mode_enter;
+
+  g_object_class_install_property (gobject_class,
+                                   PROP_ADJUSTMENT,
+                                   g_param_spec_object ("adjustment",
+                                                        _("Adjustment"),
+                                                        _("The GtkAdjustment connected to the progress bar (Deprecated)"),
+                                                        GTK_TYPE_ADJUSTMENT,
+                                                        G_PARAM_READWRITE));
+
+  g_object_class_install_property (gobject_class,
+                                   PROP_ORIENTATION,
+                                   g_param_spec_enum ("orientation",
+                                                     _("Orientation"),
+                                                     _("Orientation and growth direction of the progress bar"),
+                                                     GTK_TYPE_PROGRESS_BAR_ORIENTATION,
+                                                     GTK_PROGRESS_LEFT_TO_RIGHT,
+                                                     G_PARAM_READWRITE));
+
+  g_object_class_install_property (gobject_class,
+                                   PROP_BAR_STYLE,
+                                   g_param_spec_enum ("bar_style",
+                                                     _("Bar style"),
+                                                     _("Specifies the visual style of the bar in percentage mode (Deprecated)"),
+                                                     GTK_TYPE_PROGRESS_BAR_STYLE,
+                                                     GTK_PROGRESS_CONTINUOUS,
+                                                     G_PARAM_READWRITE));
+
+  g_object_class_install_property (gobject_class,
+                                   PROP_ACTIVITY_STEP,
+                                   g_param_spec_uint ("activity_step",
+                                                     _("Activity Step"),
+                                                     _("The increment used for each iteration in activity mode (Deprecated)"),
+                                                     -G_MAXUINT,
+                                                     G_MAXUINT,
+                                                     3,
+                                                     G_PARAM_READWRITE));
+
+  g_object_class_install_property (gobject_class,
+                                   PROP_ACTIVITY_BLOCKS,
+                                   g_param_spec_uint ("activity_blocks",
+                                                     _("Activity Blocks"),
+                                                     _("The number of blocks which can fit in the progress bar area in activity mode (Deprecated)"),
+                                                     2,
+                                                     G_MAXUINT,
+                                                     5,
+                                                     G_PARAM_READWRITE));
+
+  g_object_class_install_property (gobject_class,
+                                   PROP_DISCRETE_BLOCKS,
+                                   g_param_spec_uint ("discrete_blocks",
+                                                     _("Discrete Blocks"),
+                                                     _("The number of discrete blocks in a progress bar (when shown in the discrete style)"),
+                                                     2,
+                                                     G_MAXUINT,
+                                                     10,
+                                                     G_PARAM_READWRITE));
+  
+  g_object_class_install_property (gobject_class,
+                                  PROP_FRACTION,
+                                  g_param_spec_double ("fraction",
+                                                       _("Fraction"),
+                                                       _("The fraction of total work that has been completed"),
+                                                       0.0,
+                                                       1.0,
+                                                       0.0,
+                                                       G_PARAM_READWRITE));  
+  
+  g_object_class_install_property (gobject_class,
+                                  PROP_PULSE_STEP,
+                                  g_param_spec_double ("pulse_step",
+                                                       _("Pulse Step"),
+                                                       _("The fraction of total progress to move the bouncing block when pulsed"),
+                                                       0.0,
+                                                       1.0,
+                                                       0.1,
+                                                       G_PARAM_READWRITE));  
+  
+  g_object_class_install_property (gobject_class,
+                                  PROP_TEXT,
+                                  g_param_spec_string ("text",
+                                                       _("Text"),
+                                                       _("Text to be displayed in the progress bar"),
+                                                       "%P %%",
+                                                       G_PARAM_READWRITE));
+
 }
 
 static void
@@ -166,82 +237,92 @@ gtk_progress_bar_init (GtkProgressBar *pbar)
 }
 
 static void
-gtk_progress_bar_set_arg (GtkObject           *object,
-                         GtkArg              *arg,
-                         guint                arg_id)
+gtk_progress_bar_set_property (GObject      *object,
+                              guint         prop_id,
+                              const GValue *value,
+                              GParamSpec   *pspec)
 {
   GtkProgressBar *pbar;
 
   pbar = GTK_PROGRESS_BAR (object);
 
-  switch (arg_id)
+  switch (prop_id)
     {
-    case ARG_ADJUSTMENT:
-      gtk_progress_set_adjustment (GTK_PROGRESS (pbar), GTK_VALUE_POINTER (*arg));
+    case PROP_ADJUSTMENT:
+      gtk_progress_set_adjustment (GTK_PROGRESS (pbar),
+                                  GTK_ADJUSTMENT (g_value_get_object (value)));
+      break;
+    case PROP_ORIENTATION:
+      gtk_progress_bar_set_orientation (pbar, g_value_get_enum (value));
       break;
-    case ARG_ORIENTATION:
-      gtk_progress_bar_set_orientation (pbar, GTK_VALUE_ENUM (*arg));
+    case PROP_BAR_STYLE:
+      gtk_progress_bar_set_bar_style_internal (pbar, g_value_get_enum (value));
       break;
-    case ARG_BAR_STYLE:
-      gtk_progress_bar_set_bar_style (pbar, GTK_VALUE_ENUM (*arg));
+    case PROP_ACTIVITY_STEP:
+      gtk_progress_bar_set_activity_step_internal (pbar, g_value_get_uint (value));
       break;
-    case ARG_ACTIVITY_STEP:
-      gtk_progress_bar_set_activity_step (pbar, GTK_VALUE_UINT (*arg));
+    case PROP_ACTIVITY_BLOCKS:
+      gtk_progress_bar_set_activity_blocks_internal (pbar, g_value_get_uint (value));
       break;
-    case ARG_ACTIVITY_BLOCKS:
-      gtk_progress_bar_set_activity_blocks (pbar, GTK_VALUE_UINT (*arg));
+    case PROP_DISCRETE_BLOCKS:
+      gtk_progress_bar_set_discrete_blocks_internal (pbar, g_value_get_uint (value));
       break;
-    case ARG_DISCRETE_BLOCKS:
-      gtk_progress_bar_set_discrete_blocks (pbar, GTK_VALUE_UINT (*arg));
+    case PROP_FRACTION:
+      gtk_progress_bar_set_fraction (pbar, g_value_get_double (value));
       break;
-    case ARG_FRACTION:
-      gtk_progress_bar_set_fraction (pbar, GTK_VALUE_FLOAT (*arg));
+    case PROP_PULSE_STEP:
+      gtk_progress_bar_set_pulse_step (pbar, g_value_get_double (value));
       break;
-    case ARG_PULSE_STEP:
-      gtk_progress_bar_set_pulse_step (pbar, GTK_VALUE_FLOAT (*arg));
+    case PROP_TEXT:
+      gtk_progress_bar_set_text (pbar, g_value_get_string (value));
       break;
     default:
+      G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
       break;
     }
 }
 
 static void
-gtk_progress_bar_get_arg (GtkObject           *object,
-                         GtkArg              *arg,
-                         guint                arg_id)
+gtk_progress_bar_get_property (GObject      *object,
+                              guint         prop_id,
+                              GValue       *value,
+                              GParamSpec   *pspec)
 {
   GtkProgressBar *pbar;
 
   pbar = GTK_PROGRESS_BAR (object);
 
-  switch (arg_id)
+  switch (prop_id)
     {
-    case ARG_ADJUSTMENT:
-      GTK_VALUE_POINTER (*arg) = GTK_PROGRESS (pbar)->adjustment;
+    case PROP_ADJUSTMENT:
+      g_value_set_object (value, GTK_PROGRESS (pbar)->adjustment);
       break;
-    case ARG_ORIENTATION:
-      GTK_VALUE_ENUM (*arg) = pbar->orientation;
+    case PROP_ORIENTATION:
+      g_value_set_enum (value, pbar->orientation);
       break;
-    case ARG_BAR_STYLE:
-      GTK_VALUE_ENUM (*arg) = pbar->bar_style;
+    case PROP_BAR_STYLE:
+      g_value_set_enum (value, pbar->bar_style);
       break;
-    case ARG_ACTIVITY_STEP:
-      GTK_VALUE_UINT (*arg) = pbar->activity_step;
+    case PROP_ACTIVITY_STEP:
+      g_value_set_uint (value, pbar->activity_step);
       break;
-    case ARG_ACTIVITY_BLOCKS:
-      GTK_VALUE_UINT (*arg) = pbar->activity_blocks;
+    case PROP_ACTIVITY_BLOCKS:
+      g_value_set_uint (value, pbar->activity_blocks);
       break;
-    case ARG_DISCRETE_BLOCKS:
-      GTK_VALUE_UINT (*arg) = pbar->blocks;
+    case PROP_DISCRETE_BLOCKS:
+      g_value_set_uint (value, pbar->blocks);
       break;
-    case ARG_FRACTION:
-      GTK_VALUE_FLOAT (*arg) = gtk_progress_get_current_percentage (GTK_PROGRESS (pbar));
+    case PROP_FRACTION:
+      g_value_set_double (value, gtk_progress_get_current_percentage (GTK_PROGRESS (pbar)));
       break;
-    case ARG_PULSE_STEP:
-      GTK_VALUE_FLOAT (*arg) = pbar->pulse_fraction;
+    case PROP_PULSE_STEP:
+      g_value_set_double (value, pbar->pulse_fraction);
+      break;
+    case PROP_TEXT:
+      g_value_set_string (value, gtk_progress_bar_get_text (pbar));
       break;
     default:
-      arg->type = GTK_TYPE_INVALID;
+      G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
       break;
     }
 }
@@ -261,7 +342,6 @@ gtk_progress_bar_new_with_adjustment (GtkAdjustment *adjustment)
 {
   GtkWidget *pbar;
 
-  g_return_val_if_fail (adjustment != NULL, NULL);
   g_return_val_if_fail (GTK_IS_ADJUSTMENT (adjustment), NULL);
 
   pbar = gtk_widget_new (GTK_TYPE_PROGRESS_BAR,
@@ -277,7 +357,6 @@ gtk_progress_bar_real_update (GtkProgress *progress)
   GtkProgressBar *pbar;
   GtkWidget *widget;
 
-  g_return_if_fail (progress != NULL);
   g_return_if_fail (GTK_IS_PROGRESS (progress));
 
   pbar = GTK_PROGRESS_BAR (progress);
@@ -362,7 +441,7 @@ gtk_progress_bar_real_update (GtkProgress *progress)
       gint in_block;
       
       in_block = -1 + (gint)(gtk_progress_get_current_percentage (progress) *
-                            (gfloat)pbar->blocks);
+                            (gdouble)pbar->blocks);
       
       if (pbar->in_block != in_block)
        {
@@ -383,7 +462,6 @@ gtk_progress_bar_size_request (GtkWidget      *widget,
   PangoRectangle logical_rect;
   PangoLayout *layout;
 
-  g_return_if_fail (widget != NULL);
   g_return_if_fail (GTK_IS_PROGRESS_BAR (widget));
   g_return_if_fail (requisition != NULL);
 
@@ -392,12 +470,15 @@ gtk_progress_bar_size_request (GtkWidget      *widget,
 
   if (progress->show_text && pbar->bar_style != GTK_PROGRESS_DISCRETE)
     {
+      if (!progress->adjustment)
+       gtk_progress_set_adjustment (progress, NULL);
+
       buf = gtk_progress_get_text_from_value (progress, progress->adjustment->upper);
 
       layout = gtk_widget_create_pango_layout (widget, buf);
       pango_layout_get_pixel_extents (layout, NULL, &logical_rect);
          
-      g_object_unref (G_OBJECT (layout));
+      g_object_unref (layout);
       g_free (buf);
     }
   
@@ -446,18 +527,28 @@ gtk_progress_bar_act_mode_enter (GtkProgress *progress)
   GtkProgressBar *pbar;
   GtkWidget *widget;
   gint size;
+  GtkProgressBarOrientation orientation;
 
   pbar = GTK_PROGRESS_BAR (progress);
   widget = GTK_WIDGET (progress);
 
+  orientation = pbar->orientation;
+  if (gtk_widget_get_direction (widget) == GTK_TEXT_DIR_RTL) 
+    {
+      if (pbar->orientation == GTK_PROGRESS_LEFT_TO_RIGHT)
+       orientation = GTK_PROGRESS_RIGHT_TO_LEFT;
+      else if (pbar->orientation == GTK_PROGRESS_RIGHT_TO_LEFT)
+       orientation = GTK_PROGRESS_LEFT_TO_RIGHT;
+    }
+  
   /* calculate start pos */
 
-  if (pbar->orientation == GTK_PROGRESS_LEFT_TO_RIGHT ||
-      pbar->orientation == GTK_PROGRESS_RIGHT_TO_LEFT)
+  if (orientation == GTK_PROGRESS_LEFT_TO_RIGHT ||
+      orientation == GTK_PROGRESS_RIGHT_TO_LEFT)
     {
       size = MAX (2, widget->allocation.width / pbar->activity_blocks);
 
-      if (pbar->orientation == GTK_PROGRESS_LEFT_TO_RIGHT)
+      if (orientation == GTK_PROGRESS_LEFT_TO_RIGHT)
        {
          pbar->activity_pos = widget->style->xthickness;
          pbar->activity_dir = 0;
@@ -474,7 +565,7 @@ gtk_progress_bar_act_mode_enter (GtkProgress *progress)
     {
       size = MAX (2, widget->allocation.height / pbar->activity_blocks);
 
-      if (pbar->orientation == GTK_PROGRESS_TOP_TO_BOTTOM)
+      if (orientation == GTK_PROGRESS_TOP_TO_BOTTOM)
        {
          pbar->activity_pos = widget->style->ythickness;
          pbar->activity_dir = 0;
@@ -490,35 +581,253 @@ gtk_progress_bar_act_mode_enter (GtkProgress *progress)
 }
 
 static void
-gtk_progress_bar_paint (GtkProgress *progress)
+gtk_progress_bar_paint_activity (GtkProgressBar            *pbar,
+                                GtkProgressBarOrientation  orientation)
 {
-  GtkProgressBar *pbar;
-  GtkWidget *widget;
-  gint amount;
-  gint block_delta = 0;
-  gint space = 0;
+  GtkWidget *widget = GTK_WIDGET (pbar);
+  GtkProgress *progress = GTK_PROGRESS (pbar);
+  gint x, y, w, h;
+
+  switch (orientation)
+    {
+    case GTK_PROGRESS_LEFT_TO_RIGHT:
+    case GTK_PROGRESS_RIGHT_TO_LEFT:
+      x = pbar->activity_pos;
+      y = widget->style->ythickness;
+      w = MAX (2, widget->allocation.width / pbar->activity_blocks);
+      h = widget->allocation.height - 2 * widget->style->ythickness;
+      break;
+
+    case GTK_PROGRESS_TOP_TO_BOTTOM:
+    case GTK_PROGRESS_BOTTOM_TO_TOP:
+      x = widget->style->xthickness;
+      y = pbar->activity_pos;
+      w = widget->allocation.width - 2 * widget->style->xthickness;
+      h = MAX (2, widget->allocation.height / pbar->activity_blocks);
+      break;
+
+    default:
+      return;
+      break;
+    }
+
+  gtk_paint_box (widget->style,
+                progress->offscreen_pixmap,
+                GTK_STATE_PRELIGHT, GTK_SHADOW_OUT,
+                NULL, widget, "bar",
+                x, y, w, h);
+}
+
+static void
+gtk_progress_bar_paint_continuous (GtkProgressBar            *pbar,
+                                  gint                       amount,
+                                  GtkProgressBarOrientation  orientation)
+{
+  GtkWidget *widget = GTK_WIDGET (pbar);
+  gint x, y, w, h;
+
+  if (amount <= 0)
+    return;
+
+  switch (orientation)
+    {
+    case GTK_PROGRESS_LEFT_TO_RIGHT:
+    case GTK_PROGRESS_RIGHT_TO_LEFT:
+      w = amount;
+      h = widget->allocation.height - widget->style->ythickness * 2;
+      y = widget->style->ythickness;
+      
+      x = widget->style->xthickness;
+      if (orientation == GTK_PROGRESS_RIGHT_TO_LEFT)
+       x = widget->allocation.width - amount - x;
+      break;
+      
+    case GTK_PROGRESS_TOP_TO_BOTTOM:
+    case GTK_PROGRESS_BOTTOM_TO_TOP:
+      w = widget->allocation.width - widget->style->xthickness * 2;
+      h = amount;
+      x = widget->style->xthickness;
+      
+      y = widget->style->ythickness;
+      if (orientation == GTK_PROGRESS_BOTTOM_TO_TOP)
+       y = widget->allocation.height - amount - y;
+      break;
+      
+    default:
+      return;
+      break;
+    }
+  
+  gtk_paint_box (widget->style,
+                GTK_PROGRESS (pbar)->offscreen_pixmap,
+                GTK_STATE_PRELIGHT, GTK_SHADOW_OUT,
+                NULL, widget, "bar",
+                x, y, w, h);
+}
+
+static void
+gtk_progress_bar_paint_discrete (GtkProgressBar            *pbar,
+                                GtkProgressBarOrientation  orientation)
+{
+  GtkWidget *widget = GTK_WIDGET (pbar);
   gint i;
+
+  for (i = 0; i <= pbar->in_block; i++)
+    {
+      gint x, y, w, h, space;
+
+      switch (orientation)
+       {
+       case GTK_PROGRESS_LEFT_TO_RIGHT:
+       case GTK_PROGRESS_RIGHT_TO_LEFT:
+         space = widget->allocation.width - 2 * widget->style->xthickness;
+         
+         x = widget->style->xthickness + (i * space) / pbar->blocks;
+         y = widget->style->ythickness;
+         w = widget->style->xthickness + ((i + 1) * space) / pbar->blocks - x;
+         h = widget->allocation.height - 2 * widget->style->ythickness;
+
+         if (orientation == GTK_PROGRESS_RIGHT_TO_LEFT)
+           x = widget->allocation.width - w - x;
+         break;
+         
+       case GTK_PROGRESS_TOP_TO_BOTTOM:
+       case GTK_PROGRESS_BOTTOM_TO_TOP:
+         space = widget->allocation.height - 2 * widget->style->ythickness;
+         
+         x = widget->style->xthickness;
+         y = widget->style->ythickness + (i * space) / pbar->blocks;
+         w = widget->allocation.width - 2 * widget->style->xthickness;
+         h = widget->style->ythickness + ((i + 1) * space) / pbar->blocks - y;
+         
+         if (orientation == GTK_PROGRESS_BOTTOM_TO_TOP)
+           y = widget->allocation.height - h - y;
+         break;
+
+       default:
+         return;
+         break;
+       }
+      
+      gtk_paint_box (widget->style,
+                    GTK_PROGRESS (pbar)->offscreen_pixmap,
+                    GTK_STATE_PRELIGHT, GTK_SHADOW_OUT,
+                    NULL, widget, "bar",
+                    x, y, w, h);
+    }
+}
+
+static void
+gtk_progress_bar_paint_text (GtkProgressBar            *pbar,
+                            gint                       amount,
+                            GtkProgressBarOrientation  orientation)
+{
+  GtkProgress *progress = GTK_PROGRESS (pbar);
+  GtkWidget *widget = GTK_WIDGET (pbar);
+  
   gint x;
   gint y;
-  gfloat percentage;
-  gint size;
+  gchar *buf;
+  GdkRectangle rect;
+  PangoLayout *layout;
+  PangoRectangle logical_rect;
+  GdkRectangle prelight_clip, normal_clip;
+  
+  buf = gtk_progress_get_current_text (progress);
+  
+  layout = gtk_widget_create_pango_layout (widget, buf);
+  pango_layout_get_pixel_extents (layout, NULL, &logical_rect);
+  
+  x = widget->style->xthickness + 1 + 
+    (widget->allocation.width - 2 * widget->style->xthickness -
+     3 - logical_rect.width)
+    * progress->x_align; 
+
+  y = widget->style->ythickness + 1 +
+    (widget->allocation.height - 2 * widget->style->ythickness -
+     3 - logical_rect.height)
+    * progress->y_align;
+
+  rect.x = widget->style->xthickness;
+  rect.y = widget->style->ythickness;
+  rect.width = widget->allocation.width - 2 * widget->style->xthickness;
+  rect.height = widget->allocation.height - 2 * widget->style->ythickness;
+
+  prelight_clip = normal_clip = rect;
+
+  switch (orientation)
+    {
+    case GTK_PROGRESS_LEFT_TO_RIGHT:
+      prelight_clip.width = amount;
+      normal_clip.x += amount;
+      normal_clip.width -= amount;
+      break;
+      
+    case GTK_PROGRESS_RIGHT_TO_LEFT:
+      normal_clip.width -= amount;
+      prelight_clip.x += normal_clip.width;
+      prelight_clip.width -= normal_clip.width;
+      break;
+       
+    case GTK_PROGRESS_TOP_TO_BOTTOM:
+      prelight_clip.height = amount;
+      normal_clip.y += amount;
+      normal_clip.height -= amount;
+      break;
+      
+    case GTK_PROGRESS_BOTTOM_TO_TOP:
+      normal_clip.height -= amount;
+      prelight_clip.y += normal_clip.height;
+      prelight_clip.height -= normal_clip.height;
+      break;
+    }
+  
+  gtk_paint_layout (widget->style,
+                   progress->offscreen_pixmap,
+                   GTK_STATE_PRELIGHT,
+                   FALSE,
+                   &prelight_clip,
+                   widget,
+                   "progressbar",
+                   x, y,
+                   layout);
+  
+  gtk_paint_layout (widget->style,
+                   progress->offscreen_pixmap,
+                   GTK_STATE_NORMAL,
+                   FALSE,
+                   &normal_clip,
+                   widget,
+                   "progressbar",
+                   x, y,
+                   layout);
+
+  g_object_unref (layout);
+  g_free (buf);
+}
+
+static void
+gtk_progress_bar_paint (GtkProgress *progress)
+{
+  GtkProgressBar *pbar;
+  GtkWidget *widget;
+
+  GtkProgressBarOrientation orientation;
 
-  g_return_if_fail (progress != NULL);
   g_return_if_fail (GTK_IS_PROGRESS_BAR (progress));
 
   pbar = GTK_PROGRESS_BAR (progress);
   widget = GTK_WIDGET (progress);
 
-  if (pbar->orientation == GTK_PROGRESS_LEFT_TO_RIGHT ||
-      pbar->orientation == GTK_PROGRESS_RIGHT_TO_LEFT)
-    space = widget->allocation.width -
-      2 * widget->style->xthickness;
-  else
-    space = widget->allocation.height -
-      2 * widget->style->ythickness;
-
-  percentage = gtk_progress_get_current_percentage (progress);
-
+  orientation = pbar->orientation;
+  if (gtk_widget_get_direction (widget) == GTK_TEXT_DIR_RTL) 
+    {
+      if (pbar->orientation == GTK_PROGRESS_LEFT_TO_RIGHT)
+       orientation = GTK_PROGRESS_RIGHT_TO_LEFT;
+      else if (pbar->orientation == GTK_PROGRESS_RIGHT_TO_LEFT)
+       orientation = GTK_PROGRESS_LEFT_TO_RIGHT;
+    }
   if (progress->offscreen_pixmap)
     {
       gtk_paint_box (widget->style,
@@ -531,257 +840,113 @@ gtk_progress_bar_paint (GtkProgress *progress)
       
       if (progress->activity_mode)
        {
-         if (pbar->orientation == GTK_PROGRESS_LEFT_TO_RIGHT ||
-             pbar->orientation == GTK_PROGRESS_RIGHT_TO_LEFT)
-           {
-             size = MAX (2, widget->allocation.width / pbar->activity_blocks);
-             
-             gtk_paint_box (widget->style,
-                            progress->offscreen_pixmap,
-                            GTK_STATE_PRELIGHT, GTK_SHADOW_OUT,
-                            NULL, widget, "bar",
-                            pbar->activity_pos,
-                            widget->style->ythickness,
-                            size,
-                            widget->allocation.height - widget->style->ythickness * 2);
-             return;
-           }
-         else
-           {
-             size = MAX (2, widget->allocation.height / pbar->activity_blocks);
-             
-             gtk_paint_box (widget->style,
-                            progress->offscreen_pixmap,
-                            GTK_STATE_PRELIGHT, GTK_SHADOW_OUT,
-                            NULL, widget, "bar",
-                            widget->style->xthickness,
-                            pbar->activity_pos,
-                            widget->allocation.width - widget->style->xthickness * 2,
-                            size);
-             return;
-           }
+         gtk_progress_bar_paint_activity (pbar, orientation);
        }
-      
-      amount = percentage * space;
-      
-      if (amount > 0)
+      else
        {
-         switch (pbar->orientation)
+         gint amount;
+         gint space;
+         
+         if (orientation == GTK_PROGRESS_LEFT_TO_RIGHT ||
+             orientation == GTK_PROGRESS_RIGHT_TO_LEFT)
+           space = widget->allocation.width - 2 * widget->style->xthickness;
+         else
+           space = widget->allocation.height - 2 * widget->style->ythickness;
+         
+         amount = space *
+           gtk_progress_get_current_percentage (GTK_PROGRESS (pbar));
+         
+         if (pbar->bar_style == GTK_PROGRESS_CONTINUOUS)
            {
-             
-           case GTK_PROGRESS_LEFT_TO_RIGHT:
-             
-             if (pbar->bar_style == GTK_PROGRESS_CONTINUOUS)
-               {
-                 gtk_paint_box (widget->style,
-                                progress->offscreen_pixmap,
-                                GTK_STATE_PRELIGHT, GTK_SHADOW_OUT,
-                                NULL, widget, "bar",
-                                widget->style->xthickness,
-                                widget->style->ythickness,
-                                amount,
-                                widget->allocation.height - widget->style->ythickness * 2);
-               }
-             else
-               {
-                 x = widget->style->xthickness;
-                 
-                 for (i = 0; i <= pbar->in_block; i++)
-                   {
-                     block_delta = (((i + 1) * space) / pbar->blocks)
-                       - ((i * space) / pbar->blocks);
-                     
-                     gtk_paint_box (widget->style,
-                                    progress->offscreen_pixmap,
-                                    GTK_STATE_PRELIGHT, GTK_SHADOW_OUT,
-                                    NULL, widget, "bar",
-                                    x,
-                                    widget->style->ythickness,
-                                    block_delta,
-                                    widget->allocation.height - widget->style->ythickness * 2);
-                     
-                     x +=  block_delta;
-                   }
-               }
-             break;
-             
-           case GTK_PROGRESS_RIGHT_TO_LEFT:
-             
-             if (pbar->bar_style == GTK_PROGRESS_CONTINUOUS)
-               {
-                 gtk_paint_box (widget->style,
-                                progress->offscreen_pixmap,
-                                GTK_STATE_PRELIGHT, GTK_SHADOW_OUT,
-                                NULL, widget, "bar",
-                                widget->allocation.width - 
-                                widget->style->xthickness - amount,
-                                widget->style->ythickness,
-                                amount,
-                                widget->allocation.height -
-                                widget->style->ythickness * 2);
-               }
-             else
-               {
-                 x = widget->allocation.width - 
-                   widget->style->xthickness;
-                 
-                 for (i = 0; i <= pbar->in_block; i++)
-                   {
-                     block_delta = (((i + 1) * space) / pbar->blocks) -
-                       ((i * space) / pbar->blocks);
-                     
-                     x -=  block_delta;
-                     
-                     gtk_paint_box (widget->style,
-                                    progress->offscreen_pixmap,
-                                    GTK_STATE_PRELIGHT, GTK_SHADOW_OUT,
-                                    NULL, widget, "bar",
-                                    x,
-                                    widget->style->ythickness,
-                                    block_delta,
-                                    widget->allocation.height -
-                                    widget->style->ythickness * 2);
-                   }
-               }
-             break;
+             gtk_progress_bar_paint_continuous (pbar, amount, orientation);
 
-           case GTK_PROGRESS_BOTTOM_TO_TOP:
-
-             if (pbar->bar_style == GTK_PROGRESS_CONTINUOUS)
-               {
-                 gtk_paint_box (widget->style,
-                                progress->offscreen_pixmap,
-                                GTK_STATE_PRELIGHT, GTK_SHADOW_OUT,
-                                NULL, widget, "bar",
-                                widget->style->xthickness,
-                                widget->allocation.height - 
-                                widget->style->ythickness - amount,
-                                widget->allocation.width -
-                                widget->style->xthickness * 2,
-                                amount);
-               }
-             else
-               {
-                 y = widget->allocation.height - 
-                   widget->style->ythickness;
-                 
-                 for (i = 0; i <= pbar->in_block; i++)
-                   {
-                     block_delta = (((i + 1) * space) / pbar->blocks) -
-                       ((i * space) / pbar->blocks);
-                     
-                     y -= block_delta;
-                     
-                     gtk_paint_box (widget->style,
-                                    progress->offscreen_pixmap,
-                                    GTK_STATE_PRELIGHT, GTK_SHADOW_OUT,
-                                    NULL, widget, "bar",
-                                    widget->style->xthickness,
-                                    y,
-                                    widget->allocation.width - 
-                                    widget->style->xthickness * 2,
-                                    block_delta);
-                   }
-               }
-             break;
-             
-           case GTK_PROGRESS_TOP_TO_BOTTOM:
-             
-             if (pbar->bar_style == GTK_PROGRESS_CONTINUOUS)
-               {
-                 gtk_paint_box (widget->style,
-                                progress->offscreen_pixmap,
-                                GTK_STATE_PRELIGHT, GTK_SHADOW_OUT,
-                                NULL, widget, "bar",
-                                widget->style->xthickness,
-                                widget->style->ythickness,
-                                widget->allocation.width -
-                                widget->style->xthickness * 2,
-                                amount);
-               }
-             else
-               {
-                 y = widget->style->ythickness;
-                 
-                 for (i = 0; i <= pbar->in_block; i++)
-                   {
-                     
-                     block_delta = (((i + 1) * space) / pbar->blocks)
-                       - ((i * space) / pbar->blocks);
-                     
-                     gtk_paint_box (widget->style,
-                                    progress->offscreen_pixmap,
-                                    GTK_STATE_PRELIGHT, GTK_SHADOW_OUT,
-                                    NULL, widget, "bar",
-                                    widget->style->xthickness,
-                                    y,
-                                    widget->allocation.width -
-                                    widget->style->xthickness * 2,
-                                    block_delta);
-                     
-                     y += block_delta;
-                   }
-               }
-             break;
-             
-           default:
-             break;
+             if (GTK_PROGRESS (pbar)->show_text)
+               gtk_progress_bar_paint_text (pbar, amount, orientation);
            }
+         else
+           gtk_progress_bar_paint_discrete (pbar, orientation);
        }
-      
-      if (progress->show_text && pbar->bar_style != GTK_PROGRESS_DISCRETE)
-       {
-         gint x;
-         gint y;
-         gchar *buf;
-         GdkRectangle rect;
-         PangoLayout *layout;
-         PangoRectangle logical_rect;
+    }
+}
+
+static void
+gtk_progress_bar_set_bar_style_internal (GtkProgressBar     *pbar,
+                                        GtkProgressBarStyle bar_style)
+{
+  g_return_if_fail (GTK_IS_PROGRESS_BAR (pbar));
 
-         buf = gtk_progress_get_current_text (progress);
+  if (pbar->bar_style != bar_style)
+    {
+      pbar->bar_style = bar_style;
 
-         layout = gtk_widget_create_pango_layout (widget, buf);
-         pango_layout_get_pixel_extents (layout, NULL, &logical_rect);
-         
-         x = widget->style->xthickness + 1 + 
-           (widget->allocation.width - 2 * widget->style->xthickness -
-            3 - logical_rect.width)
-           * progress->x_align; 
-
-         y = widget->style->ythickness + 1 +
-           (widget->allocation.height - 2 * widget->style->ythickness -
-            3 - logical_rect.height)
-           * progress->y_align;
-
-         rect.x = widget->style->xthickness + 1;
-         rect.y = widget->style->ythickness + 1;
-         rect.width = widget->allocation.width -
-           2 * widget->style->xthickness - 3;
-         rect.height = widget->allocation.height -
-           2 * widget->style->ythickness - 3;
-      
-          gtk_paint_layout (widget->style,
-                            progress->offscreen_pixmap,
-                            GTK_WIDGET_STATE (widget),
-                            &rect,
-                            widget,
-                            "progressbar",
-                            x, y,
-                            layout);
-
-          g_object_unref (G_OBJECT (layout));
-         g_free (buf);
-       }
+      if (GTK_WIDGET_DRAWABLE (GTK_WIDGET (pbar)))
+       gtk_widget_queue_resize (GTK_WIDGET (pbar));
+
+      g_object_notify (G_OBJECT (pbar), "bar_style");
+    }
+}
+
+static void
+gtk_progress_bar_set_discrete_blocks_internal (GtkProgressBar *pbar,
+                                              guint           blocks)
+{
+  g_return_if_fail (GTK_IS_PROGRESS_BAR (pbar));
+  g_return_if_fail (blocks > 1);
+
+  if (pbar->blocks != blocks)
+    {
+      pbar->blocks = blocks;
+
+      if (GTK_WIDGET_DRAWABLE (GTK_WIDGET (pbar)))
+       gtk_widget_queue_resize (GTK_WIDGET (pbar));
+
+      g_object_notify (G_OBJECT (pbar), "discrete_blocks");
+    }
+}
+
+static void
+gtk_progress_bar_set_activity_step_internal (GtkProgressBar *pbar,
+                                            guint           step)
+{
+  g_return_if_fail (GTK_IS_PROGRESS_BAR (pbar));
+
+  if (pbar->activity_step != step)
+    {
+      pbar->activity_step = step;
+      g_object_notify (G_OBJECT (pbar), "activity_step");
+    }
+}
+
+static void
+gtk_progress_bar_set_activity_blocks_internal (GtkProgressBar *pbar,
+                                              guint           blocks)
+{
+  g_return_if_fail (GTK_IS_PROGRESS_BAR (pbar));
+  g_return_if_fail (blocks > 1);
+
+  if (pbar->activity_blocks != blocks)
+    {
+      pbar->activity_blocks = blocks;
+      g_object_notify (G_OBJECT (pbar), "activity_blocks");
     }
 }
 
 /*******************************************************************/
 
+/**
+ * gtk_progress_bar_set_fraction:
+ * @pbar: a #GtkProgressBar
+ * @fraction: fraction of the task that's been completed
+ * 
+ * Causes the progress bar to "fill in" the given fraction
+ * of the bar. The fraction should be between 0.0 and 1.0,
+ * inclusive.
+ * 
+ **/
 void
 gtk_progress_bar_set_fraction (GtkProgressBar *pbar,
-                               gfloat          fraction)
+                               gdouble         fraction)
 {
-  g_return_if_fail (pbar != NULL);
   g_return_if_fail (GTK_IS_PROGRESS_BAR (pbar));
 
   /* If we know the percentage, we don't want activity mode. */
@@ -792,12 +957,23 @@ gtk_progress_bar_set_fraction (GtkProgressBar *pbar,
    * we can clean up all this code.
    */
   gtk_progress_set_percentage (GTK_PROGRESS (pbar), fraction);
+
+  g_object_notify (G_OBJECT (pbar), "fraction");
 }
 
+/**
+ * gtk_progress_bar_pulse:
+ * @pbar: a #GtkProgressBar
+ * 
+ * Indicates that some progress is made, but you don't know how much.
+ * Causes the progress bar to enter "activity mode," where a block
+ * bounces back and forth. Each call to gtk_progress_bar_pulse()
+ * causes the block to move by a little bit (the amount of movement
+ * per pulse is determined by gtk_progress_bar_set_pulse_step()).
+ **/
 void
 gtk_progress_bar_pulse (GtkProgressBar *pbar)
 {  
-  g_return_if_fail (pbar != NULL);
   g_return_if_fail (GTK_IS_PROGRESS_BAR (pbar));
 
   /* If we don't know the percentage, we must want activity mode. */
@@ -807,43 +983,54 @@ gtk_progress_bar_pulse (GtkProgressBar *pbar)
   gtk_progress_bar_real_update (GTK_PROGRESS (pbar));
 }
 
+/**
+ * gtk_progress_bar_set_text:
+ * @pbar: a #GtkProgressBar
+ * @text: a UTF-8 string
+ * 
+ * Causes the given @text to appear superimposed on the progress bar.
+ **/
 void
 gtk_progress_bar_set_text (GtkProgressBar *pbar,
-                           const gchar *text)
+                           const gchar    *text)
 {
-  g_return_if_fail (pbar != NULL);
   g_return_if_fail (GTK_IS_PROGRESS_BAR (pbar));
   
-  /* We don't support formats in this interface */
-  GTK_PROGRESS (pbar)->use_text_format = FALSE;
+  gtk_progress_set_show_text (GTK_PROGRESS (pbar), text && *text);
+  gtk_progress_set_format_string (GTK_PROGRESS (pbar), text);
   
-  if (text && *text)
-    {
-      gtk_progress_set_show_text (GTK_PROGRESS (pbar), TRUE);
-      gtk_progress_set_format_string (GTK_PROGRESS (pbar), text);
-    }
-  else
-    {
-      gtk_progress_set_show_text (GTK_PROGRESS (pbar), FALSE);
-      gtk_progress_set_format_string (GTK_PROGRESS (pbar), "");
-    }
+  /* We don't support formats in this interface, but turn
+   * them back on for NULL, which should put us back to
+   * the initial state.
+   */
+  GTK_PROGRESS (pbar)->use_text_format = (text == NULL);
+  
+  g_object_notify (G_OBJECT (pbar), "text");
 }
 
+/**
+ * gtk_progress_bar_set_pulse_step:
+ * @pbar: a #GtkProgressBar
+ * @fraction: fraction between 0.0 and 1.0
+ * 
+ * Sets the fraction of total progress bar length to move the
+ * bouncing block for each call to gtk_progress_bar_pulse().
+ **/
 void
 gtk_progress_bar_set_pulse_step   (GtkProgressBar *pbar,
-                                   gfloat          fraction)
+                                   gdouble         fraction)
 {
-  g_return_if_fail (pbar != NULL);
   g_return_if_fail (GTK_IS_PROGRESS_BAR (pbar));
   
   pbar->pulse_fraction = fraction;
+
+  g_object_notify (G_OBJECT (pbar), "pulse_step");
 }
 
 void
 gtk_progress_bar_update (GtkProgressBar *pbar,
-                        gfloat          percentage)
+                        gdouble         percentage)
 {
-  g_return_if_fail (pbar != NULL);
   g_return_if_fail (GTK_IS_PROGRESS_BAR (pbar));
 
   /* Use of gtk_progress_bar_update() is deprecated ! 
@@ -853,11 +1040,18 @@ gtk_progress_bar_update (GtkProgressBar *pbar,
   gtk_progress_set_percentage (GTK_PROGRESS (pbar), percentage);
 }
 
+/**
+ * gtk_progress_bar_set_orientation:
+ * @pbar: a #GtkProgressBar
+ * @orientation: orientation of the progress bar
+ * 
+ * Causes the progress bar to switch to a different orientation
+ * (left-to-right, right-to-left, top-to-bottom, or bottom-to-top). 
+ **/
 void
 gtk_progress_bar_set_orientation (GtkProgressBar           *pbar,
                                  GtkProgressBarOrientation orientation)
 {
-  g_return_if_fail (pbar != NULL);
   g_return_if_fail (GTK_IS_PROGRESS_BAR (pbar));
 
   if (pbar->orientation != orientation)
@@ -866,61 +1060,116 @@ gtk_progress_bar_set_orientation (GtkProgressBar           *pbar,
 
       if (GTK_WIDGET_DRAWABLE (GTK_WIDGET (pbar)))
        gtk_widget_queue_resize (GTK_WIDGET (pbar));
+
+      g_object_notify (G_OBJECT (pbar), "orientation");
     }
 }
 
+/**
+ * gtk_progress_bar_get_text:
+ * @pbar: a #GtkProgressBar
+ * 
+ * Retrieves the text displayed superimposed on the progress bar,
+ * if any, otherwise %NULL. The return value is a reference
+ * to the text, not a copy of it, so will become invalid
+ * if you change the text in the progress bar.
+ * 
+ * Return value: text, or %NULL; this string is owned by the widget
+ * and should not be modified or freed.
+ **/
+G_CONST_RETURN gchar*
+gtk_progress_bar_get_text (GtkProgressBar *pbar)
+{
+  g_return_val_if_fail (GTK_IS_PROGRESS_BAR (pbar), NULL);
+
+  if (GTK_PROGRESS (pbar)->use_text_format)
+    return NULL;
+  else
+    return GTK_PROGRESS (pbar)->format;
+}
+
+/**
+ * gtk_progress_bar_get_fraction:
+ * @pbar: a #GtkProgressBar
+ * 
+ * Returns the current fraction of the task that's been completed.
+ * 
+ * Return value: a fraction from 0.0 to 1.0
+ **/
+gdouble
+gtk_progress_bar_get_fraction (GtkProgressBar *pbar)
+{
+  g_return_val_if_fail (GTK_IS_PROGRESS_BAR (pbar), 0);
+
+  return gtk_progress_get_current_percentage (GTK_PROGRESS (pbar));
+}
+
+/**
+ * gtk_progress_bar_get_pulse_step:
+ * @pbar: a #GtkProgressBar
+ * 
+ * Retrieves the pulse step set with gtk_progress_bar_set_pulse_step()
+ * 
+ * Return value: a fraction from 0.0 to 1.0
+ **/
+gdouble
+gtk_progress_bar_get_pulse_step (GtkProgressBar *pbar)
+{
+  g_return_val_if_fail (GTK_IS_PROGRESS_BAR (pbar), 0);
+
+  return pbar->pulse_fraction;
+}
+
+/**
+ * gtk_progress_bar_get_orientation:
+ * @pbar: a #GtkProgressBar
+ * 
+ * Retrieves the current progress bar orientation.
+ * 
+ * Return value: orientation of the progress bar
+ **/
+GtkProgressBarOrientation
+gtk_progress_bar_get_orientation (GtkProgressBar *pbar)
+{
+  g_return_val_if_fail (GTK_IS_PROGRESS_BAR (pbar), 0);
+
+  return pbar->orientation;
+}
+
 void
 gtk_progress_bar_set_bar_style (GtkProgressBar     *pbar,
                                GtkProgressBarStyle bar_style)
 {
-  g_return_if_fail (pbar != NULL);
   g_return_if_fail (GTK_IS_PROGRESS_BAR (pbar));
 
-  if (pbar->bar_style != bar_style)
-    {
-      pbar->bar_style = bar_style;
-
-      if (GTK_WIDGET_DRAWABLE (GTK_WIDGET (pbar)))
-       gtk_widget_queue_resize (GTK_WIDGET (pbar));
-    }
+  gtk_progress_bar_set_bar_style_internal (pbar, bar_style);
 }
 
 void
 gtk_progress_bar_set_discrete_blocks (GtkProgressBar *pbar,
                                      guint           blocks)
 {
-  g_return_if_fail (pbar != NULL);
   g_return_if_fail (GTK_IS_PROGRESS_BAR (pbar));
   g_return_if_fail (blocks > 1);
 
-  if (pbar->blocks != blocks)
-    {
-      pbar->blocks = blocks;
-
-      if (GTK_WIDGET_DRAWABLE (GTK_WIDGET (pbar)))
-       gtk_widget_queue_resize (GTK_WIDGET (pbar));
-    }
+  gtk_progress_bar_set_discrete_blocks_internal (pbar, blocks);
 }
 
 void
 gtk_progress_bar_set_activity_step (GtkProgressBar *pbar,
                                     guint           step)
 {
-  g_return_if_fail (pbar != NULL);
   g_return_if_fail (GTK_IS_PROGRESS_BAR (pbar));
 
-  if (pbar->activity_step != step)
-    pbar->activity_step = step;
+  gtk_progress_bar_set_activity_step_internal (pbar, step);
 }
 
 void
 gtk_progress_bar_set_activity_blocks (GtkProgressBar *pbar,
                                      guint           blocks)
 {
-  g_return_if_fail (pbar != NULL);
   g_return_if_fail (GTK_IS_PROGRESS_BAR (pbar));
   g_return_if_fail (blocks > 1);
 
-  if (pbar->activity_blocks != blocks)
-    pbar->activity_blocks = blocks;
+  gtk_progress_bar_set_activity_blocks_internal (pbar, blocks);
 }