Bug 1429986 - send telemetry data when users attempt to play mkv file. r=jwwang,liuche MozReview-Commit-ID: Ah3pXC4cJ7n

Bug 1420322 - Test for seamless looping; r?jwwang,padenot MozReview-Commit-ID: KE3Qsql8BW1

Bug 1420322 - Test for seamless looping; r?jwwang,padenot MozReview-Commit-ID: Awbs24pb8SB

Bug 1429986 - send telemetry data when users attempt to play mkv file. r?jwwang,liuche MozReview-Commit-ID: Ah3pXC4cJ7n

Bug 1420322 - Test for seamless looping; r?jwwang,padenot MozReview-Commit-ID: 1zGsRn3D01t

Bug 1420322 - Test for seamless looping; r?jwwang,padenot MozReview-Commit-ID: FSCXNubPSJe

Bug 1429986 - send telemetry data when users attempt to play mkv file. r?jwwang,liuche MozReview-Commit-ID: BR7aWPTgfcW

Bug 1429986 - send telemetry data when users attempt to play mkv file. r?jwwang,francois MozReview-Commit-ID: saPc2Ha1To

Bug 1421505 - Return the correct duration while seamless looping is on; r?jwwang

Bug 1420322 - Test for seamless looping; r?jwwang,padenot MozReview-Commit-ID: FONRhnYOjzx

Bug 1429284 - Expose compositor dropped frame count to HTMLMediaElement's debug info. r=jwwang MozReview-Commit-ID: GYtHxdsGSk1

Bug 1420322 - Test for seamless looping; r?jwwang,padenot MozReview-Commit-ID: 5EYBEeahJBz

Bug 1420322 - Test for seamless looping; r?jwwang,padenot MozReview-Commit-ID: JJScDcW8IvI

Bug 1426291 - Use DataMutex<T> to synchronize CDMCaps. r=jwwang MozReview-Commit-ID: NLnoItNZKP

Bug 1426291 - Add Rust style mutex template which wraps data with the mutex that synchronizes it. r=jwwang Rust's std::sync::Mutex has some nice properties. Its philosphy is to lock data, rather than code. It wraps data you're trying to make thread safe with a mutex, and in order to get a reference to the wrapped data you need to lock the mutex and access it through an intermediate layer. This is good, as the mutex that's protecting access to the data is explicitly associated with the data, and it's impossible to forget to take the lock before accessing the data. This patch adds a similar mutex wrapper to Media Playback code. If it works well, we can look at moving it into xpcom. MozReview-Commit-ID: 4APAic6Fh8m

Bug 1426291 - Use DataMutex<T> to synchronize CDMCaps. r=jwwang MozReview-Commit-ID: NLnoItNZKP

Bug 1426291 - Use DataMutex<T> to synchronize CDMCaps. r=jwwang MozReview-Commit-ID: NLnoItNZKP

Bug 1426291 - Use DataMutex<T> to synchronize CDMCaps. r=jwwang MozReview-Commit-ID: NLnoItNZKP

Bug 1426291 - Use DataMutex<T> to synchronize CDMCaps. r=jwwang MozReview-Commit-ID: NLnoItNZKP

Bug 1426291 - Add Rust style mutex template which wraps data with the mutex that synchronizes it. r=jwwang Rust's std::sync::Mutex has some nice properties. Its philosphy is to lock data, rather than code. It wraps data you're trying to make thread safe with a mutex, and in order to get a reference to the wrapped data you need to lock the mutex and access it through an intermediate layer. This is good, as the mutex that's protecting access to the data is explicitly associated with the data, and it's impossible to forget to take the lock before accessing the data. This patch adds a similar mutex wrapper to Media Playback code. If it works well, we can look at moving it into xpcom. MozReview-Commit-ID: 4APAic6Fh8m

Bug 1426291 - Use DataMutex<T> to synchronize CDMCaps. r=jwwang MozReview-Commit-ID: NLnoItNZKP

Bug 1426291 - Add Rust style mutex template which wraps data with the mutex that synchronizes it. r=jwwang Rust's std::sync::Mutex has some nice properties. Its philosphy is to lock data, rather than code. It wraps data you're trying to make thread safe with a mutex, and in order to get a reference to the wrapped data you need to lock the mutex and access it through an intermediate layer. This is good, as the mutex that's protecting access to the data is explicitly associated with the data, and it's impossible to forget to take the lock before accessing the data. This patch adds a similar mutex wrapper to Media Playback code. If it works well, we can look at moving it into xpcom. MozReview-Commit-ID: 4APAic6Fh8m

Bug 1426291 - Use DataMutex<T> to synchronize CDMCaps. r=jwwang MozReview-Commit-ID: NLnoItNZKP

Bug 1426291 - Add Rust style mutex template which wraps data with the mutex that synchronizes it. r=jwwang Rust's std::sync::Mutex has some nice properties. It's philosphy is to lock data, rather than code. It wraps data you're trying to make thread safe with a mutex, and in order to get a reference to the wrapped data you need to lock the mutex and access it through an intermediate layer. This is good, as the mutex that's protecting access to the data is explicitly associated with the data, and it's impossible to forget to take the lock before accessing the data. This patch adds a similar mutex wrapper to Media Playback code. If it works well, we can look at moving it into xpcom. MozReview-Commit-ID: 4APAic6Fh8m

Bug 1426291 - Use DataMutex<T> to synchronize CDMCaps. r=jwwang MozReview-Commit-ID: NLnoItNZKP

Bug 1426291 - Add Rust style mutex template which wraps data with the mutex that synchronizes it. r=jwwang Rust's std::sync::Mutex has some nice properties. It's philosphy is to lock data, rather than code. It wraps data you're trying to make thread safe with a mutex, and in order to get a reference to the wrapped data you need to lock the mutex and access it through an intermediate layer. This is good, as the mutex that's protecting access to the data is explicitly associated with the data, and it's impossible to forget to take the lock before accessing the data. This patch adds a similar mutex wrapper to Media Playback code. If it works well, we can look at moving it into xpcom. MozReview-Commit-ID: 4APAic6Fh8m

Bug 1426291 - Use DataMutex<T> to synchronize CDMCaps. r?jwwang MozReview-Commit-ID: NLnoItNZKP

Bug 1426291 - Add Rust style mutex template which wraps data with the mutex that synchronizes it. r?jwwang Rust's std::sync::Mutex has some nice properties. It's philosphy is to lock data, rather than code. It wraps data you're trying to make thread safe with a mutex, and in order to get a reference to the wrapped data you need to lock the mutex and access it through an intermediate layer. This is good, as the mutex that's protecting access to the data is explicitly associated with the data, and it's impossible to forget to take the lock before accessing the data. This patch adds a similar mutex wrapper to Media Playback code. If it works well, we can look at moving it into xpcom. MozReview-Commit-ID: 4APAic6Fh8m

Bug 1424930 - remove telemetry for tunneled playback support on Android devices. r=jwwang MozReview-Commit-ID: HKJ0gBBfRKT

Bug 1422263 - turn on fullscreen video orientation lock for all channels. r=jwwang MozReview-Commit-ID: EGCx0AdH4Yy

Bug 1424930 - remove telemetry for tunneled playback support on Android devices. r?jwwang MozReview-Commit-ID: HKJ0gBBfRKT

Bug 1422263 - turn on fullscreen video orientation lock for all channels. r?jwwang MozReview-Commit-ID: EGCx0AdH4Yy

Bug 1424647: Prevent race on AllPromiseHolder::mPromise. r?jwwang If any of the promises gets resolved/rejected prior MozPromise::All completing, it would return nullptr MozReview-Commit-ID: Lqhv2t2upvF

Bug 1424647: Prevent race on AllPromiseHolder::mPromise. r=jwwang If any of the promises gets resolved/rejected prior MozPromise::All completing, it would return nullptr MozReview-Commit-ID: Lqhv2t2upvF

Bug 1424647: Prevent race on AllPromiseHolder::mPromise. r?jwwang If any of the promises gets resolved/rejected prior MozPromise::All completing, it would return nullptr MozReview-Commit-ID: Lqhv2t2upvF

Bug 1424647: Prevent race on AllPromiseHolder::mPromise. r?jwwang If any of the promises gets resolved/rejected prior MozPromise::All completing, it would return nullptr MozReview-Commit-ID: Lqhv2t2upvF

Bug 1421004 - Break the mDocument reference cycle at Destroy(). r=jwwang,smaug We break the cycle reference between VideoDocument/Plugindocument and MediaDocumentStreamListener when the document be destroyed. MozReview-Commit-ID: Aowla95zMgg

Bug 1407810 - Use DDMOZ_LOG in media stack - r=jwwang Mostly-mechanical replacement of MOZ_LOG with DDMOZ_LOG, usually just removing the class name and `this` pointer (as they are already implicitly recorded). Some files needed a bit more work when logging was done from helper classes or static functions. MozReview-Commit-ID: IeJJmzYqWMQ

Bug 1407810 - DDMOZ_LOG to replace MOZ_LOG and also output to DDLogger - r=jwwang This macro may be used to replace MOZ_LOG in most files, where it is used in non-static member functions of classes that are registered with DDLoggedTypeTraits. MozReview-Commit-ID: A8DYmGoMv7f

Bug 1407810 - Use DDLogger in media stack - r=jwwang Mostly-mechanical additions: - Log constructions&destructions, usually by just inheriting from DecoderDoctorLifeLogger, otherwise with explicit log commands (for internal classes for which DecoderDoctorTraits can't be specialized), - Log links between most objects, e.g.: Media element -> decoder -> state machine -> reader -> demuxer -> resource, etc. And logging some important properties and events (JS events, duration change, frames being decoded, etc.) More will be added later on, from just converting MOZ_LOGs, and as needed. MozReview-Commit-ID: KgNhHSz35t0

Bug 1407810 - webidl for HTMLMediaElement.mozRequestDebugLog - r=jwwang,smaug MozReview-Commit-ID: EdCZEiQQjAa

Bug 1407810 - Implement HTMLMediaElement::Moz{Enable,Request}DebugLog - r=jwwang MozReview-Commit-ID: 7Hhy6qBVjDk

Bug 1407810 - Enable DDLogging from the beginning if MOZ_LOG contains DDLogger - r=jwwang This allows logging from the command line without using a webextension. MozReview-Commit-ID: 6pkDbLXzz3X

Bug 1407810 - DDMediaLogs::SizeOf accounts for shallow message queue size - r=jwwang MozReview-Commit-ID: 9irARpytVoh

Bug 1407810 - Tweak atomics in MultiWriterQueue - r=jwwang This queue may be used heavily when multiple threads are running media code that logs thousands of messages per second, so using less strict memory ordering can help with speed. Benchmarks show an improvement of up to twice the queueing speed in some situations. MozReview-Commit-ID: 70UOL8XAZGp

Bug 1407810 - Make DecoderDoctorLogger::sLogState 'ReleaseAcquire' - r=jwwang sLogState is accessed at least once for every DDLog call, even when logging is disabled, so we want it to be as quick as possible for minimal impact on most users. MozReview-Commit-ID: AMstgXmixrv

Bug 1407810 - Use DDMOZ_LOG in media stack - r?jwwang Mostly-mechanical replacement of MOZ_LOG with DDMOZ_LOG, usually just removing the class name and `this` pointer (as they are already implicitly recorded). Some files needed a bit more work when logging was done from helper classes or static functions. MozReview-Commit-ID: IeJJmzYqWMQ

Bug 1407810 - DDMOZ_LOG to replace MOZ_LOG and also output to DDLogger - r?jwwang This macro may be used to replace MOZ_LOG in most files, where it is used in non-static member functions of classes that are registered with DDLoggedTypeTraits. MozReview-Commit-ID: A8DYmGoMv7f

Bug 1407810 - Use DDLogger in media stack - r?jwwang Mostly-mechanical additions: - Log constructions&destructions, usually by just inheriting from DecoderDoctorLifeLogger, otherwise with explicit log commands (for internal classes for which DecoderDoctorTraits can't be specialized), - Log links between most objects, e.g.: Media element -> decoder -> state machine -> reader -> demuxer -> resource, etc. And logging some important properties and events (JS events, duration change, frames being decoded, etc.) More will be added later on, from just converting MOZ_LOGs, and as needed. MozReview-Commit-ID: KgNhHSz35t0

Bug 1407810 - webidl for HTMLMediaElement.mozRequestDebugLog - r?jwwang,smaug MozReview-Commit-ID: EdCZEiQQjAa