Closed Bug 294056 Opened 19 years ago Closed 19 years ago

User-specified sound, when activated in Preferences, fails to play after file download completes

Categories

(SeaMonkey :: Download & File Handling, defect)

x86
Windows 98
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
seamonkey1.0beta

People

(Reporter: kraftycats, Assigned: csthomas)

Details

(Keywords: fixed1.8, regression)

Attachments

(1 file, 2 obsolete files)

User-Agent:       Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8b2) Gecko/20050510 Mnenhy/0.7.2.0
Build Identifier: Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8b2) Gecko/20050510 Mnenhy/0.7.2.0

Under Preferences | Downloads | "When a download completes," a user can choose
"Play a sound" or "Show an alert," both checkbox options.  If the former is
chosen, a sound file (usually .wav) needs (?) to be specified.  (AFAIK, there is
no "default" sound for this option:  it must be user-specified if "Play a sound"
is chosen.)

This option last worked as expected, in conjunction with "Show an alert" or by
itself, as of 1.8a5.  Starting with the 1.8bx branch, the sound no longer plays.
 The "Show an alert" option does not appear to be affected.

This behavior occurs with a new or used profile.

Reproducible: Always

Steps to Reproduce:
1. Under Preferences | Downloads | "When a download completes," check the box
next to "Play a sound."  Browse to the location of the sound file you wish to
use and choose it.  Test the sound file by clicking the Preview button; sound
plays as expected

2. Download a file (any file) using the browser.

3. Sound should play upon completion of download.




Using Win 98 SE (up to date on all patches), Mostly Crystal Small theme (a
constant; also used with 1.8a5; theme does not appear to affect behavior),
Mnenhy 0.7.2.

Two Notations:

1) The sound notification for new mail remains unaffected.

2) The specified sound file for download completion is listed in the option box
with the syntax:  file:///[drive]/[folder]/[file].  This is the same syntax as
used with 1.8a6, and is also the same for the new mail sound notification (as
found via about:config).
(In reply to comment #0)

Addendum: I was in error.  With the 1.8bx branch, the mail.biff.play_sound.url
is listed as a string under about:config, but is blank in my 5/10/05 nightly. 
Only the pref browser.download.finished_sound_url has the sound file listed as a
string with the aforementioned syntax.
This is an automated message, with ID "auto-resolve01".

This bug has had no comments for a long time. Statistically, we have found that
bug reports that have not been confirmed by a second user after three months are
highly unlikely to be the source of a fix to the code.

While your input is very important to us, our resources are limited and so we
are asking for your help in focussing our efforts. If you can still reproduce
this problem in the latest version of the product (see below for how to obtain a
copy) or, for feature requests, if it's not present in the latest version and
you still believe we should implement it, please visit the URL of this bug
(given at the top of this mail) and add a comment to that effect, giving more
reproduction information if you have it.

If it is not a problem any longer, you need take no action. If this bug is not
changed in any way in the next two weeks, it will be automatically resolved.
Thank you for your help in this matter.

The latest beta releases can be obtained from:
Firefox:     http://www.mozilla.org/projects/firefox/
Thunderbird: http://www.mozilla.org/products/thunderbird/releases/1.5beta1.html
Seamonkey:   http://www.mozilla.org/projects/seamonkey/
(In reply to comment #2)
Bug persists in SeaMonkey 1.0a, which was installed with a new profile on Win
98SE OS.
Confirming with SeaMonkey 1.0 branch build 2005092310 on Windows XP SP2. The
Preview button to the right of the Browse button where you set the sound in the
Preferences also doesn't work.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Sorry... the Preview button *does* work, but the sound still does not play after
the download is complete.
I'll give a helping hand. :)

Here's the regression window:
20050113 hours: 08 (build ID), 10 (directory listing)
20050114 hours: 09 (build ID), 10 (directory listing)
Keywords: regression
Switch back to GetService.
Assignee: download-manager → cst
Status: NEW → ASSIGNED
Attachment #200393 - Flags: superreview?(bzbarsky)
Attachment #200393 - Flags: review?(timeless)
Comment on attachment 200393 [details] [diff] [review]
patch (backs out fix from bug 277852)

File a followup to make this work right, please.
Attachment #200393 - Flags: superreview?(bzbarsky) → superreview+
Comment on attachment 200393 [details] [diff] [review]
patch (backs out fix from bug 277852)

http://lxr.mozilla.org/seamonkey/search?string=@mozilla.org/sound

I'm confused.
Attachment #200393 - Attachment is obsolete: true
Attachment #200393 - Flags: superreview+
Attachment #200393 - Flags: review?(timeless)
Attached patch patch (obsolete) — Splinter Review
The download manager lives longer than anything in the nsDownload, so the sound
gets to finish now.
Attachment #200404 - Flags: superreview?(neil.parkwaycc.co.uk)
Attachment #200404 - Flags: review?(db48x)
Whiteboard: [cst: r? sr?]
Target Milestone: --- → Seamonkey1.0beta
Attachment #200404 - Flags: review?(db48x) → review?(cbiesinger)
Comment on attachment 200404 [details] [diff] [review]
patch

please add a newline before:

+  nsCOMPtr<nsISound> mSoundInterface;
Attachment #200404 - Flags: review?(cbiesinger) → review+
Comment on attachment 200404 [details] [diff] [review]
patch

> nsDownloadManager::nsDownloadManager() : mBatches(0)
> {
>+  mSoundInterface = do_CreateInstance("@mozilla.org/sound;1");
> }
I'd prefer the use of the initializer list for initializers. Either way though
you might never use it...

>+#include "nsISound.h"
Presumably you don't need to include this in the .cpp any more?
Attachment #200404 - Flags: superreview?(neil.parkwaycc.co.uk) → superreview+
Attached patch patch v2Splinter Review
carrying forward sr=neil
Attachment #200404 - Attachment is obsolete: true
Attachment #200477 - Flags: superreview+
Attachment #200477 - Flags: review?(cbiesinger)
Attachment #200477 - Flags: review?(cbiesinger) → review+
Whiteboard: [cst: r? sr?] → [cst: landme]
Comment on attachment 200477 [details] [diff] [review]
patch v2

SeaMonkey-only patch.
Attachment #200477 - Flags: approval1.8rc1?
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Whiteboard: [cst: landme] → [cst: fixed-trunk a?]
Attachment #200477 - Flags: approval1.8rc1? → approval1.8rc1+
Keywords: fixed1.8
Whiteboard: [cst: fixed-trunk a?]
Verified fixed in 2005102613.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: