199201 - sound settings blocks stalked messages if WAV sound file is set

Status: RESOLVED FIXED

(Other Applications :: ChatZilla, defect)

Description

[bioram]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.3b) Gecko/20030210
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.3b) Gecko/20030210

Set any WAV as a notification sound for private message tab or stalked message.
No private messages or stalked messages are shown.

Reproducible: Always

Steps to Reproduce:



Expected Results:  
WAV should be played and all messages should be shown.

Comment 1

[James Nord]

Also ChatZilla 0.8.23 [Mozilla rv:1.4a/20030318] WinXP

(although I get the private messages but in the main channel window) prefs are
set to display private messages in new window

Removing the custom sound and things work as expected.

Comment 2

[Samuel Sieb]

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

Comment 3

[Daniel Bachran]

Might be caused by the implementation of bug 64462 (see
http://bugzilla.mozilla.org/show_bug.cgi?id=64462#c154) which fits in nicely
with the timeframe of somwhere between mozilla 1.1 and mozilla 1.2beta.

Also of interest is bug 199201, which might cover the same problem.

Comment 4

[Koike Kazuhiko]

Attachment: patch v1

The cause is nsSound::Play() doesn't work. See bug 179138.

This patch handles the exception.

Comment 5

[Daniel Bachran]

I have just tested patch v1 and it works fine for me. I.e.: on /testdisplay I do
see all lines of output now, regardless of the stalk sound setting (beep or wave
sound file).

But I still don't hear the wav sound file being played. As just discussed with
Koike this bug here seems to be about the exception handling. Perhaps the
playing problem might be fixed with bug 179138.

Comment 6

[Koike Kazuhiko]

This bug will be fixed with bug 207436.

Comment 7

[James Ross]

I don't think this is Linux specific... I have the (seemingly) same problem on
Windows 2000.

Comment 8

[Robert Ginda]

Comment on attachment 119505 [details] [diff] [review]
patch v1

I moved the try/catch inside of playSound.

Comment 9

[Robert Ginda]

marking fixed, 0.8.34 checked in.