I want to add a reftest to verify the last frame of bipbop.mp3. The testcase flow: 1. play the bipbop.mp4 2. listen the onended event 3. check the screen Since we are checking the last frame, I think we can keep the preference skipToNextKeyFrame TRUE.

Comment on attachment 8885589 [details] Bug 1378285 - Last frame reftest for bipbop.mp4 . https://reviewboard.mozilla.org/r/156436/#review161986 ::: dom/media/test/reftest/bipbop_300_215kbps.mp4.lastframe.html:9 (Diff revision 2) > +<script type="text/javascript"> > +function doTest() { > + var video = document.getElementById("v1"); > + video.src = "../bipbop_300_215kbps.mp4"; > + video.play(); > + video.addEventListener("ended", function() { nit : modify this listener as one-time-usage. ``` video.addEventListener("ended", function() { // do something }, {once: true}); ``` ::: dom/media/test/reftest/bipbop_300_215kbps.mp4.lastframe.html:10 (Diff revision 2) > +function doTest() { > + var video = document.getElementById("v1"); > + video.src = "../bipbop_300_215kbps.mp4"; > + video.play(); > + video.addEventListener("ended", function() { > + video.seenEnded = true; why you need this?

Comment on attachment 8886079 [details] Bug 1378285 - Modify the generateREF.html to resolve the output limit of console. https://reviewboard.mozilla.org/r/156884/#review161988

Comment on attachment 8886908 [details] Bug 1378285 - Modify the generateREF.html to resolve the output limit of console. https://reviewboard.mozilla.org/r/157634/#review162754

Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/autoland/rev/90497fbb54a6 Last frame reftest for bipbop.mp4 . r=alwu https://hg.mozilla.org/integration/autoland/rev/820a85e09ac3 Modify the generateREF.html to resolve the output limit of console. r=alwu

https://hg.mozilla.org/mozilla-central/rev/90497fbb54a6 https://hg.mozilla.org/mozilla-central/rev/820a85e09ac3