Closed Bug 1360423 Opened 7 years ago Closed 7 years ago

Remove the ability to send synchronous notifications from MediaEventSource

Categories

(Core :: Audio/Video: Playback, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

References

Details

Attachments

(1 file)

It turns out that sync notification is a bad idea which is easy to be misused and could results in unexpected reentrant  call flow.

Since it has no users after the mass media code refactoring, it is a good time to remove it now to prevent future users.
Assignee: nobody → jwwang
Priority: -- → P3
See Also: → 1281090
Attachment #8863233 - Flags: review?(gsquelart)
Thanks!
Merged backout
https://hg.mozilla.org/mozilla-central/rev/cf048c7572d7
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: