Bug 248585 - currentTime progress even when playback has stalled
Summary: currentTime progress even when playback has stalled
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: Other
Hardware: Mac (Apple Silicon) Unspecified
: P2 Normal
Assignee: Jean-Yves Avenard [:jya]
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-11-30 22:22 PST by Jean-Yves Avenard [:jya]
Modified: 2022-12-06 02:23 PST (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jean-Yves Avenard [:jya] 2022-11-30 22:22:57 PST
This problem got exposed in bug 236722.

In bug 236722, playback doesn't actually start unless the entire media has been fetched.
However, we can see that the progress bar is moving forward as if it did.

It shouldn't. Somehow we are not detecting that the underlying playback framework has stalled and we fail to notify the content process which in turn will incorrectly estimate the currentTime.
Comment 1 Radar WebKit Bug Importer 2022-11-30 22:23:25 PST
<rdar://problem/102846199>
Comment 2 Jean-Yves Avenard [:jya] 2022-11-30 22:29:26 PST
actually, this has already been lodged. It got fixed but changes were reverted

*** This bug has been marked as a duplicate of bug 230210 ***
Comment 3 Jean-Yves Avenard [:jya] 2022-12-02 15:13:40 PST
issue in bug 230210 can't be reproduced so let's start with a new bug
Comment 4 Jean-Yves Avenard [:jya] 2022-12-02 15:47:56 PST
Pull request: https://github.com/WebKit/WebKit/pull/7103
Comment 5 Jean-Yves Avenard [:jya] 2022-12-02 15:51:10 PST
problem can be seen when attempting to open the URL https://mediaserver.avenard.org/test/89041245.py?name=foo&stallOffset=100000&stallDuration=20&chunkSize=1024 , playback will not start until 20s after the first frame has been displayed
Comment 6 EWS 2022-12-06 02:23:45 PST
Committed 257403@main (384fb5b098b8): <https://commits.webkit.org/257403@main>

Reviewed commits have been landed. Closing PR #7103 and removing active labels.