1347829 - [webvtt] fix webvtt-file-format-parsing/webvtt-cue-text-parsing-rules/tests/timestamps.html

Status: RESOLVED FIXED

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

Comment 1

[Benjamin Chen [:bechen]]

Attachment: Bug 1347829 - If the timestamp "<00:00.005" missing the last ">" character, we still parse it.

Review commit: https://reviewboard.mozilla.org/r/121254/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/121254/

Comment 2

[Benjamin Chen [:bechen]]

https://github.com/w3c/webvtt/issues/342

Comment 3

[Alastor Wu [:alwu]]

Comment on attachment 8848361 [details]
Bug 1347829 - If the timestamp "<00:00.005" missing the last ">" character, we still parse it.

https://reviewboard.mozilla.org/r/121254/#review123794

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8db1c2fd27c9
If the timestamp "<00:00.005" missing the last ">" character, we still parse it. r=alwu

Comment 5

[Iris Hsiao [:ihsiao]]

backed this out for web platform tests failures like https://treeherder.mozilla.org/logviewer.html#?job_id=84969899&repo=autoland&lineNumber=3141

Comment 6

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cb4e80abb488
Backed out changeset 8db1c2fd27c9 for web platform tests failures

Comment 7

[Benjamin Chen [:bechen]]

Need to rebase because bug 1344627 move the files.

Comment 8

[Benjamin Chen [:bechen]]

Comment on attachment 8848361 [details]
Bug 1347829 - If the timestamp "<00:00.005" missing the last ">" character, we still parse it.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/121254/diff/1-2/

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7ba1abadab3f
If the timestamp "<00:00.005" missing the last ">" character, we still parse it. r=alwu

Comment 10

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/7ba1abadab3f