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)
Tracking
(Not tracked)
VERIFIED
FIXED
seamonkey1.0beta
People
(Reporter: kraftycats, Assigned: csthomas)
Details
(Keywords: fixed1.8, regression)
Attachments
(1 file, 2 obsolete files)
4.46 KB,
patch
|
Biesinger
:
review+
csthomas
:
superreview+
asa
:
approval1.8rc1+
|
Details | Diff | Splinter Review |
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).
Reporter | ||
Comment 1•19 years ago
|
||
(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.
Comment 2•19 years ago
|
||
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/
Reporter | ||
Comment 3•19 years ago
|
||
(In reply to comment #2) Bug persists in SeaMonkey 1.0a, which was installed with a new profile on Win 98SE OS.
Comment 4•19 years ago
|
||
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
Comment 5•19 years ago
|
||
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
Assignee | ||
Comment 7•19 years ago
|
||
Switch back to GetService.
Assignee: download-manager → cst
Status: NEW → ASSIGNED
Attachment #200393 -
Flags: superreview?(bzbarsky)
Attachment #200393 -
Flags: review?(timeless)
Comment 8•19 years ago
|
||
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+
Assignee | ||
Comment 9•19 years ago
|
||
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)
Assignee | ||
Comment 10•19 years ago
|
||
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)
Assignee | ||
Updated•19 years ago
|
Whiteboard: [cst: r? sr?]
Target Milestone: --- → Seamonkey1.0beta
Assignee | ||
Updated•19 years ago
|
Attachment #200404 -
Flags: review?(db48x) → review?(cbiesinger)
Comment 11•19 years ago
|
||
Comment on attachment 200404 [details] [diff] [review] patch please add a newline before: + nsCOMPtr<nsISound> mSoundInterface;
Attachment #200404 -
Flags: review?(cbiesinger) → review+
Comment 12•19 years ago
|
||
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+
Assignee | ||
Comment 13•19 years ago
|
||
carrying forward sr=neil
Attachment #200404 -
Attachment is obsolete: true
Attachment #200477 -
Flags: superreview+
Attachment #200477 -
Flags: review?(cbiesinger)
Updated•19 years ago
|
Attachment #200477 -
Flags: review?(cbiesinger) → review+
Assignee | ||
Updated•19 years ago
|
Whiteboard: [cst: r? sr?] → [cst: landme]
Assignee | ||
Comment 14•19 years ago
|
||
Comment on attachment 200477 [details] [diff] [review] patch v2 SeaMonkey-only patch.
Attachment #200477 -
Flags: approval1.8rc1?
Assignee | ||
Updated•19 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Whiteboard: [cst: landme] → [cst: fixed-trunk a?]
Updated•19 years ago
|
Attachment #200477 -
Flags: approval1.8rc1? → approval1.8rc1+
You need to log in
before you can comment on or make changes to this bug.
Description
•