Created attachment 8567307 [details] xpcshell failure log [Tracking Requested - why for this release]: Test permafail when Gecko 38 merges to Beta. Rumor has it that bug 1089867 is wanted for Fx37 too, so requesting tracking there as well. https://treeherder.mozilla.org/logviewer.html#?job_id=5077501&repo=try https://treeherder.mozilla.org/logviewer.html#?job_id=5077492&repo=try
https://hg.mozilla.org/try/raw-rev/a116b0b86186 is the raw diff for the patch if you want to test locally.
status-firefox37: --- → affected
status-firefox38: --- → affected
tracking-firefox37: ? → +
tracking-firefox38: ? → +
Assignee: nobody → spohl.mozilla.bugs
I'm able to reproduce these failures locally. For some reason, when we run into these failures we're disabling Adobe EME after we enabled it via the pref... Still trying to make sense of it all.
Created attachment 8567438 [details] [diff] [review] Patch Ok, looks like the media.eme.enabled pref is set to false by default when we're run as beta. The tests failed to force this pref to true during initialization and hence the tests failed. To be clear: the mistake was in our tests and the client code appears to be working as intended.
Attachment #8567438 - Flags: review?(gfritzsche)
With this patch, all tests succeed with local builds on OSX and Linux.
4 years ago
Attachment #8567438 - Flags: review?(gfritzsche) → review+
Comment on attachment 8567438 [details] [diff] [review] Patch Approval Request Comment [Feature/regressing bug #]: Adobe EME [User impact if declined]: When bug 1089867 merges to beta, GMP tests will start failing. We may therefore not be able to land bug 1089867 for Firefox 37 unless this patch lands at the same time. [Describe test coverage new/current, TreeHerder]: Local testing showed that all issues discovered by RyanVM are fixed with this patch. [Risks and why]: None that I'm aware of. [String/UUID change made/needed]: none
Status: NEW → RESOLVED
Last Resolved: 4 years ago
status-firefox38: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
Verified on Try too :)
Status: RESOLVED → VERIFIED
status-firefox38: fixed → verified
Comment on attachment 8567438 [details] [diff] [review] Patch Test-only changes don't need approval. Personally, I'd just include these fixes in the roll-up patch in the other bug.
Thanks, Ryan! I've rolled it up into the patch for uplift in bug 1089867, but I'm unable to set the approval flag back to a+. Are you able to do so?
In this case, I think it's fine to just use the original approval since the only change was the test-only addition.
You need to log in before you can comment on or make changes to this bug.