1253184 - [Meta] Simplify the seeking pipeline of MDSM

Status: NEW

(Core :: Audio/Video: Playback, defect, P3)

Description

[JW Wang [:jwwang] [:jw_wang]]

Tracking bugs to simplify/optimize the code of seeking pipeline in MDSM.

Comment 1

[Chris Pearce [:cpearce (Not reading bugmail)]]

Note: we added telementry to track how often HTMLMediaElement.fastSeek() is used:

https://telemetry.mozilla.org/new-pipeline/dist.html#!cumulative=0&end_date=2016-02-29&keys=__none__!__none__!__none__!__none__&max_channel_version=nightly%252F47&measure=VIDEO_FASTSEEK_USED&min_channel_version=null&product=Firefox&sanitize=1&sort_keys=submissions&start_date=2016-01-25&table=0&trim=1&use_submission_date=0

We haven't had the probe in long enough to conclude that no results means fastSeek() isn't being used, but if we find it's not being used, I think we should seriously consider removing fastSeek(), since in general it just confuses people.

Comment 2

[JW Wang [:jwwang] [:jw_wang]]

(In reply to Chris Pearce (:cpearce) from comment #1)
> Note: we added telementry to track how often HTMLMediaElement.fastSeek() is
> used:
> 
> https://telemetry.mozilla.org/new-pipeline/dist.html#!
> cumulative=0&end_date=2016-02-29&keys=__none__!__none__!__none__!
> __none__&max_channel_version=nightly%252F47&measure=VIDEO_FASTSEEK_USED&min_c
> hannel_version=null&product=Firefox&sanitize=1&sort_keys=submissions&start_da
> te=2016-01-25&table=0&trim=1&use_submission_date=0
> 
> We haven't had the probe in long enough to conclude that no results means
> fastSeek() isn't being used, but if we find it's not being used, I think we
> should seriously consider removing fastSeek(), since in general it just
> confuses people.

We can run accurate seek underneath HTMLMediaElement.fastSeek() and remove the code about fastSeek without breaking the web. Should there be such a demand in the future and we then implement it again. It saves us efforts for maintaining features not used by user at all.

Comment 3

[Anthony Jones (:ajones, :kentuckyfriedtakahe, :k17e)]

Mass change P2 -> P3