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)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: KWierso, Unassigned)
References
Details
(Keywords: intermittent-failure)
No description provided.
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 6•9 years ago
|
||
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)
Comment 7•9 years ago
|
||
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)
Comment 8•9 years ago
|
||
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)
Comment 9•9 years ago
|
||
(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)
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 16•9 years ago
|
||
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)
Comment 17•9 years ago
|
||
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)
Comment 18•9 years ago
|
||
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)
Comment 19•9 years ago
|
||
These are the same hosts that are being investigated in bug 1207160 and bug 1200180
Comment 20•9 years ago
|
||
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)
Comment 21•9 years ago
|
||
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)
Comment 22•9 years ago
|
||
(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)
Updated•9 years ago
|
Component: Web Audio → General Automation
Product: Core → Release Engineering
QA Contact: catlee
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•7 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•