Having TrackID in StreamTracks.h means that MediaInfo.h, which includes StreamTracks.h, ends up pulling in the MSG and a bunch of DOM bindings code. If we move it out to its own file, we'll get a simpler include graph, at least for the playback code.

Comment on attachment 8902515 [details] Bug 1395021 - Move TrackID definition from StreamTracks.h into TrackID.h. https://reviewboard.mozilla.org/r/174100/#review179370

Comment on attachment 8902516 [details] Bug 1395021 - Fix up includes to account for StreamTracks.h not being indirectly included in so many places. https://reviewboard.mozilla.org/r/174102/#review179372

It would be easier to detect over/insufficient includes if we build all files under dom/media in non-unified mode.

Pushed by cpearce@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/620ce0f4d360 Move TrackID definition from StreamTracks.h into TrackID.h. r=jwwang https://hg.mozilla.org/integration/autoland/rev/1ed072a5827f Fix up includes to account for StreamTracks.h not being indirectly included in so many places. r=jwwang

https://hg.mozilla.org/mozilla-central/rev/620ce0f4d360 https://hg.mozilla.org/mozilla-central/rev/1ed072a5827f