Closed
Bug 241482
Opened 21 years ago
Closed 21 years ago
Editor pops up if multimedia isn't installed
Categories
(Core :: XUL, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: mkaply, Assigned: mkaply)
Details
(Keywords: fixed1.7)
Attachments
(2 files)
1.66 KB,
patch
|
jhpedemonte
:
review+
mkaply
:
superreview+
mkaply
:
approval1.7+
|
Details | Diff | Splinter Review |
473 bytes,
patch
|
Details | Diff | Splinter Review |
Really annoying bug.
If you don't have multimedia installed, we pass WAV files to the WPS and they
open up in E.EXE.
This wouldn't be so bad except that a WAV file is played EVERY TIME typeahead
fine fails.
Need to fix this for 1.7.
Assignee | ||
Comment 1•21 years ago
|
||
There are two cases here, system sound and when someone asks us to play a WAV.
I decided to beep in all system sound cases, and just return failure if someone
tries to play a WAV.
Assignee | ||
Updated•21 years ago
|
Attachment #146870 -
Flags: review?(pedemont)
Updated•21 years ago
|
Attachment #146870 -
Flags: review?(pedemont) → review+
Assignee | ||
Comment 2•21 years ago
|
||
Comment on attachment 146870 [details] [diff] [review]
Check for MMPM install
sr=blizzard (platform specific)
a=mkaply (OS/2 only)
Attachment #146870 -
Flags: superreview+
Attachment #146870 -
Flags: approval1.7+
Assignee | ||
Comment 3•21 years ago
|
||
Fix checked in.
Comment 4•21 years ago
|
||
If I understand the code correctly, the MMPM module never really gets used, so
it would be best, to unload it at the end of this method, after checking that
it is there.
As the code I talk about came from this bug, I attach it here rather than
opening a new bug.
(I wonder if this is related to something I have experienced a few times over
the last weeks: After starting to debug Mozilla with IDebug I afterwards cannot
run any programs that use MMOS2. Binary programs do not give any error, but
play.cmd complains that it cannot access/load a particular mci routine. I had
to reboot to clear this problem.)
You need to log in
before you can comment on or make changes to this bug.
Description
•