Closed Bug 1179521 Opened 6 years ago Closed 6 years ago

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

Categories

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

defect

Tracking

()

RESOLVED FIXED
mozilla42
Tracking Status
firefox42 --- fixed

People

(Reporter: cpearce, Assigned: cpearce)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

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.
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.
Assignee: nobody → cpearce
Status: NEW → ASSIGNED
Attachment #8629116 - Flags: review?(gsquelart)
Attachment #8629116 - Flags: review?(gsquelart) → review+
https://hg.mozilla.org/mozilla-central/rev/ba1a49f438a6
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
You need to log in before you can comment on or make changes to this bug.