GNOME Bugzilla – Bug 339000
totem / qtdemux fails to redirect to relative links in mov files
Last modified: 2006-05-14 17:52:45 UTC
if you open a remote quicktime file with a link in totem, i.e.: totem http://movies.apple.com/movies/sony_pictures/silent_hill/silent_hill_480p.mov totem will fail to follow the link to silent_hill_h480p.mov not sure if this is doe to totem failing to see that this is a relative link or qtdemux doing something wrong
Probably best to fix this in totem rather than just the GStreamer backend => re-assigning.
Created attachment 63862 [details] [review] propsed patch
xine-lib will give us non-relative redirections, which is why it worked. So instead, let's move this into the backends. 2006-04-24 Bastien Nocera <hadess@hadess.net> * src/backend/bacon-video-widget-gst-0.10.c: (bvw_handle_element_message): * src/backend/bacon-video-widget-gst.c: (bacon_video_widget_signal_idler): * src/backend/video-utils.c: (totem_resolve_relative_link): * src/backend/video-utils.h: * src/totem-mozilla-viewer.c: (on_got_redirect): * src/totem.c: (on_got_redirect): Make the GStreamer backends resolve relative redirects, as the xine-lib backend does (Closes: #339000) * src/plparse/totem-pl-parser.c: (totem_pl_parser_write_m3u): Save the full path when we can't get a relative path when saving m3u playlists (Closes: #338974)
It seems this isn't quite fixed. For the Firefox plugin, it's using fd://0 for the input, and that breaks the relative->absolute url logic. It ends up trying to redirect to fd://new-file-name. Also, it seems that for Xine it only posts absolute urls when the input mrl is http:// or rtsp:// - so it breaks in the same way. On the other hand with 1.4.0, this worked for me because it used the 'orig_filename' to perform the redirection at the mozilla-viewer.c level.
Oh btw, for examples try anything currently on trailers.apple.com - they all have relative urls in the mov files these days: http://trailers.apple.com/trailers/disney/piratesofcaribbean2/large.html (http://movies.apple.com/movies/disney/piratesofthecaribbean2/piratesofthecaribbean2-tsr3_h.480.mov )
Created attachment 65309 [details] [review] fix redirection again This patch fixes redirection for relative paths in the firefox viewer, at least when using the GStreamer backend.
I also fixed the 0.8 backend in gnome-2-14 2006-05-14 Bastien Nocera <hadess@hadess.net> * NEWS: upd * src/backend/bacon-video-widget-gst-0.10.c: (bvw_handle_element_message): * src/backend/bacon-video-widget-gst.c: (bacon_video_widget_signal_idler): * src/backend/video-utils.c: (totem_resolve_relative_link): * src/totem-mozilla-viewer.c: (on_got_redirect): * src/totem.c: (on_got_redirect): Patch from Tim-Philipp Müller <tim at centricular dot net> to handle relative links in the front-end, as it knows the canonical mrl for the stream, and the backend doesn't, fixes Apple trailers playback in the Mozilla plugin (Closes: #339000)
Thank you for backporting this to the 0.8 backend in the 2.14 branch as well, I honestly completely forget about it (out of sight, out of mind I guess).