Re: [PATCH v31 0/4] MR2808: SNI tray implementation

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/win32u/snidrv/dbus.c:
> - UINT id;
> - unsigned int notification_id;
>
> +};
> -
>
> +static struct list standalone_notification_list = LIST_INIT( standalone_notification_list );
> -
>
> +static pthread_mutex_t standalone_notifications_mutex = PTHREAD_MUTEX_INITIALIZER;
> -
> -
>
> +#define BALLOON_SHOW_MIN_TIMEOUT 10000
> +#define BALLOON_SHOW_MAX_TIMEOUT 30000
> -
>
> +static void* dbus_module = NULL;
> +static const char* kde_watcher_interface_name = "org.kde.StatusNotifierWatcher";
> +static const char* freedesktop_watcher_interface_name = "org.freedesktop.StatusNotifierWatcher";
> +static const char* watcher_interface_name = "org.kde.StatusNotifierWatcher";

I don't think we want to have DE specific implementations. This should use the FDO standard, or if it's not standard enough yet, it's probably too early to implement it.