Bug 235087 - [Cocoa] HLS stream currentTime sometimes jumps backwards
Summary: [Cocoa] HLS stream currentTime sometimes jumps backwards
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Eric Carlson
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-01-11 15:54 PST by Eric Carlson
Modified: 2022-03-01 14:38 PST (History)
13 users (show)

See Also:

Attachments
Patch (193.57 KB, patch)
2022-01-11 16:11 PST, Eric Carlson 		no flags Details | Formatted Diff | Diff
Patch (193.60 KB, patch)
2022-01-11 16:57 PST, Eric Carlson 		no flags Details | Formatted Diff | Diff
Patch (194.70 KB, patch)
2022-01-12 10:03 PST, Eric Carlson 		no flags Details | Formatted Diff | Diff
Patch (194.58 KB, patch)
2022-01-12 11:08 PST, Eric Carlson 		no flags Details | Formatted Diff | Diff
Patch (192.92 KB, patch)
2022-01-12 12:13 PST, Eric Carlson 		no flags Details | Formatted Diff | Diff
Patch (192.92 KB, patch)
2022-01-12 13:21 PST, Eric Carlson 		no flags Details | Formatted Diff | Diff
Patch (192.45 KB, patch)
2022-01-12 15:34 PST, Eric Carlson 		no flags Details | Formatted Diff | Diff
Patch for landing (192.39 KB, patch)
2022-02-28 14:32 PST, Eric Carlson 		no flags Details | Formatted Diff | Diff
Patch for landing (192.39 KB, patch)
2022-03-01 12:16 PST, Eric Carlson 		ews-feeder: commit-queue-
Details | Formatted Diff | Diff
Show Obsolete (8) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Carlson 2022-01-11 15:54:58 PST 
An HLS stream's video.currentTime sometimes jumps backwards briefly.
Comment 1 Eric Carlson 2022-01-11 15:55:12 PST 
rdar://83831093
Comment 2 Eric Carlson 2022-01-11 16:11:37 PST 
Created attachment 448886 [details]
Patch
Comment 3 Eric Carlson 2022-01-11 16:57:02 PST 
Created attachment 448893 [details]
Patch
Comment 4 Eric Carlson 2022-01-12 10:03:27 PST 
Created attachment 448960 [details]
Patch
Comment 5 Eric Carlson 2022-01-12 11:08:25 PST 
Created attachment 448969 [details]
Patch
Comment 6 Eric Carlson 2022-01-12 12:13:38 PST 
Created attachment 448984 [details]
Patch
Comment 7 Eric Carlson 2022-01-12 13:21:27 PST 
Created attachment 448989 [details]
Patch
Comment 8 Eric Carlson 2022-01-12 15:34:33 PST 
Created attachment 449004 [details]
Patch
Comment 9 Robert Bryer 2022-02-09 05:34:50 PST 
Hi Eric, in the current Safari TP 139, I'm seeing currentTime jumping back to 0 when seeking for live streams like so:

timeupdate – 7193.6532422809705
set currentTime – 7173.88901530305
seeking – 7173.889107168126
timeupdate – 7173.889107168126
timeupdate – 0
seeked – 0
timeupdate – 0
timeupdate – 7173.988310672026 (correct timeupdates continue)
timeupdate – 7174.13682652897


Is this related to this issue, or should I fork it as a different bug?
Comment 10 Eric Carlson 2022-02-28 14:32:15 PST 
Created attachment 453430 [details]
Patch for landing
Comment 11 Eric Carlson 2022-03-01 12:16:43 PST 
Created attachment 453520 [details]
Patch for landing
Comment 12 EWS 2022-03-01 14:29:15 PST 
Committed r290676 (247947@main): <https://commits.webkit.org/247947@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 453520 [details].
Comment 13 Eric Carlson 2022-03-01 14:38:31 PST 
The Windows test failures:

- js/function-apply-many-args.html
- svg/animations/animate-keytimes-crash.html
- svg/as-image/img-zoom-svg-stylesheet.html


are unrelated.