(In reply to Paul Adenot (:padenot) from comment #12) > Pushed without the test, too much people are complaining about this bug, and > I have no time as per now to make the test pass on try. Leaving this open so > I can push the test sometime. > > https://hg.mozilla.org/integration/mozilla-inbound/rev/1bb461d484b2 This bug is about the test.
Component: Audio/Video → Audio/Video: Playback
JW, do you have a test that does something like: > mediaElement.playbackRate = 2.0; > ...wait a bit... > mediaElement.playbackRate = 1.0; and that checks that everything works as intended ?
Maybe dom/media/test/test_playback_rate.html is what you want.
Yeah I know I wrote it, but when I wrote it, I didn't include a test for this, but you've made quite a lot of fixes to this particular feature, so I was wondering if you remember adding a test for this?
No. I think we should add one. test_playback_rate.html only tests playbackRate changes from 1 to some other values but not the other way around. In fact, I would like to split test_playback_rate.html into more small tests. For now test_playback_rate.html involves several playbackRate changes during the test and sometimes it is hard to debug which session goes wrong.
You need to log in before you can comment on or make changes to this bug.