e7882b5cd8f6ba4bce49b048115f2961a94dc706: Bug 1208371 - Forward declare MediaStreamTrack in MediaPipeline.h. r?bwc draft
Andreas Pehrson <pehrsons@gmail.com> - Fri, 22 Jan 2016 11:33:43 +0800 - rev 342177
Push 13352 by pehrsons@gmail.com at Fri, 18 Mar 2016 13:49:47 +0000
Bug 1208371 - Forward declare MediaStreamTrack in MediaPipeline.h. r?bwc MozReview-Commit-ID: C6PCMIEJVQD
fcd8215288dafb7169db09cad658d6512e76a8f7: Bug 1208371 - Ensure a media element's ImageContainer is protected when playing a stream. r?mt,jesup draft
Andreas Pehrson <pehrsons@gmail.com> - Thu, 04 Feb 2016 09:27:09 +0800 - rev 342176
Push 13352 by pehrsons@gmail.com at Fri, 18 Mar 2016 13:49:47 +0000
Bug 1208371 - Ensure a media element's ImageContainer is protected when playing a stream. r?mt,jesup HTMLMediaElement needs special protection when playing a stream since its ImageContainer can outlive the video track of a stream. Consider for instance when a (cross-origin) video track is removed from a DOMMediaStream by a user and the remaining video track (non-CORS) does not yet contain any actual video frames. The HTMLMediaElement will display a frame from the removed track but the DOMMediaStream's principal has been updated to not include the principal from the removed track. With this patch we handle this by letting VideoFrameContainer notify HTMLMediaElement when it has flushed out all video frames belonging to a certain PrincipalHandle. I.e., when a new PrincipalHandle has been applied to the underlying ImageContainer. MozReview-Commit-ID: LvIZPl6Rdgj
092b59187d402e81da8007c1c569bb61da351a75: Bug 1208371 - Ensure DOMMediaStream principals reflect what could reside in their playback streams. r?mt,jesup draft
Andreas Pehrson <pehrsons@gmail.com> - Wed, 16 Mar 2016 16:00:34 +0100 - rev 342175
Push 13352 by pehrsons@gmail.com at Fri, 18 Mar 2016 13:49:47 +0000
Bug 1208371 - Ensure DOMMediaStream principals reflect what could reside in their playback streams. r?mt,jesup Calculating a principal when adding a track is easy - just combine the new track principal into the stream's principal. When removing a track it's a bit trickier. The DOMMediaStream has to wait until the MediaStreamGraph has removed the track from the underlying playback stream. We do this by letting the MediaStreamGraph return a Pledge (single threaded Promise) when blocking a track in a stream (the way we end removed tracks). The pledge gets passed to the MediaStreamGraph and when the block has been applied it is passed back to the main thread where it is finally resolved and the DOMMediaStream may recompute its principal once all outstanding track removals have been applied. MozReview-Commit-ID: 3QP0YcDyfGf
a049665ffbf62f44964bd7e6bc0c45ca7668a195: Bug 1208371 - Hook up MediaPipeline with PrincipalHandle. r?mt,bwc draft
Andreas Pehrson <pehrsons@gmail.com> - Thu, 03 Mar 2016 17:30:39 +0100 - rev 342174
Push 13352 by pehrsons@gmail.com at Fri, 18 Mar 2016 13:49:47 +0000
Bug 1208371 - Hook up MediaPipeline with PrincipalHandle. r?mt,bwc MozReview-Commit-ID: DLyLZu7kC3p
5c80a236a26a1154431bf3c8998ce539cf00ea08: Bug 1208371 - Hook up DecodedStream with PrincipalHandle. r?mt,jwwang draft
Andreas Pehrson <pehrsons@gmail.com> - Tue, 02 Feb 2016 13:14:13 +0800 - rev 342173
Push 13352 by pehrsons@gmail.com at Fri, 18 Mar 2016 13:49:47 +0000
Bug 1208371 - Hook up DecodedStream with PrincipalHandle. r?mt,jwwang MozReview-Commit-ID: 1LmiFyfjhsr
146465c1d9a3fd779fe144b8ba2c487f75dee0e5: Bug 1208371 - Add a PrincipalHandleListener to MediaStreamTrack. r?mt,jesup draft
Andreas Pehrson <pehrsons@gmail.com> - Tue, 15 Mar 2016 10:52:25 +0100 - rev 342172
Push 13352 by pehrsons@gmail.com at Fri, 18 Mar 2016 13:49:47 +0000
Bug 1208371 - Add a PrincipalHandleListener to MediaStreamTrack. r?mt,jesup MozReview-Commit-ID: JvhXrlKPZAC
edc6a379b339b69be5e598d14b455968d0a1ec9b: Bug 1208371 - Add `NotifyPrincipalHandleChanged` to MediaStreamTrackListener. r?mt,jesup draft
Andreas Pehrson <pehrsons@gmail.com> - Tue, 26 Jan 2016 16:45:25 +0800 - rev 342171
Push 13352 by pehrsons@gmail.com at Fri, 18 Mar 2016 13:49:47 +0000
Bug 1208371 - Add `NotifyPrincipalHandleChanged` to MediaStreamTrackListener. r?mt,jesup MozReview-Commit-ID: Hq0XtqIiMAO
c5beacf7bb63e4eb9c0c8e521cccd4c73dc68f82: Bug 1208371 - Add PrincipalHandle to MediaChunks. r?mt,jesup draft
Andreas Pehrson <pehrsons@gmail.com> - Tue, 15 Mar 2016 14:27:41 +0100 - rev 342170
Push 13352 by pehrsons@gmail.com at Fri, 18 Mar 2016 13:49:47 +0000
Bug 1208371 - Add PrincipalHandle to MediaChunks. r?mt,jesup PrincipalHandle is a thread safe pointer to a holder of (the main-thread-only nsIPrincipal) that can be passed around the MSG. A MediaStreamTrack whose source has just updated its principal, sets the new principal aside (as its "pending principal"), and combines the new principal into its current principal. Then the source starts passing the new principal to the MediaStreamGraph as a PrincipalHandle. Changes to a track's PrincipalHandle on the MSG will be surfaced through the MediaStreamTrackListener API. These changes are dispatched to main thread and compared to a MediaStreamTrack's pending principal. In case of a match the track knows the correct principal is flowing and can move the pending principal to be the current principal and update any main thread principal observers. MozReview-Commit-ID: D0JXGWhQFFU
265d381baeca0228a72b2bb48871270d608fa352: Bug 1208371 - Implement operator!= for nsMainThreadPtrHandle. r?bholley draft
Andreas Pehrson <pehrsons@gmail.com> - Thu, 04 Feb 2016 00:24:52 +0800 - rev 342169
Push 13352 by pehrsons@gmail.com at Fri, 18 Mar 2016 13:49:47 +0000
Bug 1208371 - Implement operator!= for nsMainThreadPtrHandle. r?bholley MozReview-Commit-ID: EjlvLi9VsFA
b772f4d4bc19b82712cb4b876bc4c783add0cce2: Bug 1208371 - Move FindTrack from SourceMediaStream to MediaStream. r?roc draft
Andreas Pehrson <pehrsons@gmail.com> - Thu, 21 Jan 2016 19:25:25 +0800 - rev 342168
Push 13352 by pehrsons@gmail.com at Fri, 18 Mar 2016 13:49:47 +0000
Bug 1208371 - Move FindTrack from SourceMediaStream to MediaStream. r?roc MozReview-Commit-ID: 2BzGdGdbpy0
323442c4e049bac5f4cf210980c79aa573e4d21b: Bug 1208371 - Improve reliability of test_mediarecorder_record_timeslice.html. r?jwwang draft
Andreas Pehrson <pehrsons@gmail.com> - Wed, 06 Jan 2016 11:56:27 +0800 - rev 342167
Push 13352 by pehrsons@gmail.com at Fri, 18 Mar 2016 13:49:47 +0000
Bug 1208371 - Improve reliability of test_mediarecorder_record_timeslice.html. r?jwwang MozReview-Commit-ID: HFstfYnRyJP
f558878e2449672141fec69b0ce58464a986998f: Bug 1208371 - Wait for "loadedmetadata" before setting up nodes in test_mediaElementAudioSourceNodeCrossOrigin.html. r?padenot draft
Andreas Pehrson <pehrsons@gmail.com> - Tue, 05 Jan 2016 11:17:16 +0800 - rev 342166
Push 13352 by pehrsons@gmail.com at Fri, 18 Mar 2016 13:49:47 +0000
Bug 1208371 - Wait for "loadedmetadata" before setting up nodes in test_mediaElementAudioSourceNodeCrossOrigin.html. r?padenot MozReview-Commit-ID: HruqsknDHi7
9a334eadae40baa2d5f69ba4b60f455500c7d380: Bug 1208371 - Clear output canvas on each drawImage(). r?jib draft
Andreas Pehrson <pehrsons@gmail.com> - Tue, 05 Jan 2016 10:16:32 +0800 - rev 342165
Push 13352 by pehrsons@gmail.com at Fri, 18 Mar 2016 13:49:47 +0000
Bug 1208371 - Clear output canvas on each drawImage(). r?jib Otherwise we'd risk reading old pixels in case we draw something transparent. MozReview-Commit-ID: LjNXE4Rmrmt
517b9deed7065d9318278799c1135b4a3dc1eff8: Bug 1208371 - Don't dispatch conduit destruction to main thread when not needed. r?bwc draft
Andreas Pehrson <pehrsons@gmail.com> - Tue, 05 Jan 2016 10:16:32 +0800 - rev 342164
Push 13352 by pehrsons@gmail.com at Fri, 18 Mar 2016 13:49:47 +0000
Bug 1208371 - Don't dispatch conduit destruction to main thread when not needed. r?bwc In case MediaPipeline gets destructed on main thread we cannot dispatch to main thread. In this case, destroy the conduits directly. The easiest way to test this that I have found is to shut Firefox down in the middle of a PeerConnection mochitest, when the PeerConnection is still active. MozReview-Commit-ID: 3gLhtIKTkx1
f338639b37cf3371012436f1f9ad163b37719f36: Bug 1208371 - Guard LoadManagerSingleton with a WeakPtr. r?pkerr draft
Andreas Pehrson <pehrsons@gmail.com> - Tue, 05 Jan 2016 10:16:32 +0800 - rev 342163
Push 13352 by pehrsons@gmail.com at Fri, 18 Mar 2016 13:49:47 +0000
Bug 1208371 - Guard LoadManagerSingleton with a WeakPtr. r?pkerr LoadManagerSingleton has a separate shutdown path (xpcom-shutdown) from its users (Audio/VideoConduit - garbage collected). These have appeared racy, so in some cases the singleton was destructed before the users had deregistered (e.g., when conduits destructed by SnowWhiteKiller). A WeakPtr can solve this. MozReview-Commit-ID: AVrpd3QqOGx
88eb2907e497528f3d13833782ae4986146f6fdd: Bug 1208371 - Add disconnect method to test helper AudioStreamAnalyser to improve performance on slow devices. r?padenot draft
Andreas Pehrson <pehrsons@gmail.com> - Tue, 05 Jan 2016 10:16:32 +0800 - rev 342162
Push 13352 by pehrsons@gmail.com at Fri, 18 Mar 2016 13:49:47 +0000
Bug 1208371 - Add disconnect method to test helper AudioStreamAnalyser to improve performance on slow devices. r?padenot MozReview-Commit-ID: 4MYF004uK8R
8d33b3ab5ce16ebe13985cd2c698fc04c4891258: Bug 1208371 - Remove debug canvas from mediaStreamConstructors test to help android perf. r?padenot draft
Andreas Pehrson <pehrsons@gmail.com> - Tue, 05 Jan 2016 10:16:32 +0800 - rev 342161
Push 13352 by pehrsons@gmail.com at Fri, 18 Mar 2016 13:49:47 +0000
Bug 1208371 - Remove debug canvas from mediaStreamConstructors test to help android perf. r?padenot MozReview-Commit-ID: 62lDtTvZ5g2
05765fcea4f9ce4f6af95622b76d512cbdb41ee9: Bug 1208371 - Enable test_peerConnection_trackDisabling.html on android 4.3 debug. r?jib draft
Andreas Pehrson <pehrsons@gmail.com> - Tue, 05 Jan 2016 10:16:32 +0800 - rev 342160
Push 13352 by pehrsons@gmail.com at Fri, 18 Mar 2016 13:49:47 +0000
Bug 1208371 - Enable test_peerConnection_trackDisabling.html on android 4.3 debug. r?jib Disabling the audio analyser debug canvas gave us enough perf to enable the test reliably here. MozReview-Commit-ID: AGEfsD4pyME
241934caf9ea136daa4926fa5b5b603f697bbeb9: Bug 1208371 - Remove debug canvas from peerConnection_trackDisabling test to help android perf. r?padenot draft
Andreas Pehrson <pehrsons@gmail.com> - Tue, 05 Jan 2016 10:16:31 +0800 - rev 342159
Push 13352 by pehrsons@gmail.com at Fri, 18 Mar 2016 13:49:47 +0000
Bug 1208371 - Remove debug canvas from peerConnection_trackDisabling test to help android perf. r?padenot MozReview-Commit-ID: 7EWJ5GaO6fK
09e81c59240b19aa0fbe913cf60fbbff1dc1eb10: Bug 1208371 - Remove debug canvas from addTrackRemoveTrack test to help android perf. r?padenot draft
Andreas Pehrson <pehrsons@gmail.com> - Tue, 05 Jan 2016 10:16:31 +0800 - rev 342158
Push 13352 by pehrsons@gmail.com at Fri, 18 Mar 2016 13:49:47 +0000
Bug 1208371 - Remove debug canvas from addTrackRemoveTrack test to help android perf. r?padenot MozReview-Commit-ID: Ejyvk1Vc66k
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 +300000 tip