110385 - Sound is not asynchronous on linux

Status: RESOLVED WORKSFORME

(Core :: Widget: Gtk, defect)

Description

[Akkana Peck]

If you do anything involving sound beyond the system beep (e.g. set sounds for
new mail notification or for chatzilla msgs), the UI locks up for several
seconds before it plays the sound.

This is mentioned in bug 94522 and bug 104174, but there doesn't seem to be a
bug that covers the issue, so I'm filing it.  There's discussion of some
possible implementations in bug 104174.

Comment 1

[Akkana Peck]

Marking helpwanted -- I know this isn't a high priority, I just wanted a bug
tracking the issue in case anyone decided to work on it.  Hyatt's not the right
owner for this, but I don't know who is (nobody?)

Comment 2

[David Hyatt]

--> default owner

Comment 3

[Aaron Leventhal]

Anyone want to be the hero and fix this so we can use sound on Linux/UNIX?

Comment 4

[Kyle Yuan]

cc simford

Comment 5

[geraldo]

just wanted to report that this bug is still active in Mozilla1.7b on
Linux/Mandrake9.1 with oss sounddrivers.

but what happens on my system differs a little bit from the above discribed: the
UI (and all other Mozilla releated windows/applications) locks up from the
moment on a wav-file is played till one second before it stops playing. so if
you play a 1-minute sound Mozilla almost is locked up for this minute!

Comment 6

[Jim Booth]

(In reply to comment #5)

I'm seeing something very similar to what geraldo describes, but with 1.8a2 on
Windows ME, and it has started sometime since early June 2004.  Not sure if
Mozilla is at fault, as other wav files are affected too.  I have Googled for
this problem on the net and have found nothing, so I'm leaving a note here in
case anyone else has seen this on Windows.

Comment 7

[Vidar Haarr (not reading bugmail)]

Implementing this should be easy enough if you just take a look at our nsISound
impl for Mac.

Comment 8

[Joey Minta]

*** Bug 337702 has been marked as a duplicate of this bug. ***

Comment 10

[Sylvain Pasche]

Maybe this should be marked as a duplicate of bug 250186 which has a patch.

(bug 250186 should also move to Core/Widget as this is not mailnews specific)

Comment 11

[O. Atsushi (Torisugari)]

Attachment: Patch (another thread to play sound)

By the way, if I understand correctly, PR_LoadLibrary("libesd.so.0")...
is a kind of method to avoid linking to GPL things explicitly. The
copyright terms of the original nsSound are "(C) 2000", and Esound's
license is shifted from GPL to GPL/LGPL dual on 2003/09/03.

i.e. I can have mozilla include "esd.h" and link to "libesd.so", right?

Anyway, I don't like to call the sound's shutdown() on GTK's shutdown,
now that they are on different threads.

Comment 12

[O. Atsushi (Torisugari)]

Attachment: Patch (another thread to play sound, ESD/ALSA)

Comment 13

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Whats the status of this patch? Does this also work for alsa-based systems?

Comment 15

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Any updates?

Comment 16

[O. Atsushi (Torisugari)]

Oops, please wait a moment.

Before discussing my obsolete patch, probably we should
look into HTML5's audio implementation. We don't need
nsISound any more, or we can reuse the code.

At least we should adopt the same backend to make
the system requirement minimum.

Comment 17

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

I worked similar bug on Win32 at bug 498079. But perhaps, I should create an async nsISound methods caller class in XP level in the future...

And perhaps, the nsISound implementation shouldn't be created per each sound playing, so the wrapper service might fix many problems I think.

Comment 18

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

I filed bug 520417, it makes the sounds are played on another thread.

Comment 20

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

By the fix of bug 520417, the wav file playing is executed on another thread. However, canberra still plays the system event sound (e.g., popup a menu sound, open a dialog sound) on main thread. But we don't find any problems by this. If you have problems, please file a new bug and cc me. The canberra implementation was implemented in gecko 1.9.1, so, it's not a scope of this bug.

Comment 21

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

bug 520417 is backed out.

Comment 22

[Karl Tomlinson (:karlt)]

(In reply to comment #20)
> However, canberra still plays the system event sound (e.g., popup a menu sound,
> open a dialog sound) on main thread.

It looks like canberra uses a separate thread:
"This function will start playback in the background. It will not wait until playback completed."
http://0pointer.de/lennart/projects/libcanberra/gtkdoc/libcanberra-canberra.html#ca-context-play

Comment 23

[Wayne Mery (:wsmwk)]

this causes >"normal" severity issue in thunderbird, so setting severity to major.

sorry for the commentary

Comment 27

[Rolf Leggewie]

Is this still an issue?

Comment 28

[Launchpad]

Broomer68 added the following comment to Launchpad bug report 194573:

no, not anymore.


2013/8/19 Bugzilla-mozilla-org-rolf <[e-mail]>

> Is this still an issue?
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/194573
>
> Title:
>   sound in thunderbird
>
> Status in Mozilla Thunderbird Mail and News:
>   Confirmed
> Status in “thunderbird” package in Ubuntu:
>   Triaged
>
> Bug description:
>   Binary package hint: thunderbird
>
>   bug could also be in alsa or pulse or something
>
>   Instead of the new mail sound, thunderbird just freezes, screen
> darkens,and several minutes later unfreezes.
>   This is with ubuntu 8.4 with all latest upgrades.
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/thunderbird/+bug/194573/+subscriptions
>


-- 
http://launchpad.net/bugs/194573