5b0cffc026a4c17283e64c93d885df411b62ae4d: Bug 1404997 - P25. Add MSG and WebRTC MediaThreadType. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 15 Dec 2017 18:39:33 +0100 - rev 396671
Push 33104 by nerli@mozilla.com at Sun, 17 Dec 2017 21:43:51 +0000
Bug 1404997 - P25. Add MSG and WebRTC MediaThreadType. r=gerald Additionally, remove no longer relevant MediaThreadType documentation as all MediaDataDecoder API are now asynchronous and we no longer have cancellable taskqueues. MozReview-Commit-ID: 1F0YUhNniAn
32c31f7924afe3a8bf441a5da89634af218b7e63: Bug 1404997 - P24. Make AutoTaskQueue deliver runnables to nsIEventTarget. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 15 Dec 2017 18:14:02 +0100 - rev 396670
Push 33104 by nerli@mozilla.com at Sun, 17 Dec 2017 21:43:51 +0000
Bug 1404997 - P24. Make AutoTaskQueue deliver runnables to nsIEventTarget. r=gerald Similar to bug 1300118 for TaskQueue. MozReview-Commit-ID: 8jIifvs2r4U
a695b9c49eda544cef788728505b4a0e985d1fce: Bug 1404997 - P23. Strongly enforced that our destination buffer is big enough. r=padenot
Jean-Yves Avenard <jyavenard@mozilla.com> - Wed, 13 Dec 2017 18:44:30 +0100 - rev 396669
Push 33104 by nerli@mozilla.com at Sun, 17 Dec 2017 21:43:51 +0000
Bug 1404997 - P23. Strongly enforced that our destination buffer is big enough. r=padenot MozReview-Commit-ID: A1kLsH75SzX
09a1076a2a9fa173923d293b6496ca812e49d142: Bug 1404997 - P22. Make mConduit release on main thread consistent. r=pehrsons
Jean-Yves Avenard <jyavenard@mozilla.com> - Mon, 11 Dec 2017 00:32:00 +0100 - rev 396668
Push 33104 by nerli@mozilla.com at Sun, 17 Dec 2017 21:43:51 +0000
Bug 1404997 - P22. Make mConduit release on main thread consistent. r=pehrsons There's no need for a custom class to perform this task. MozReview-Commit-ID: JxpDQVM97fl
4b0c8c6ed4ccd0ae02d4540b5727428f4ebbe07c: Bug 1404997 - P21. Make MediaPipelineReceiveAudio listener asynchronous. r=pehrsons
Jean-Yves Avenard <jyavenard@mozilla.com> - Mon, 11 Dec 2017 00:16:24 +0100 - rev 396667
Push 33104 by nerli@mozilla.com at Sun, 17 Dec 2017 21:43:51 +0000
Bug 1404997 - P21. Make MediaPipelineReceiveAudio listener asynchronous. r=pehrsons We keep the synchronous version that will be used in bug 1424653 MozReview-Commit-ID: JTGaRYm20ca
c27eaf506c73c88f90d5b1966046b4e173d00825: Bug 1404997 - P20. Make MediaStreamListener::NotifyPull asynchronous. r=padenot
Jean-Yves Avenard <jyavenard@mozilla.com> - Sun, 10 Dec 2017 21:33:43 +0100 - rev 396666
Push 33104 by nerli@mozilla.com at Sun, 17 Dec 2017 21:43:51 +0000
Bug 1404997 - P20. Make MediaStreamListener::NotifyPull asynchronous. r=padenot The operations is done in two ways: 1- Process all the MediaStreamListener at once, which returns a promise that will be resolved once the operation is completed. 2- As the Cubeb audio callback must be resolved immediately, the MSG will wait for all the promises to be resolved until it continues the operation of feeding the callback the necessary data. This will allow to parallelize the stream's tracks' audio decoding. MozReview-Commit-ID: EeoDvxnJyWV
f9b7dc9f7e08bca6862b1706660b042ec1ccbe4b: Bug 1404997 - P19. Use new Await method with WebrtcMediaDataDecoder. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 08 Dec 2017 17:30:28 +0100 - rev 396665
Push 33104 by nerli@mozilla.com at Sun, 17 Dec 2017 21:43:51 +0000
Bug 1404997 - P19. Use new Await method with WebrtcMediaDataDecoder. r=gerald MozReview-Commit-ID: nRQL312aDE
737f227413b71fa1df005fbbaf1b3372547dcae7: Bug 1404997 - P18. Add Await convenience methods. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 08 Dec 2017 13:35:29 +0100 - rev 396664
Push 33104 by nerli@mozilla.com at Sun, 17 Dec 2017 21:43:51 +0000
Bug 1404997 - P18. Add Await convenience methods. r=gerald Takes either a MozPromise or an AllPromiseType and will execute the resolve/reject functions synchronously once the promise has resolved/rejected. MozReview-Commit-ID: EyfMTPtA1Lu
038600d1d2c893a2ba3a389bb477f43363366a9b: Bug 1404997 - P17. Split ExtractPendingInput into two methods. r=padenot
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 07 Dec 2017 20:52:56 +0100 - rev 396663
Push 33104 by nerli@mozilla.com at Sun, 17 Dec 2017 21:43:51 +0000
Bug 1404997 - P17. Split ExtractPendingInput into two methods. r=padenot There were two steps happening inside ExtractPendingInput: 1- Retrieve the data from the StreamTracks 2- Process any pending pending states change We split it so that the retrieval from the StreamTrack can be promisified in an upcoming change MozReview-Commit-ID: 53O4fXWMDGL
c67e47661a426965217e433f2c1c36e95318d917: Bug 1404997 - P16. Properly finish the SourceMediaStream during shutdown. r=padenot
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 07 Dec 2017 20:20:53 +0100 - rev 396662
Push 33104 by nerli@mozilla.com at Sun, 17 Dec 2017 21:43:51 +0000
Bug 1404997 - P16. Properly finish the SourceMediaStream during shutdown. r=padenot Despite the name of the function, the original SourceMediaStream::Finish() (consequently renamed FinishPending) didn't actually finished the stream, but instead set a bool that would indicate to completely finish the stream once ExtractPendingInput ran. But here it could never run again. So actually do what the original fix intended to do (bug 1410829) MozReview-Commit-ID: 1hHiOLiovG
336263c26cd28b476c8cc5f97b369696e9db6274: Bug 1404997 - P15. Move MSG::FinishStream logic to MediaStream. r=padenot
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 07 Dec 2017 16:12:19 +0100 - rev 396661
Push 33104 by nerli@mozilla.com at Sun, 17 Dec 2017 21:43:51 +0000
Bug 1404997 - P15. Move MSG::FinishStream logic to MediaStream. r=padenot It is good practice for the MSG to now know the implementation details of the MediaStream. Additionally, this will allow to make a thread-safe version later., MozReview-Commit-ID: CTacCLSeKRP
f212c87ea00327ae1a06c98ddc502ae60c72efcb: Bug 1404997 - P14. Rename members to clarify the finish meaning. r=padenot
Jean-Yves Avenard <jyavenard@mozilla.com> - Wed, 06 Dec 2017 19:00:22 +0100 - rev 396660
Push 33104 by nerli@mozilla.com at Sun, 17 Dec 2017 21:43:51 +0000
Bug 1404997 - P14. Rename members to clarify the finish meaning. r=padenot We have different concept of "finish" between the base class and its hierarchy. Attempt to clear the sitatuation by renaming the members and related methods. MozReview-Commit-ID: vFsXhMK5GY
169cd2016bab36bc61810be0fefa8c7bd70bb49c: Bug 1404997 - P13. Move ExtractPendingInput logic to SourceMediaStream. r=padenot
Jean-Yves Avenard <jyavenard@mozilla.com> - Wed, 06 Dec 2017 16:55:56 +0100 - rev 396659
Push 33104 by nerli@mozilla.com at Sun, 17 Dec 2017 21:43:51 +0000
Bug 1404997 - P13. Move ExtractPendingInput logic to SourceMediaStream. r=padenot The MSG shouldn't have to know about the inner details of the SourceMediaStream MozReview-Commit-ID: 2S81SPzy09E
bafa97eff7a7901a1fa55f8aec814a1a57f0b49d: Bug 1404997 - P12. Remove unused TrackAddedCallback class. r=pehrsons
Jean-Yves Avenard <jyavenard@mozilla.com> - Tue, 05 Dec 2017 11:38:11 +0100 - rev 396658
Push 33104 by nerli@mozilla.com at Sun, 17 Dec 2017 21:43:51 +0000
Bug 1404997 - P12. Remove unused TrackAddedCallback class. r=pehrsons MozReview-Commit-ID: HGe6RjwwC5z
3d3bfe24beae7b14d1f72c11776873a51a9588a3: Bug 1404997 - P11. More constness fixes. r=pehrsons
Jean-Yves Avenard <jyavenard@mozilla.com> - Tue, 05 Dec 2017 00:07:52 +0100 - rev 396657
Push 33104 by nerli@mozilla.com at Sun, 17 Dec 2017 21:43:51 +0000
Bug 1404997 - P11. More constness fixes. r=pehrsons MozReview-Commit-ID: 5qgO43V9R9P
df9c45f285365c709fe5445871e368da3179dfdb: Bug 1404997 - P9. Remove unused member and accessor. r=pehrsons
Jean-Yves Avenard <jyavenard@mozilla.com> - Sun, 17 Dec 2017 10:15:59 +0100 - rev 396656
Push 33104 by nerli@mozilla.com at Sun, 17 Dec 2017 21:43:51 +0000
Bug 1404997 - P9. Remove unused member and accessor. r=pehrsons MozReview-Commit-ID: HAO4NTXHJp1
54fb95187c203351a5c75125a0913585f8b8795d: Bug 1404997 - P8. Follow coding style for members and methods. r=drno,padenot
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 08 Dec 2017 21:32:38 +0100 - rev 396655
Push 33104 by nerli@mozilla.com at Sun, 17 Dec 2017 21:43:51 +0000
Bug 1404997 - P8. Follow coding style for members and methods. r=drno,padenot It allows to more easily distinguish between methods from upstream webrtc.org and our code. MozReview-Commit-ID: ILQhEAYbSmc
e83401dbf3498b7a09f2857e90f7caa0f0c66bf7: Bug 1404997 - P7. Simplify played time calculations. r=pehrsons
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 08 Dec 2017 20:58:09 +0100 - rev 396654
Push 33104 by nerli@mozilla.com at Sun, 17 Dec 2017 21:43:51 +0000
Bug 1404997 - P7. Simplify played time calculations. r=pehrsons Now that the graph rate match the one out of NetEQ, we can remove an unecessary conversion. Additionally, move a member from the base case to the only one where it's used. MozReview-Commit-ID: II5mdcl0vhK
b299ba22a6a210cc4fa270e58a1846a4c0a92c08: Bug 1404997 - P6. Fix constness were applicable. r=pehrsons
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 30 Nov 2017 20:48:48 +0100 - rev 396653
Push 33104 by nerli@mozilla.com at Sun, 17 Dec 2017 21:43:51 +0000
Bug 1404997 - P6. Fix constness were applicable. r=pehrsons MozReview-Commit-ID: JPlZpRz4A9w
40967b7fe56bec7cf2f70f81d88c0244fb9a61e6: Bug 1404997 - P5. Fix constness and remove redundant virtual keyword. r=mattwoodrow
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 30 Nov 2017 19:03:02 +0100 - rev 396652
Push 33104 by nerli@mozilla.com at Sun, 17 Dec 2017 21:43:51 +0000
Bug 1404997 - P5. Fix constness and remove redundant virtual keyword. r=mattwoodrow Additionally, various C++ code and style cleanup MozReview-Commit-ID: 7BCtQzFGaOc
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 tip