Closed
Bug 1138942
Opened 9 years ago
Closed 9 years ago
Add Autophone specific video test manifest
Categories
(Testing :: Mochitest, defect)
Testing
Mochitest
Tracking
(firefox40 fixed)
RESOLVED
FIXED
mozilla40
Tracking | Status | |
---|---|---|
firefox40 | --- | fixed |
People
(Reporter: bc, Unassigned)
Details
Attachments
(1 file, 1 obsolete file)
1.27 KB,
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
Autophone currently runs several mochitests via existing manifests in the tree: https://wiki.mozilla.org/Auto-tools/Projects/AutoPhone/Autophone_for_developers#Tests The intent is to test video and audio playback and the full sets of tests are not needed and actually take too long to execute on the available devices. This manifest is intended to provide a place for video tests to be added specifically for Autophone. ted: I'm piggy backing on ahal's Makefile.in since we haven't removed it yet in favor of moz.build. Is this ok for now? snorp: this is a preliminary patch with the tests you listed yesterday. Before I ask for a formal review, can we get a full list of tests? Perhaps you could invite some of the media folks to provide input on which tests we would like?
Attachment #8571958 -
Flags: feedback?(ted)
Attachment #8571958 -
Flags: feedback?(snorp)
Comment on attachment 8571958 [details]
autophone-video
I think 'autophone-media' is a better name for the file, but looks like a good start to me.
Anthony, can you or one of your folks let Bob know what we should put in here? The idea is for Autophone to only test the media bits that are significantly affected by hardware or OS differences.
Flags: needinfo?(ajones)
Attachment #8571958 -
Flags: feedback?(snorp) → feedback+
Video reftests would probably be the best approach. This will test our ability to decode video on each different piece of hardware. We need to add a reftest for MP4.
Flags: needinfo?(ajones)
Reporter | ||
Comment 3•9 years ago
|
||
works in local testing on autophone.
Attachment #8571958 -
Attachment is obsolete: true
Attachment #8571958 -
Flags: feedback?(ted)
Attachment #8588142 -
Flags: review?(jmaher)
Comment 4•9 years ago
|
||
Comment on attachment 8588142 [details] [diff] [review] autophone-media.patch Review of attachment 8588142 [details] [diff] [review]: ----------------------------------------------------------------- it would be cool if we could use the new tagging that :ahal is working on. It might be a bit premature to do it, but that seems like the right longer term solution. maybe file a bug to do that. The work was done in bug 987360.
Attachment #8588142 -
Flags: review?(jmaher) → review+
Reporter | ||
Comment 5•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/021f785c5e9b I've also been testing using reftests for: layout/reftests/ogg-video/reftest.list layout/reftests/webm-video/reftest.list Once the mochitest manifest merges, I'll run tests on treeherder staging and if all is well file a bug to get these on trychooser. Hopefully we can discontinue the old autophone mochitests early next week and turn these on.
Status: NEW → ASSIGNED
Comment 6•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/021f785c5e9b
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox40:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
You need to log in
before you can comment on or make changes to this bug.
Description
•