Bug 34881 – [GStreamer] Notify about size changes

• Status: RESOLVED
• Resolution: FIXED
• Component: Media

[Sebastian Dröge (slomo)]

Hi,
the attached patch allows webkitwebsrc to notify about size changes until https://bugzilla.gnome.org/show_bug.cgi?id=609423 is fixed. Once that bug is fixed, the added code automatically isn't used anymore.

This is required for bug #30004.

[Sebastian Dröge (slomo)]

Created attachment 48629 [details]
0001-Notify-about-size-changes-in-the-WebKit-GStreamer-so.patch

[Sebastian Dröge (slomo)]

Created attachment 48640 [details]
0001-Notify-about-size-changes-in-the-WebKit-GStreamer-so.patch

Some minor updates, see the ChangeLog entry.

[Philippe Normand]

I successfully tested this patch with the one in Bug 30004 :)

[Gustavo Noronha (kov)]

Comment on attachment 48640 [details]
0001-Notify-about-size-changes-in-the-WebKit-GStreamer-so.patch

 2  *  Copyright (C) 2009,2010 Sebastian Dröge <[e-mail]>

Add a space here (between the , and 2010 =))

 249     // than 20% of data. This allows us to need some time
 250     // to unpause the SoupMessage and not starve the pipeline
 251     // until it is finally unpaused.

I don't get this comment. What does 'allows us to need some time' mean? I assume you want to say that you are raising the ammount of data that is buffered before the message is paused, so that you lower the risk of delays in unpausing starving the pipeline, right?

r- for the confusing comment, I believe the rest of the patch looks good, but we need to get this comment straight.

[Sebastian Dröge (slomo)]

Created attachment 48821 [details]
0001-Notify-about-size-changes-in-the-WebKit-GStreamer-so.patch

[WebKit Commit Bot]

Comment on attachment 48821 [details]
0001-Notify-about-size-changes-in-the-WebKit-GStreamer-so.patch

Clearing flags on attachment: 48821

Committed r54864: <http://trac.webkit.org/changeset/54864>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.