1047307 - Remove the check of onremovetrack in "test_mediatrack_consuming_mediastream.html"

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

[Shelly Lin [:shelly]]

Fix of bug 1035482, original comment in bug 1038494, comment 8.

Comment 1

[Shelly Lin [:shelly]]

Attachment: 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!

Comment 2

[Shelly Lin [:shelly]]

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

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/9b53034ae4c9

Comment 4

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/9b53034ae4c9

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/7671c6af65a4