Fix notify::label emmision in GtkMenuItem

Emit notify::label in GtkMenuItem also when label is changed through GtkAction.

Refactor GtkMenuItem and remove duplicated code for GtkLabel creation. Reset
the accel-widget back to the GtkMenuItem itself when there is no action related
to the GtkMenuItem anymore.

Add test for notify::label emmisions.

Fixes bug 612574 - GtkMenuItem does not emit notify::label when label is
changed through GtkAction.

2 files changed, 95 insertions, 0 deletions

diff --git a/gtk/tests/Makefile.am b/gtk/tests/Makefile.am
index 7056dcd073..d257f2b316 100644
--- a/gtk/tests/Makefile.am
+++ b/gtk/tests/Makefile.am
@@ -91,4 +91,8 @@ TEST_PROGS			+= expander
 expander_SOURCES		 = expander.c
 expander_LDADD		 = $(progs_ldadd)
 
+TEST_PROGS			+= action
+action_SOURCES			 = action.c
+action_LDADD			 = $(progs_ldadd)
+
 -include $(top_srcdir)/git.mk
diff --git a/gtk/tests/action.c b/gtk/tests/action.c
new file mode 100644
index 0000000000..97b25d0d76
--- /dev/null
+++ b/gtk/tests/action.c
@@ -0,0 +1,91 @@
+/* GtkAction tests.
+ *
+ * Authors: Jan Arne Petersen <jpetersen@openismus.com>
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the
+ * Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+ * Boston, MA 02111-1307, USA.
+ */
+
+#include <gtk/gtk.h>
+
+/* Fixture */
+
+typedef struct
+{
+  GtkAction *action;
+} ActionTest;
+
+static void
+action_test_setup (ActionTest    *fixture,
+                   gconstpointer  test_data)
+{
+  fixture->action = gtk_action_new ("name", "label", NULL, NULL);
+}
+
+static void
+action_test_teardown (ActionTest    *fixture,
+                      gconstpointer  test_data)
+{
+  g_object_unref (fixture->action);
+}
+
+static void
+notify_count_emmisions (GObject    *object,
+			GParamSpec *pspec,
+			gpointer    data)
+{
+  unsigned int *i = data;
+  (*i)++;
+}
+
+static void
+menu_item_label_notify_count (ActionTest    *fixture,
+                              gconstpointer  test_data)
+{
+  GtkMenuItem *item = gtk_menu_item_new ();
+  unsigned int emmisions = 0;
+
+  g_signal_connect (item, "notify::label",
+		    G_CALLBACK (notify_count_emmisions), &emmisions);
+
+  gtk_activatable_do_set_related_action (GTK_ACTIVATABLE (item),
+					 fixture->action);
+
+  g_assert_cmpuint (emmisions, ==, 1);
+
+  gtk_action_set_label (fixture->action, "new label");
+
+  g_assert_cmpuint (emmisions, ==, 2);
+
+  g_object_unref (item);
+}
+
+/* main */
+
+int
+main (int    argc,
+      char **argv)
+{
+  gtk_test_init (&argc, &argv, NULL);
+
+  g_test_add ("/Action/MenuItem/label-notify-count",
+              ActionTest, NULL,
+              action_test_setup,
+              menu_item_label_notify_count,
+              action_test_teardown);
+
+  return g_test_run ();
+}
+