So we can remove MediaDecoder::IsSameOriginMedia which is called by MDSM on a non-main thread.

Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=a5a6ea639d82

Bug 1191173 - Mirror MediaDecoder::mSameOriginMedia in MDSM.

Comment on attachment 8644095 [details] MozReview Request: Bug 1191173 - Mirror MediaDecoder::mSameOriginMedia in MDSM. https://reviewboard.mozilla.org/r/15227/#review13655 ::: dom/media/MediaDecoder.cpp:416 (Diff revision 1) > + "MediaDecoder::mSameOriginMedia (Canonical)") it seems that the usual mozilla is to always add a trailing , even on the last member, or have it at the front. (I like the later better) How often does mSameOriginMedia changes ? Should it be set once and for all in the decoder ? If that's the case and it's known when creating the state machine it could be passed as parameter instead.

https://reviewboard.mozilla.org/r/15227/#review13655 > it seems that the usual mozilla is to always add a trailing , even on the last member, or have it at the front. (I like the later better) Yeah, I also like the later better. But for now I just follow the existing style of MediaDecoder. I will file a new bug to fix the style. MediaDecoder::UpdateSameOriginStatus() is called by HTMLMediaElement::NotifyDecoderPrincipalChanged() which is called when new data with different principals are downloaded (see MediaCacheStream::NotifyDataReceived). So it is not immutable.

Thanks for the review.