1179521 - [EME] Replace test_eme_obs_notification.html with test for "mediakeys-request" notification

Status: RESOLVED FIXED

(Core :: Audio/Video, defect, P2)

Description

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

In bug 1127416 we added an observer service notification when an EME media was loaded. We no longer use it. We should remove that.

We now use the "mediakeys-request" notification, dispatched by MediaKeySystemAccess::NotifyObservers(). We should have a unit test for that, and remove the old test, test_eme_obs_notification.html.

Comment 1

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

Attachment: Patch: Add test_eme_request_notifications

Add test_eme_request_notifications and remove test_eme_obs_notification.

We can't test all of the mediakeys-request notifications, as we'd need to be using Adobe EME, which we can't test on our Treeherder machines. We can test most of the cases though.

Comment 2

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

https://treeherder.mozilla.org/#/jobs?repo=try&revision=e22b9bd34f81

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/ba1a49f438a6

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/ba1a49f438a6