Closed Bug 1392508 Opened 7 years ago Closed 7 years ago

Reduce include-graph from MediaData.h

Categories

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

55 Branch
enhancement

Tracking

()

RESOLVED FIXED
mozilla57
Tracking Status
firefox57 --- fixed

People

(Reporter: cpearce, Assigned: cpearce)

References

Details

Attachments

(1 file)

Including MediaData.h explodes to including some unfortunate things, some of which we don't need.
Comment on attachment 8899679 [details] Bug 1392508 - Don't include nsRect.h, nsSize.h and nsIMemoryReporter.h in MediaData.h as they're unused. https://reviewboard.mozilla.org/r/170990/#review176156 r+ assuming you've tried a non-unified build.
Attachment #8899679 - Flags: review?(gsquelart) → review+
Pushed by cpearce@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0cb8d97fe926 Don't include nsRect.h, nsSize.h and nsIMemoryReporter.h in MediaData.h as they're unused. r=gerald
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: