Closed Bug 1208270 Opened 9 years ago Closed 9 years ago

Intermittent test_bug1056032.html | We should be able to decode an mp3 using decodeAudioData but couldn't

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: KWierso, Unassigned)

References

Details

(Keywords: intermittent-failure)

09:00:35 INFO - 242 INFO TEST-START | dom/media/webaudio/test/test_bug1056032.html 09:00:35 INFO - ++DOMWINDOW == 58 (09798B00) [pid = 3992] [serial = 93] [outer = 1A524200] 09:00:36 INFO - [3992] WARNING: CoCreateInstance failed, hr=80040154: file c:/builds/moz2_slave/m-in-w32-d-0000000000000000000/build/src/dom/media/directshow/DirectShowUtils.cpp, line 195 09:00:36 INFO - [3992] WARNING: dmoWrapper Init failed, hr=80040154: file c:/builds/moz2_slave/m-in-w32-d-0000000000000000000/build/src/dom/media/directshow/DirectShowUtils.cpp, line 244 09:00:36 INFO - [3992] WARNING: NS_ENSURE_TRUE(SUCCEEDED(hr)) failed: file c:/builds/moz2_slave/m-in-w32-d-0000000000000000000/build/src/dom/media/directshow/DirectShowReader.cpp, line 170 09:00:36 INFO - [3992] WARNING: Decoder=a9c7760 ReadMetadata failed, rv=80004005 HasValidMedia=0: file c:/builds/moz2_slave/m-in-w32-d-0000000000000000000/build/src/dom/media/MediaDecoderReader.cpp, line 279 09:00:36 INFO - TEST-INFO | started process screenshot 09:00:36 INFO - TEST-INFO | screenshot: exit 0 09:00:36 INFO - 243 INFO TEST-UNEXPECTED-FAIL | dom/media/webaudio/test/test_bug1056032.html | We should be able to decode an mp3 using decodeAudioData but couldn't This is happening because we're getting a class not registered error (0x80040154) when calling <http://hg.mozilla.org/mozilla-central/annotate/6256ec9113c1/dom/media/directshow/DirectShowReader.cpp#l162>. Chris, does this ring any bells?
Flags: needinfo?(cpearce)
That happens when the machine doesn't have the codecs installed. This is probably a new Windows Enterprise N machine without the Media Feature Pack installed right?
Flags: needinfo?(cpearce) → needinfo?(ehsan)
catlee: We need to ensure that the "Media Feature Pack" is installed on our mochitest machines. Based on the test failures logged in this bug, it's not installed on at least t-w732-ix-001 and t-w732-ix-117. A whole raft of tests have been disabled on Windows by the sheriffs, and they could have been disabled due to this. All the Windows machines in which are logged in bug 1187903 for Windows failures should be checked. We need a fail-safe way to ensure that new Windows test machines are capable of playing media. Can we stop using the "N" variant of Windows (which doesn't ship with platform audio/video decoders), or at least add procedures to ensure that when imaging Windows N machines that the Media Feature Pack is installed?
Flags: needinfo?(catlee)
(In reply to Chris Pearce (:cpearce) from comment #7) > That happens when the machine doesn't have the codecs installed. This is > probably a new Windows Enterprise N machine without the Media Feature Pack > installed right? I have no idea. 302 Chris. :-)
Flags: needinfo?(ehsan)
Mark/Q, can we make sure we're installing the media packs? I know we discussed switching to non-N installs at one point, but I don't know if we ever went through with that.
Flags: needinfo?(q)
Flags: needinfo?(mcornmesser)
We are using the non-N image, but we are installing the media feature pack during the mdt installation. Is there a machine we are seeing the issue on that I can jump on and take a look?
Flags: needinfo?(mcornmesser)
The machines listed in the Treeherder Robot comments above are suspect: t-w732-ix-001 and t-w732-ix-117. I think the failures on t-w732-ix-003, t-w732-ix-135 and t-w864-ix-113 were caused by me landing a bad patch that broke MP3, it's been backed out. So I don't think the fail logs for those machines necessarily indicate that they're missing the Media Feature Pack. So check t-w732-ix-001 and t-w732-ix-117 first. But if you do find that t-w732-ix-001 and t-w732-ix-117 don't have the Media Feature Pack for Windows N/KN, then we should audit all Windows N machines to ensure they're all going to work.
Flags: needinfo?(mcornmesser)
These are the same hosts that are being investigated in bug 1207160 and bug 1200180
Blocks: 1207160, 1200180
I verified that 001 and 117 do not have the media pack installed. Also we should note that all Windows testers are based on the N version.
Flags: needinfo?(mcornmesser)
Depends on: 1209577
media pack install reinstated via gpo with some tweaks per child bug. Windows 10 is not being based on the N version luckily.
Flags: needinfo?(q)
(In reply to Q from comment #21) > media pack install reinstated via gpo with some tweaks per child bug. > > Windows 10 is not being based on the N version luckily. Did you check all Windows slaves, or only the two listed in comment 20? We need to ensure all existing slaves, and all new ones that get provisioned in future, have the Media Feature Pack installed.
Flags: needinfo?(q)
GPO is set to distribute to all slaves that require it.
Flags: needinfo?(q)
Component: Web Audio → General Automation
Product: Core → Release Engineering
QA Contact: catlee
Can we mark this one as FIXED?
Flags: needinfo?(catlee)
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.