1557185 - [webvtt] enable wpt 'too_many_cues.html'.

Status: RESOLVED FIXED

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

Description

[Alastor Wu [:alwu]]

There are several things we have to do in order to enable this test,

1. do not use <br> in reference files
2. eliminate text's length in order to avoid line break, which is not somthing we would like to test in this test
3. add line number for each cue in order to easily know how many cue would be displayed in the screen

Comment 1

[Alastor Wu [:alwu]]

We might enable too_many_cues_wrapped.html in this bug as well.

Comment 2

[Alastor Wu [:alwu]]

Attachment: Bug 1557185 - part1 : use recommended font size for Ahem.

As the line box's height is computed by the font size and the line-height (default is 1.2x), so the actual height is around 11px (9*1.2).

The video's height is 180px, which means we can put at most 16 lines within video's rendering area.

Therefore, for too_many_cues.vtt, triming the showing cues number to 16. For too_many_cues_wrapped.vtt, triming the total cues number to 16.

Comment 3

[Alastor Wu [:alwu]]

Attachment: Bug 1557185 - part2 : correct CSS style in reference files in order to show cue correctly.

<br> has 1 app unit width, which would push text 1 app unit left and cause the positioning difference.

We should use <div> to enforce them break line without having extra space in the line box.

Comment 4

[Alastor Wu [:alwu]]

Attachment: Bug 1557185 - part3 : enable wpt 'too_many_cues.html' and 'too_many_cues_wrapped.html'.

Comment 5

[BugBot [:suhaib / :marco/ :calixte]]

Bugbug thinks this bug is a task, but please change it back in case of error.

Comment 6

[Alastor Wu [:alwu]]

too_many_cues.vtt would fail on Windows because of different line box height. I found that the line box's height varies by different platforms and is also depending on different system languages, because those setting might have different default font.

While I was running the test on different situation, I saw different line box's height. On my local Windows 10, the height was 10.6; on Windows on tty server, the height was 11.6; on my local Linux with English as default language, the height was 11.08; setting language to Chinese, the height was 13.

That makes me hard to know how many line I should show in the reference file....

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/69150f666319
part1 : use recommended font size for Ahem. r=heycam
https://hg.mozilla.org/integration/autoland/rev/6ead08887181
part2 : correct CSS style in reference files in order to show cue correctly. r=heycam
https://hg.mozilla.org/integration/autoland/rev/5607d2f3ef7d
part3 : enable wpt 'too_many_cues.html' and 'too_many_cues_wrapped.html'. r=heycam

Comment 8

[Andreea Pavel [:apavel]]

Backed out part3 for causing too_many_cues_wrapped.html failures

Push with failures https://treeherder.mozilla.org/#/jobs?repo=autoland&selectedJob=254466184&resultStatus=testfailed%2Cbusted%2Cexception&revision=5607d2f3ef7db1adb3f42102c905d47ed8030c49

Failure log https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=254466184&repo=autoland&lineNumber=10471

Backout: https://hg.mozilla.org/integration/autoland/rev/502b93be5c7c

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/367d45a1ef48
part3 : enable wpt 'too_many_cues.html' and 'too_many_cues_wrapped.html'. r=heycam

Comment 10

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/69150f666319
https://hg.mozilla.org/mozilla-central/rev/6ead08887181
https://hg.mozilla.org/mozilla-central/rev/367d45a1ef48

Comment 11

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/17702 for changes under testing/web-platform/tests

Comment 12

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Can't merge web-platform-tests PR due to failing upstream checks:
Github PR https://github.com/web-platform-tests/wpt/pull/17702
* Taskcluster (pull_request) (https://tools.taskcluster.net/task-group-inspector/#/Wn9BaoVdT2GYp1dXWDxFsQ)

Comment 13

[Alastor Wu [:alwu]]

Hi, James,

This failure upstream check is due to lacking of the fuzzy annotation on upstream, could you help me manually merge this change?

INFO Found 4800 pixels different, maximum difference per channel 2
TEST-UNEXPECTED-FAIL | /webvtt/rendering/cues-with-video/processing-model/too_many_cues_wrapped.html

Thank you.

Comment 14

[James Graham [:jgraham]]

Done! Thanks for the clear explaination.