Open
Bug 1355786
Opened 8 years ago
Updated 2 years ago
H.264 WebRTC tests do not run on Android
Categories
(Core :: WebRTC, defect, P3)
Core
WebRTC
Tracking
()
NEW
People
(Reporter: dminor, Unassigned)
References
Details
The work mentioned in Bug 1149374 seems to be necessary but perhaps not sufficient to get these tests running.
Affected tests:
test_peerConnection_basicH264Video.html
test_peerConnection_bug1042791.html
test_peerConnection_scaleResolution.html (uses both VP8 and H.264, VP8 works)
There may be more affected tests, I'll add them there as I find them.
Reporter | ||
Updated•8 years ago
|
Rank: 25
Priority: -- → P2
Comment 1•8 years ago
|
||
dminor: FYI, These tests will be part of the new Mdm2 tests I am adding in Bug 1353509. I did an initial try run where they were enabled, and you can see the results at: https://treeherder.allizom.org/#/jobs?repo=try&revision=18eafb2b0e186e6f05ffbacdf097d3d64179d2cc&filter-searchStr=Mdm2
Reporter | ||
Comment 2•8 years ago
|
||
I tried a quick rebase of the patches at Bug 1267141 and running test_peerConnection_basicH264Video.html. It still fails.
I think there are probably some misunderstanding here.
These three webrtc/peerconection tests are not related to bug 1149374 or 1267141, which are for EME API.
May I know why we change the skip-if comment here?
https://hg.mozilla.org/integration/mozilla-inbound/rev/bba81310ba7b
The original comment should be correct:
H.264 should be present in the SDP offer - didn't expect -1, but got it, many other failures
Reporter | ||
Comment 4•8 years ago
|
||
(In reply to Munro Mengjue Chiang [:mchiang] from comment #3)
> I think there are probably some misunderstanding here.
> These three webrtc/peerconection tests are not related to bug 1149374 or
> 1267141, which are for EME API.
>
> May I know why we change the skip-if comment here?
> https://hg.mozilla.org/integration/mozilla-inbound/rev/bba81310ba7b
>
> The original comment should be correct:
> H.264 should be present in the SDP offer - didn't expect -1, but got it,
> many other failures
The reason I changed the comment was to have a bug to track getting these tests enabled. I guess I misinterpreted the work in Bug 1043403 as making it possible for the fake H.264 implementation to work on B2G and thought we had to do something similar for Android. At any rate, that manifest is going to be removed any day now. Once that is done, I'll update any H.264 related failures to point to this bug, rather than Bug 1149374.
Comment 5•8 years ago
|
||
FYI, I ran these tests against my local devices using the soon to be available manifests from bug 1358118
https://treeherder.allizom.org/#/jobs?repo=try&revision=18185e3acf6c06af05b11953f7970331e12babb9&selectedJob=87058128
Comment 6•7 years ago
|
||
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3
Comment 7•3 years ago
|
||
https://treeherder.mozilla.org/jobs?repo=try&revision=31f603da1dc0d7d9ec53dc4db38da2338b3239dc
This still does not work.
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•