[Bug 737293] dashdemux: fix gst_mpdparser_get_rep_idx_with_max_bandwidth return

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=737293
  GStreamer | gst-plugins-bad | 1.4.1

--- Comment #7 from Brendan Long <self at brendanlong.com> 2014-10-20 11:17:43 UTC ---
As a proof of concept, I tried adding a GTimer to souphttpsrc to time requests
and expose it as a property ("request-time"). A lot of test DASH files have
tiny MPDs (~4 KB) and in those cases we always get useless values because the
download time is dominated by latency, but once I tried with a real file I got
reasonable values.

With
http://www-itec.uni-klu.ac.at/ftp/datasets/mmsys12/Valkaama/MPDs/Valkaama_1s_act_isoffmain_DIS_23009_1_v_2_1c2_2011_08_30.mpd
(1 hour 33 minutes long according to
http://www-itec.uni-klu.ac.at/dash/?page_id=207): 

0:00:04.778981598  7349       0xac5590 WARN             souphttpsrc
gstsouphttpsrc.c:1286:gst_soup_http_src_got_body_cb:<source> Downloaded 4165938
bytes in 4.601469 seconds, download speed is 7242796 bits/s

With
http://download.tsi.telecom-paristech.fr/gpac/DASH_CONFORMANCE/TelecomParisTech/mp4-main-multi/mp4-main-multi-mpd-AV-BS.mpd
(listed in dashdemux's README):

0:00:02.626909926  7416      0x24f3590 WARN             souphttpsrc
gstsouphttpsrc.c:1286:gst_soup_http_src_got_body_cb:<source> Downloaded 24281
bytes in 2.458095 seconds, download speed is 79023 bits/s

What do you think of this approach?

If we do this, I assume we would want to send this as some sort of event
instead of a property?

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.