Closed
Bug 1331528
Opened 9 years ago
Closed 9 years ago
WebM alpha needs a reftest
Categories
(Core :: Audio/Video: Playback, defect)
Core
Audio/Video: Playback
Tracking
()
RESOLVED
FIXED
mozilla53
| Tracking | Status | |
|---|---|---|
| firefox53 | --- | fixed |
People
(Reporter: kkoorts, Assigned: kkoorts)
References
Details
Attachments
(1 file)
We need a way to check that WebM alpha is working. A ref test can indicate if the alpha channel is being rendered correctly.
| Comment hidden (mozreview-request) |
|
||
Comment 2•9 years ago
|
||
| mozreview-review | ||
Comment on attachment 8827307 [details]
Bug 1331528 - Added reftest for WebM alpha.
https://reviewboard.mozilla.org/r/105026/#review105854
::: layout/reftests/webm-video/webm-alpha-ref.html:2
(Diff revision 1)
> +<!DOCTYPE HTML>
> +<html class="reftest-wait">
I think we can remove the wait class and the script here. The black page should be ready by the time the load event is fired (which prompts the harness to snapshot the page).
::: layout/reftests/webm-video/webm-alpha.html:2
(Diff revision 1)
> +<!DOCTYPE HTML>
> +<html class="reftest-wait">
It may be safe to remove the wait here depending on intended test behaviour. Based on the poster2 test (and looking through the spec) I think you can rely on that the test will have loaded enough data to display its video poster (an image to be shown before playing, or if no image is given then the first frame) by the time the harness snapshots.
If the video contains frames with the same content, and you don't need to start playback, you could remove the waits, and the on\* event handlers and rely on alpha being displayed correctly in the poster.
|
|
||
Comment 3•9 years ago
|
||
| mozreview-review-reply | ||
Comment on attachment 8827307 [details]
Bug 1331528 - Added reftest for WebM alpha.
https://reviewboard.mozilla.org/r/105026/#review105854
I recall there being something during development that may have prompted the adding of the waits to the tests. I've tested without the waits that I suggested could be removed, and appear to be getting desired behaviour (along with breakages if I change the video), but if I've overlooked something please let me know.
| Comment hidden (mozreview-request) |
|
|
||
Comment 5•9 years ago
|
||
| mozreview-review | ||
Comment on attachment 8827307 [details]
Bug 1331528 - Added reftest for WebM alpha.
https://reviewboard.mozilla.org/r/105026/#review106046
LGTM.
Attachment #8827307 -
Flags: review?(bvandyk) → review+
Comment on attachment 8827307 [details]
Bug 1331528 - Added reftest for WebM alpha.
https://reviewboard.mozilla.org/r/105026/#review106048
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/44b7d75e21e8
Added reftest for WebM alpha. r=SingingTree
Keywords: checkin-needed
|
||
Comment 8•9 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox53:
--- → fixed
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in
before you can comment on or make changes to this bug.
Description
•