775257 - Gstreamer backend on Meego N9 produce extra noise in audio playback

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

[Oleg Romashin (:romaxa)]

Attachment: Fix gstreamer audio sink for N9 / Meego

I've compiled Mobile FF for N9 with gstreamer backend and found there are weird noise for audio stream.

After some experiments I found that caps= and width= need to be changed in order to make it works on N9 pulseaudio.
Final fix should looks like this:
+#ifdef MOZ_PLATFORM_MAEMO
+        "appsink name=audiosink sync=true caps=audio/x-raw-int,"
+        "channels={1,2},rate=48000,width=16,endianness=1234", TRUE, NULL);
+#else
         "appsink name=audiosink sync=true caps=audio/x-raw-float,"
         "channels={1,2},rate=44100,width=32,endianness=1234", TRUE, NULL);
+#endif

Also I changed rate to 48kHz as rillian suggested, it works just fine

Comment 1

[Ralph Giles (:rillian)]

Comment on attachment 643548 [details] [diff] [review]
Fix gstreamer audio sink for N9 / Meego

Review of attachment 643548 [details] [diff] [review]:
-----------------------------------------------------------------

r+ with the proper #ifdef check.

Alessandro, any comments?

::: content/media/gstreamer/nsGStreamerReader.cpp
@@ +129,5 @@
>        G_CALLBACK(&nsGStreamerReader::EventProbeCb), this);
>    gst_object_unref(sinkpad);
>  
>    mAudioSink = gst_parse_bin_from_description("capsfilter name=filter ! "
> +#ifdef MOZ_PLATFORM_MAEMO

Please switch the order and use

#ifdef MOZ_SAMPLE_TYPE_FLOAT32

instead. That will be less brittle than depending on the MAEMO build in particular.

Comment 2

[Alessandro Decina]

Comment on attachment 643548 [details] [diff] [review]
Fix gstreamer audio sink for N9 / Meego

Yeah looks good

Comment 3

[Oleg Romashin (:romaxa)]

Attachment: Fix gstreamer audio sink for N9 / Meego

Comment 4

[Ralph Giles (:rillian)]

Comment on attachment 643634 [details] [diff] [review]
Fix gstreamer audio sink for N9 / Meego

Looks good, thanks. Have you tested this version on both maemo and desktop builds with --enable-gstreamer?

There's no point pushing this to try; it doesn't affect code we normally build.

Comment 5

[Oleg Romashin (:romaxa)]

I tested on maemo, but not on desktop

Comment 6

[Oleg Romashin (:romaxa)]

https://hg.mozilla.org/mozilla-central/rev/6865bdd5ca5a