1057879 - Enable MP4Reader on Windows

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

[Chris Pearce [:cpearce (Not reading bugmail)]]

We should enable MP4Reader on Windows to replace WMFReader to fix bug 882537 and move towards having the same MP4 demuxer on all platforms.

We should probably target Gecko 35 for this.

Comment 1

[Chris Pearce [:cpearce (Not reading bugmail)]]

Try push:

https://tbpl.mozilla.org/?tree=Try&rev=48955bc91064

Only 1 <video> related test failure on Windows in about 116 runs.

Failure was:

WINNT 6.2 try debug test mochitest-1 on 2014-08-23 02:34:38 PDT for push 48955bc91064

Download Raw Log
Summary

17753 INFO TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_autoplay_contentEditable.html | Test timed out. 17754 INFO TEST-OK | /tests/content/media/test/test_autoplay_contentEditable.html | took 324234ms
17856 INFO TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_buffered.html | Test timed out. 17857 INFO TEST-OK | /tests/content/media/test/test_buffered.html | took 329920ms
17909 INFO TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_bug465498.html | Test timed out. 17910 INFO TEST-OK | /tests/content/media/test/test_bug465498.html | took 328825ms
17972 INFO TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_bug493187.html | Test timed out. 17973 INFO TEST-UNEXPECTED-FAIL | (SimpleTest/TestRunner.js) | 4 test timeouts, giving up. 17974 INFO TEST-UNEXPECTED-FAIL | (SimpleTest/TestRunner.js) | Skipping 496 remaining tests. 17975 INFO dumping last 4 message(s)
17981 INFO TEST-UNEXPECTED-FAIL | (SimpleTest/TestRunner.js) | /tests/content/media/test/test_bug493187.html - finished in a non-clean fashion, probably because it didn't call SimpleTest.finish()17982 INFO TEST-OK | (SimpleTest/TestRunner.js) |
Return code: 1

Comment 2

[Chris Pearce [:cpearce (Not reading bugmail)]]

Attachment: Screenshot of Try run

Comment 3

[Chris Pearce [:cpearce (Not reading bugmail)]]

(In reply to Chris Pearce (:cpearce) from comment #2)
> Created attachment 8478029 [details]
> Screenshot of Try run

(Failures on Win7 Debug here are in WebAudio and test_sessionhistory)

Comment 4

[Chris Pearce [:cpearce (Not reading bugmail)]]

Attachment: Patch: Enable MP4Reader on Windows.

Comment 5

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/efe5844c9a5b

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/efe5844c9a5b