Comment 10 for bug 334344

Revision history for this message
In , Chinmay-patel (chinmay-patel) wrote :

Created an attachment (id=364831)
Adding Libnotify Support

I have checked that this patch works fine when Libnotify is not installed in the machine.
The only issue with this patch is that it shows libnotify notifications only with trunk build, not with 1.9.1 branch.

In the trunk there is a check at line
http://mxr.mozilla.org/mozilla-central/source/toolkit/components/alerts/src/nsAlertsService.cpp#78 available. It checks wether Libnotify is available or not. If it there, use Libnotify notifications.

But this check is not available in the 1.9.1 branch.

I have checked the patch with latest mozilla trunk and it worked pretty well.