1362107 - Add missing includes to WebMDemuxer

Status: RESOLVED FIXED

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

Description

[Ralph Giles (:rillian)]

Moving stdint.h fix from bug 1314147. My editor doesn't do unified builds, so it complains about undefined symbols on the WebMDemuxer source files.

Comment 1

[Ralph Giles (:rillian)]

Attachment: Bug 1362107 - Add missing includes to WebMDemuxer.

This was working because of our unified build, but having
the correct definitions for all types used by the header
helps editor navigation tools.

Review commit: https://reviewboard.mozilla.org/r/136248/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/136248/

Comment 2

[Jean-Yves Avenard [:jya]]

Comment on attachment 8864576 [details]
Bug 1362107 - Add missing includes to WebMDemuxer.

https://reviewboard.mozilla.org/r/136248/#review139288

::: dom/media/webm/WebMDemuxer.h:11
(Diff revision 1)
>  #if !defined(WebMDemuxer_h_)
>  #define WebMDemuxer_h_
>  
>  #include "nsTArray.h"
>  #include "MediaDataDemuxer.h"
> +#include "MediaResource.h"

a pity we can't use forward declaration for all the MediaResource here, problem is MediaResourceIndex being used on the stack

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/621a61001365
Add missing includes to WebMDemuxer. r=jya

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/621a61001365