Bug 33662 - [GTK] RTP/RTSP streams playback support
Summary: [GTK] RTP/RTSP streams playback support
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
Depends on:
Reported: 2010-01-14 02:18 PST by Philippe Normand
Modified: 2010-02-19 00:31 PST (History)
6 users (show)

See Also:

proposed patch (6.47 KB, patch)
2010-02-17 03:48 PST, Philippe Normand
eric: review+
Details | Formatted Diff | Diff
updated patch (4.19 KB, patch)
2010-02-18 06:48 PST, Philippe Normand
gustavo: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Philippe Normand 2010-01-14 02:18:53 PST
Our player should be able to play rtp/rtsp streams like this one:


I think the issue is that the pipeline changes state to NOPREROLL (because of the live source) and this code path doesn't seem well supported at the moment.
Comment 1 Philippe Normand 2010-02-17 03:48:18 PST
Created attachment 48884 [details]
proposed patch
Comment 2 Sebastian Dröge (slomo) 2010-02-17 08:35:59 PST
Looks good I guess
Comment 3 Eric Seidel (no email) 2010-02-17 15:19:10 PST
Comment on attachment 48884 [details]
proposed patch

OK.  Looks sane.
Comment 4 Philippe Normand 2010-02-18 01:24:13 PST
The new test takes about 2.5 seconds to complete here. I don't think it is very good and Xan thinks the same. 

Gustavo, Eric, would you have an idea on how to improve the speed of the test?
Comment 5 Gustavo Noronha (kov) 2010-02-18 06:26:04 PST
Comment on attachment 48884 [details]
proposed patch

I'd turn this into a manual test, I think, given it needs to get to the interwebs, which I'd think is a no-no for layout tests. Also, the URI changing is quite likely, I'd guess?
Comment 6 Philippe Normand 2010-02-18 06:48:30 PST
Created attachment 49010 [details]
updated patch

With a manual test as suggested by Gustavo. The same URL is btw used
in media/unsupported-rtsp.html which we skip.
Comment 7 Gustavo Noronha (kov) 2010-02-18 11:33:21 PST
Comment on attachment 49010 [details]
updated patch

Manual test looks good,thanks!
Comment 8 Philippe Normand 2010-02-19 00:31:23 PST
Landed in r55005. Thanks :)