Closed Bug 1422662 Opened 7 years ago Closed 7 years ago

Move BaseMediaResource::Close() into MediaResource class

Categories

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

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla59
Tracking Status
firefox59 --- fixed

People

(Reporter: cpearce, Assigned: cpearce)

References

Details

Attachments

(1 file)

For the GeckoMedia project, I need to be able to shutdown an abstract MediaResource, so I would like to move BaseMediaResource::Close() up the inheritance hierarchy into MediaResource.
Comment on attachment 8934045 [details]
Bug 1422662 - Move BaseMediaResource::Close() into MediaResource class.

https://reviewboard.mozilla.org/r/204980/#review210464
Attachment #8934045 - Flags: review?(jwwang) → review+
Pushed by cpearce@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7a2629417a47
Move BaseMediaResource::Close() into MediaResource class. r=jwwang
Backout by ccoroiu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8295eb601d27
Backed out changeset 7a2629417a47 for bustage r=backout on a CLOSED TREE
Pushed by cpearce@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0fe153d0bc4a
Move BaseMediaResource::Close() into MediaResource class. r=jwwang
https://hg.mozilla.org/mozilla-central/rev/0fe153d0bc4a
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: