Bug 93143 - Negative timestamps for TextTrackCue
Summary: Negative timestamps for TextTrackCue
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: BrowserCompat, InRadar, WPTImpact
Depends on:
Blocks: 128412 85035
  Show dependency treegraph
 
Reported: 2012-08-03 10:54 PDT by Anna Cavender
Modified: 2023-08-18 09:52 PDT (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Anna Cavender 2012-08-03 10:54:46 PDT
It is not clear what the expected behavior is when negative startTime and endTime values are specified for TextTrackCue objects.  This is currently being discussed at the WHATWG:
https://www.w3.org/Bugs/Public/show_bug.cgi?id=18480

Once a decision has been made, this bug will implement the solution.
Comment 1 Eric Carlson 2012-09-11 14:10:58 PDT
http://trac.webkit.org/changeset/124645 changed our behavior, but leaving this open pending resolution of the w3c bug.
Comment 2 Eric Carlson 2014-02-07 16:46:16 PST
The changes made for r124645 are incorrect, the spec now says:

The text track cue start time and text track cue end time can be negative. (The current playback position can never be negative, though, so cues entirely before time zero cannot be active.)
Comment 3 Ahmad Saleem 2023-07-28 00:56:39 PDT
Blink Commit (for Merge): https://chromium-review.googlesource.com/c/chromium/src/+/863270

^ Locally doing this make us pass at least two WPT test. Can give it a go?

@Eric - any recommendation?
Comment 4 Eric Carlson 2023-07-28 08:46:21 PDT
(In reply to Ahmad Saleem from comment #3)
> Blink Commit (for Merge):
> https://chromium-review.googlesource.com/c/chromium/src/+/863270
> 
> ^ Locally doing this make us pass at least two WPT test. Can give it a go?
> 
> @Eric - any recommendation?

Yes, I think it would be good to make this change.
Comment 5 Radar WebKit Bug Importer 2023-07-28 08:46:32 PDT
<rdar://problem/113035924>
Comment 6 EWS 2023-08-18 09:52:50 PDT
Committed 267042@main (fb0116deed64): <https://commits.webkit.org/267042@main>

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