Disable ClearKey EME test on 10.6

RESOLVED FIXED in Firefox 42

Status

()

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: jya, Assigned: jya)

Tracking

(Blocks: 1 bug)

Trunk
mozilla42
Points:
---

Firefox Tracking Flags

(firefox42 fixed)

Details

Attachments

(2 attachments)

(Assignee)

Description

4 years ago
The current EME clearkey tests intermittently timeout in 10.6.

The current code path will never be used on 10.6 other than in those tests and ultimately, it will use a GMP ClearKey plugin rather than what is currently there.

Decision was made to completely disable those tests on 10.6 instead.
(Assignee)

Comment 1

4 years ago
Attachment #8636406 - Flags: review?(cpearce)
Comment on attachment 8636406 [details] [diff] [review]
Disable EME tests on 10.6.

Review of attachment 8636406 [details] [diff] [review]:
-----------------------------------------------------------------

You should have a todo("Test disabled on this platform"); in each of these.

I think you are better to avoid calling into the MediaTestManager at all. You should basically do:

if (!MacOSX106()) {
  SimpleleTest.waitForExplicitFinish();
  SetupEMEPref(beginTest);  
} else {
  todo("Test disabled on this platform");
}
Attachment #8636406 - Flags: review?(cpearce) → review-
(Assignee)

Comment 3

4 years ago
Attachment #8636900 - Flags: review?(cpearce)
Comment on attachment 8636900 [details] [diff] [review]
Disable EME tests on 10.6.

Review of attachment 8636900 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks.
Attachment #8636900 - Flags: review?(cpearce) → review+
https://hg.mozilla.org/mozilla-central/rev/59b1af051414
Assignee: nobody → jyavenard
Status: NEW → RESOLVED
Last Resolved: 4 years ago
status-firefox42: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
You need to log in before you can comment on or make changes to this bug.