1185814 - Disable ClearKey EME test on 10.6

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

[Jean-Yves Avenard [:jya]]

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.

Comment 1

[Jean-Yves Avenard [:jya]]

Attachment: Disable EME tests on 10.6.

Comment 2

[Chris Pearce [:cpearce (Not reading bugmail)]]

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");
}

Comment 3

[Jean-Yves Avenard [:jya]]

Attachment: Disable EME tests on 10.6.

Comment 4

[Chris Pearce [:cpearce (Not reading bugmail)]]

Comment on attachment 8636900 [details] [diff] [review]
Disable EME tests on 10.6.

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

Thanks.

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/59b1af051414

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/59b1af051414