Akkana Peck Reporter
	Description • 22 years ago

I've been helping Aaron test bug 176602, but it turns out nsSound::Play doesn't
do anything on Redhat 7.3 (haven't tried other distros yet).  Sound on this box
works fine: I use sound in lots of other apps.

Chris, if you're the wrong person, sorry, please reassign.  Your name seemed to
be on other linux sound bugs so I took a guess.

I am getting to nsSound::OnStreamComplete, and if I un-ifdef the printf I see
f: 1 | c: 1 | sps: -18 | abps: -36 | ba: 2 | bps: 16 | rate: -36

The write at the end writes 1444 bytes (exactly the length of notfound.wav) to
the fd, returning 1444.  But no sound is heard, whether or not I have esd
running.  The wav file plays fine outside of mozilla using play (from
sox-12.17.3-4), or if I start esd and run esdplay on the wav file.

Other testing reports (in bug 176602) say that it doesn't work on stock Redhat 8
either, but it does work after installing the upgrade "arts-2.2.2-1.i386.rpm". 
No reports yet from any non-Redhat systems; I'll try pulling a debian tree one
of these days.

This is with gtk1.  gtk2 doesn't currently build for me, probably a configure
issue (it's not finding my gtk2 include files) though I have re-run configure
after setting MOZ_WIDGET_TOOLKIT=gtk2.  (Is there any way yet to have both gtk
and gtk2 widget libraries build automatically in the same tree?)

	Kyle Yuan
	Comment 1 • 22 years ago

Akkana, put those 2 options into your .mozconfig file, you can get a gtk2 build
easily :)

ac_add_options --enable-default-toolkit=gtk2
ac_add_options --disable-gtktest

my report for RedHat 8.0 is wrong, it just works for gtk2 build. The
arts-2.2.2-1.i386.rpm was installed for fixing the segment fault issue of xmms.

	sairuh (rarely reading bugmail)
	Comment 2 • 22 years ago

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

	Marc Loiselle
	Comment 3 • 22 years ago

See my patch for bug 177280.
GET_DWORD macro was incorrect and rate was not being set corrected.

	Marc Loiselle
	Comment 4 • 21 years ago

Fixes problem with playback of some wav files.	Implements playSystemSound as
well.

	Koike Kazuhiko
	Comment 5 • 21 years ago

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

	Marc Loiselle
	Comment 6 • 21 years ago

pavlov and blizzard, can I get a r/sr?

	Marc Loiselle
	Comment 7 • 21 years ago

blizzard, can I get a sr?

	Marc Loiselle
	Comment 8 • 21 years ago

Comment on attachment 124035 [details] [diff] [review]
Patch for nsSound.cpp v1

Can we get this in for 1.4?

	Christopher Blizzard (:blizzard) Assignee
	Comment 9 • 21 years ago

Comment on attachment 124035 [details] [diff] [review]
Patch for nsSound.cpp v1

sr=blizzard

	Christian :Biesinger (don't email me, ping me on IRC)
	Comment 10 • 21 years ago

I checked the patch here in, hope you don't mind:
Checking in widget/src/gtk/nsSound.cpp;
/cvsroot/mozilla/widget/src/gtk/nsSound.cpp,v  <--  nsSound.cpp
new revision: 1.35; previous revision: 1.34
done

leaving open to maybe get it on 1.4 branch.

Does Gtk2 need a similar patch?

	Christopher Blizzard (:blizzard) Assignee
	Comment 11 • 21 years ago

I imagine it does.

	Marc Loiselle
	Comment 13 • 21 years ago

Previous patch had a syntax error.  I don't have a gtk2 build environment.

	Christian :Biesinger (don't email me, ping me on IRC)
	Comment 14 • 21 years ago

The Gtk2 patch here works - I created an identical patch and tested it.

	Christopher Blizzard (:blizzard) Assignee
	Comment 15 • 21 years ago

Comment on attachment 125249 [details] [diff] [review]
Patch for GTK2 nsSound v1.1

sr=blizzard and I checked it in.  I changed a few style-related things, but
it's the same patch.

	Asa Dotzler [:asa]
	Comment 16 • 21 years ago

Comment on attachment 124035 [details] [diff] [review]
Patch for nsSound.cpp v1

moving approval request forward.

	Marc Loiselle
	Comment 17 • 21 years ago

Comment on attachment 124035 [details] [diff] [review]
Patch for nsSound.cpp v1

Can we get this in for 1.4.1?

	Asa Dotzler [:asa]
	Comment 18 • 21 years ago

mozilla1.4 shipped. unsetting blocking1.4 request.

	Asa Dotzler [:asa]
	Comment 19 • 21 years ago

blizzard, cbiesinger, can you guys get these patches into the 1.4 branch? We're
short on time. 

	Asa Dotzler [:asa]
	Comment 20 • 21 years ago

Comment on attachment 124035 [details] [diff] [review]
Patch for nsSound.cpp v1

a=asa (on behalf of drivers) for checkin to the 1.4 branch

	Christian :Biesinger (don't email me, ping me on IRC)
	Comment 21 • 21 years ago

OK, both gtk1 and gtk2 patch checked into the 1.4 branch:

Checking in gtk/nsSound.cpp;
/cvsroot/mozilla/widget/src/gtk/nsSound.cpp,v  <--  nsSound.cpp
new revision: 1.34.20.1; previous revision: 1.34
done
Checking in gtk2/nsSound.cpp;
/cvsroot/mozilla/widget/src/gtk2/nsSound.cpp,v  <--  nsSound.cpp
new revision: 1.5.4.1; previous revision: 1.5
done

This means this bug is fixed, I would say. marking so.