Summary: Fix mochitest "test_mediatrack_consuming_mediastream.html" → Remove the check of onremovetrack in "test_mediatrack_consuming_mediastream.html"
Assignee: nobody → slin
Created attachment 8466124 [details] [diff] [review] Test case fix Hi roc, I think it should be fine to remove the check of onremovetrack on video element, because 1. It should really be checked at the onended event handler of MediaStream or the onended event handler of MediaStreamTrack, and we implement them neither, and 2. We are still checking the length of video.audioTracks and video.videoTracks. Thanks!
Attachment #8466124 - Flags: review?(roc)
Try server result with failing platform: (Android 2.3 Emulator try opt test mochitest-4 ) https://tbpl.mozilla.org/?tree=Try&rev=0bc5c4782685 (b2g_emulator_vm try opt test mochitest-3) https://tbpl.mozilla.org/?tree=Try&rev=72b4bd387fe4
Attachment #8466124 - Flags: review?(roc) → review+
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
status-firefox32: --- → unaffected
status-firefox33: --- → fixed
status-firefox34: --- → fixed
You need to log in before you can comment on or make changes to this bug.