Merge lp:~timchen119/notify-osd/notify-osd.precise.lp404658 into lp:ubuntu/precise/notify-osd

[Tim Chen]

Provide the "x-canonical-private-synchronous" to work with notifications different with LAYOUT_ICON_INDICATOR layout.
Currently the "x-canonical-private-synchronous" interface only provide this layout to support brightness and volume control.

This make following command behave correctly.
$notify-send "RF-status" "enabled" -h string:x-canonical-private-synchronous:1 -h string:image_path:/tmp/test.jpg

Canonical OEM projects need this function to

* enable OSD for enable/disable WiFi or bluetooth with a hotkey
* enable OSD for display switching with a hotkey

[lp:~timchen119/notify-osd/notify-osd.precise.lp404658]

63. By Tim Chen on 2013-10-30

* Add changelog

[Sebastien Bacher (seb128)]

Thank you for the work, settings as rejected for the same reason than https://code.launchpad.net/~timchen119/notify-osd/notify-osd.lp404658/+merge/192467

Unmerged revisions

63. By Tim Chen on 2013-10-30

* Add changelog

62. By Tim Chen on 2013-10-24

  Provide the "x-canonical-private-synchronous" to work with notifications different with LAYOUT_ICON_INDICATOR layout.
  Currently the "x-canonical-private-synchronous" interface only provide this layout to support brightness and volume control.

  This make following command behave correctly.
  $notify-send "RF-status" "enabled" -h string:x-canonical-private-synchronous:1 -h string:image_path:/tmp/test.jpg

  Canonical OEM projects need this function to

  * enable OSD for enable/disable WiFi or bluetooth with a hotkey
  * enable OSD for display switching with a hotkey

=== modified file 'debian/changelog'
--- debian/changelog	2012-04-10 15:19:23 +0000
+++ debian/changelog	2013-10-30 07:06:13 +0000
@@ -1,3 +1,13 @@
+notify-osd (0.9.34-0ubuntu3) precise; urgency=low
+
+  * Provide the "x-canonical-private-synchronous" to work with notifications
+    different with LAYOUT_ICON_INDICATOR layout.
+    Currently the "x-canonical-private-synchronous" interface only provide 
+    this layout to support brightness and volume control.
+    (LP: #404658)
+
+ -- Jian-Ding Chen (timchen119) <[e-mail]>  Wed, 30 Oct 2013 15:02:28 +0800
+
 notify-osd (0.9.34-0ubuntu2) precise; urgency=low
 
   * Backport Cimi's color tweaks improvements (lp: #977974)

=== modified file 'src/stack.c'
--- src/stack.c	2011-07-11 18:24:23 +0000
+++ src/stack.c	2013-10-30 07:06:13 +0000
@@ -629,6 +629,29 @@
 		g_free (sender);
 	}
 
+	if (hints)
+	{
+		data   = (GValue*) g_hash_table_lookup (hints, "x-canonical-private-synchronous");
+		compat = (GValue*) g_hash_table_lookup (hints, "synchronous");
+		if ((data && G_VALUE_HOLDS_STRING (data)) || (compat && G_VALUE_HOLDS_STRING (compat)))
+		{
+			if (sync_bubble != NULL
+			    && IS_BUBBLE (sync_bubble))
+			{
+				g_object_unref (bubble);
+				bubble = sync_bubble;
+				/* Setup default value for default layout. */
+				bubble_set_value (bubble, -2);
+			}
+
+			if (data && G_VALUE_HOLDS_STRING (data))
+				bubble_set_synchronous (bubble, g_value_get_string (data));
+
+			if (compat && G_VALUE_HOLDS_STRING (compat))
+				bubble_set_synchronous (bubble, g_value_get_string (compat));
+		}
+	}
+
 	if (new_bubble && hints)
 	{
 		data   = (GValue*) g_hash_table_lookup (hints, "x-canonical-append");
@@ -662,27 +685,6 @@
 
 	if (hints)
 	{
-		data   = (GValue*) g_hash_table_lookup (hints, "x-canonical-private-synchronous");
-		compat = (GValue*) g_hash_table_lookup (hints, "synchronous");
-		if ((data && G_VALUE_HOLDS_STRING (data)) || (compat && G_VALUE_HOLDS_STRING (compat)))
-		{
-			if (sync_bubble != NULL
-			    && IS_BUBBLE (sync_bubble))
-			{
-				g_object_unref (bubble);
-				bubble = sync_bubble;
-			}
-
-			if (data && G_VALUE_HOLDS_STRING (data))
-				bubble_set_synchronous (bubble, g_value_get_string (data));
-
-			if (compat && G_VALUE_HOLDS_STRING (compat))
-				bubble_set_synchronous (bubble, g_value_get_string (compat));
-		}
-	}
-
-	if (hints)
-	{
 		data = (GValue*) g_hash_table_lookup (hints, "value");
 		if (data && G_VALUE_HOLDS_INT (data))
 			bubble_set_value (bubble, g_value_get_int (data));