searching for reviewer(jya)
413220444b15: Bug 1400674 - Fix the algorithm of filling audio gaps with silence. r=jya, a=ritu
JW Wang <jwwang@mozilla.com> - Fri, 13 Oct 2017 17:24:24 +0800 - rev 432595
Push 8003 by ryanvm@gmail.com at 2017-10-18 22:23 +0000
Bug 1400674 - Fix the algorithm of filling audio gaps with silence. r=jya, a=ritu MozReview-Commit-ID: Hjsboq6PdlN
4dd7832a9238: Bug 1405025 - Part 2: Revert Bug 1390443 P1. r=jya, a=ritu
JW Wang <jwwang@mozilla.com> - Fri, 06 Oct 2017 15:58:59 +0800 - rev 432502
Push 7969 by ryanvm@gmail.com at 2017-10-13 22:47 +0000
Bug 1405025 - Part 2: Revert Bug 1390443 P1. r=jya, a=ritu Since we don't use state-mirroring to dispatch nextFrameStatus changes, we can now revert the workaround of bug 1390443 P1. See bug 1390443 comment 0 for more details. MozReview-Commit-ID: FRxXUnGC3x2
57c10a793a80: Bug 1405025 - Part 1: Ensure 'seeking' is fired before 'waiting'. r=jya, a=ritu
JW Wang <jwwang@mozilla.com> - Fri, 06 Oct 2017 15:48:38 +0800 - rev 432501
Push 7969 by ryanvm@gmail.com at 2017-10-13 22:47 +0000
Bug 1405025 - Part 1: Ensure 'seeking' is fired before 'waiting'. r=jya, a=ritu Use MediaEventSource instead of state-mirroring to notify nextFrameStatus changes so we have more control over the order of events. MozReview-Commit-ID: 3DGtMbghEQm
452ca306a258: Bug 1403412 - Disable VP9 estimizer on Mac. r=jya, a=sledru
Kaku Kuo <kaku@mozilla.com> - Thu, 28 Sep 2017 00:17:29 +0800 - rev 432029
Push 7871 by ryanvm@gmail.com at 2017-10-02 14:28 +0000
Bug 1403412 - Disable VP9 estimizer on Mac. r=jya, a=sledru MozReview-Commit-ID: 8RTe7lVSRwl
b108bdcaff9d: Bug 1402681 - Part 2: Add log. r=jya, a=sledru
Alastor Wu <alwu@mozilla.com> - Tue, 26 Sep 2017 17:48:39 +0800 - rev 432019
Push 7870 by ryanvm@gmail.com at 2017-10-02 13:47 +0000
Bug 1402681 - Part 2: Add log. r=jya, a=sledru MozReview-Commit-ID: 8qINxSCKrvg
23eb5e8f9114: Bug 1402681 - Part 1: Do not queue the task without task queue. r=jya, a=sledru
Alastor Wu <alwu@mozilla.com> - Tue, 26 Sep 2017 17:48:19 +0800 - rev 432018
Push 7870 by ryanvm@gmail.com at 2017-10-02 13:47 +0000
Bug 1402681 - Part 1: Do not queue the task without task queue. r=jya, a=sledru Since the source buffer is a wrapped native, it would be unlinked twice and then the TrackBuffersManager::Detach() would also be called twice. The first detach task would clear the task queue of TrackBuffersManager, and then we won't accept any new task without task queue. The second detach task should not be executed. MozReview-Commit-ID: AWTzVbRH5B1
4ff9ba1e5310: Bug 1388125 - CloneableWithRangeMediaResource for streams implementing nsICloneableInputStreamWithRange, r=jya
Andrea Marchesini <amarchesini@mozilla.com> - Mon, 18 Sep 2017 13:34:43 +0200 - rev 430973
Push 7775 by ryanvm@gmail.com at 2017-09-19 01:22 +0000
Bug 1388125 - CloneableWithRangeMediaResource for streams implementing nsICloneableInputStreamWithRange, r=jya
ab042a1a2b84: Bug 1388125 - Move ChannelMediaResource to separate files, r=jya
Andrea Marchesini <amarchesini@mozilla.com> - Mon, 18 Sep 2017 13:33:49 +0200 - rev 430972
Push 7775 by ryanvm@gmail.com at 2017-09-19 01:22 +0000
Bug 1388125 - Move ChannelMediaResource to separate files, r=jya
ab2aef354a6d: Bug 1388125 - Move FileMediaResource to separate files, r=jya
Andrea Marchesini <amarchesini@mozilla.com> - Mon, 18 Sep 2017 13:31:08 +0200 - rev 430971
Push 7775 by ryanvm@gmail.com at 2017-09-19 01:22 +0000
Bug 1388125 - Move FileMediaResource to separate files, r=jya
7d2d46e29a15: Bug 1388125 - FileMediaResource should use BlobImpl->Size() instead of stream::Available() to know the size, r=jya
Andrea Marchesini <amarchesini@mozilla.com> - Mon, 18 Sep 2017 13:31:00 +0200 - rev 430970
Push 7775 by ryanvm@gmail.com at 2017-09-19 01:22 +0000
Bug 1388125 - FileMediaResource should use BlobImpl->Size() instead of stream::Available() to know the size, r=jya
074e1de206f9: Bug 1400758 - part3 : report the error description with InitPromise for ffmpeg decoders. r=jya
Alastor Wu <alwu@mozilla.com> - Mon, 18 Sep 2017 18:10:05 +0800 - rev 430953
Push 7775 by ryanvm@gmail.com at 2017-09-19 01:22 +0000
Bug 1400758 - part3 : report the error description with InitPromise for ffmpeg decoders. r=jya MozReview-Commit-ID: 3MGmU34JY3t
1d0e5cf9e932: Bug 1400758 - part2 : report the error description with InitPromise for agnostic decoders. r=jya
Alastor Wu <alwu@mozilla.com> - Mon, 18 Sep 2017 18:10:00 +0800 - rev 430952
Push 7775 by ryanvm@gmail.com at 2017-09-19 01:22 +0000
Bug 1400758 - part2 : report the error description with InitPromise for agnostic decoders. r=jya MozReview-Commit-ID: BxTTHJPwDNq
67ce16d81d3c: Bug 1400758 - part1 : report the error description with InitPromise for Apple's decoders. r=jya
Alastor Wu <alwu@mozilla.com> - Mon, 18 Sep 2017 18:05:21 +0800 - rev 430951
Push 7775 by ryanvm@gmail.com at 2017-09-19 01:22 +0000
Bug 1400758 - part1 : report the error description with InitPromise for Apple's decoders. r=jya MozReview-Commit-ID: 8rifWLm8BPt
c86dac25f66f: Bug 1394239 - don't return video frames that MDSM won't show. r=jya
John Lin <jolin@mozilla.com> - Mon, 11 Sep 2017 12:50:48 +0800 - rev 430920
Push 7775 by ryanvm@gmail.com at 2017-09-19 01:22 +0000
Bug 1394239 - don't return video frames that MDSM won't show. r=jya In some Android ROMs, MediaCodec doesn't allocate additional buffers to reduce consumer starvation and will not work when MDSM grips most recently returned frame before rearching seek target. Implement SetSeekThreshold() to get actual seek target to check if video buffers can be released back to remote decoder immediately. MozReview-Commit-ID: 7IetuVxCXc0
413dd42f4028: Bug 1400195 - remove xp constraint for tests. r=jya
Alastor Wu <alwu@mozilla.com> - Sat, 16 Sep 2017 02:14:23 +0800 - rev 430698
Push 7768 by ryanvm@gmail.com at 2017-09-16 16:13 +0000
Bug 1400195 - remove xp constraint for tests. r=jya MozReview-Commit-ID: BawMiEVPNFA
69fec8b3a6a0: Bug 1398139 - part4 : enable test. r=jya
Alastor Wu <alwu@mozilla.com> - Fri, 15 Sep 2017 18:05:21 +0800 - rev 430693
Push 7768 by ryanvm@gmail.com at 2017-09-16 16:13 +0000
Bug 1398139 - part4 : enable test. r=jya Fix the fail by patch1, so we can re-enable it. MozReview-Commit-ID: It3JkvQzAdk
39e86cbf34d5: Bug 1398139 - part3 : check current time should be increased correctly in new chained part. r=jya
Alastor Wu <alwu@mozilla.com> - Fri, 15 Sep 2017 18:05:18 +0800 - rev 430692
Push 7768 by ryanvm@gmail.com at 2017-09-16 16:13 +0000
Bug 1398139 - part3 : check current time should be increased correctly in new chained part. r=jya MozReview-Commit-ID: 3ctF8SAVcs9
9fe90be81abb: Bug 1398139 - part2 : adjust sample's start time for chained ogg. r=jya
Alastor Wu <alwu@mozilla.com> - Fri, 15 Sep 2017 18:05:10 +0800 - rev 430691
Push 7768 by ryanvm@gmail.com at 2017-09-16 16:13 +0000
Bug 1398139 - part2 : adjust sample's start time for chained ogg. r=jya For chained ogg files, the new part would contain new timestamp from zero, so we need to add the duration of previously decoded data to make sure the current time is correct. MozReview-Commit-ID: Bb1lCiKz4uQ
784819df6f88: Bug 1397141 - part8 : update test for video under 48x48. r=jya
Alastor Wu <alwu@mozilla.com> - Wed, 13 Sep 2017 15:38:24 +0800 - rev 430024
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1397141 - part8 : update test for video under 48x48. r=jya In patch3, we remove the minimum resolution check, now the video under 48x48 can be playback successfully. Therefore, removing them from error test and we should ensure they can be playback. MozReview-Commit-ID: BvLtr4DN1hU
578f746a2721: Bug 1397141 - part7 : update error description in MFR. r=jya
Alastor Wu <alwu@mozilla.com> - Wed, 13 Sep 2017 15:38:06 +0800 - rev 430023
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1397141 - part7 : update error description in MFR. r=jya MozReview-Commit-ID: 9Sb5ogX2Bf2
14249cad9e8e: Bug 1397141 - part6 : use MediaResult to replace nsresult r=jya
Alastor Wu <alwu@mozilla.com> - Wed, 13 Sep 2017 15:37:50 +0800 - rev 430022
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1397141 - part6 : use MediaResult to replace nsresult r=jya Return MediaResult instead of using nsresult, because it can contain more detailed error information. We could also return this error with our rejected decode promise. MozReview-Commit-ID: 80yEAbxqvWu
5af7f08bcaab: Bug 1397141 - part5 : update error description from GPU process. r=jya,mattwoodrow
Alastor Wu <alwu@mozilla.com> - Wed, 13 Sep 2017 15:37:24 +0800 - rev 430021
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1397141 - part5 : update error description from GPU process. r=jya,mattwoodrow MozReview-Commit-ID: 9aKyYftBnUo
57edd7246a04: Bug 1397141 - part4 : change mLastError type to MediaResult r=jya
Alastor Wu <alwu@mozilla.com> - Wed, 13 Sep 2017 15:05:52 +0800 - rev 430020
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1397141 - part4 : change mLastError type to MediaResult r=jya Change mLastError type to MediaResult and send it as parameter to PDM::CreateVideoDecoder in order to get detailed error description. MozReview-Commit-ID: 4sIRXTHsrzr
81a7bf2ee098: Bug 1397141 - part3 : remove the minimum resolution check. r=jya
Alastor Wu <alwu@mozilla.com> - Wed, 13 Sep 2017 15:05:45 +0800 - rev 430019
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1397141 - part3 : remove the minimum resolution check. r=jya After bug 1392143, we won't enable HW decoding for the resolution < 132 pixels. In addition, software decoder doesn't have the minimum resolution limit, so we can remove the minimum resolution check. MozReview-Commit-ID: 7MiLpwjiq3s
f67f016f5688: Bug 1397141 - part2 : move resolution constraints to WMFVideoMFTManager r=jya
Alastor Wu <alwu@mozilla.com> - Wed, 13 Sep 2017 15:05:37 +0800 - rev 430018
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1397141 - part2 : move resolution constraints to WMFVideoMFTManager r=jya WMFDecoderModule should only focus on whether the mime type is supported or not. Let WMFVideoMFTManager do the checking. MozReview-Commit-ID: K6jPfrntu7s
81e046d67cae: Bug 1397141 - part1 : update error description for getting more details r=jya
Alastor Wu <alwu@mozilla.com> - Wed, 13 Sep 2017 15:05:25 +0800 - rev 430017
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1397141 - part1 : update error description for getting more details r=jya We should report the more detailed error when creating the decoder failed, instead of just reporting "can not create decoder". MozReview-Commit-ID: 8vunP5c3zzI
a13295273ef7: Bug 1399359 - Enable clang-format option SplitEmptyFunction to have empty constructor correctly placed r=andi,jya
Sylvestre Ledru <sledru@mozilla.com> - Wed, 13 Sep 2017 08:39:43 +0200 - rev 430016
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1399359 - Enable clang-format option SplitEmptyFunction to have empty constructor correctly placed r=andi,jya MozReview-Commit-ID: 9tRSMN89iuM
f6f5863e50a8: Bug 1397141 - part8 : update test for video under 48x48. r=jya
Alastor Wu <alwu@mozilla.com> - Wed, 13 Sep 2017 15:06:22 +0800 - rev 430000
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1397141 - part8 : update test for video under 48x48. r=jya In patch3, we remove the minimum resolution check, now the video under 48x48 can be playback successfully. Therefore, removing them from error test and we should ensure they can be playback. MozReview-Commit-ID: BvLtr4DN1hU
eab2e401c60f: Bug 1397141 - part7 : update error description in MFR. r=jya
Alastor Wu <alwu@mozilla.com> - Wed, 13 Sep 2017 15:06:14 +0800 - rev 429999
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1397141 - part7 : update error description in MFR. r=jya MozReview-Commit-ID: 5QPdFCMesi5
c78c096d0be9: Bug 1397141 - part6 : use MediaResult to replace nsresult r=jya
Alastor Wu <alwu@mozilla.com> - Wed, 13 Sep 2017 15:06:07 +0800 - rev 429998
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1397141 - part6 : use MediaResult to replace nsresult r=jya Return MediaResult instead of using nsresult, because it can contain more detailed error information. We could also return this error with our rejected decode promise. MozReview-Commit-ID: HrI3QKlSJC
6dade48b3326: Bug 1397141 - part5 : update error description from GPU process. r=jya,mattwoodrow
Alastor Wu <alwu@mozilla.com> - Wed, 13 Sep 2017 15:05:59 +0800 - rev 429997
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1397141 - part5 : update error description from GPU process. r=jya,mattwoodrow MozReview-Commit-ID: CuBZOtjh4g2
9140f22570ea: Bug 1397141 - part4 : change mLastError type to MediaResult r=jya
Alastor Wu <alwu@mozilla.com> - Wed, 13 Sep 2017 15:05:52 +0800 - rev 429996
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1397141 - part4 : change mLastError type to MediaResult r=jya Change mLastError type to MediaResult and send it as parameter to PDM::CreateVideoDecoder in order to get detailed error description. MozReview-Commit-ID: 4sIRXTHsrzr
57ab4c51c72b: Bug 1397141 - part3 : remove the minimum resolution check. r=jya
Alastor Wu <alwu@mozilla.com> - Wed, 13 Sep 2017 15:05:45 +0800 - rev 429995
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1397141 - part3 : remove the minimum resolution check. r=jya After bug 1392143, we won't enable HW decoding for the resolution < 132 pixels. In addition, software decoder doesn't have the minimum resolution limit, so we can remove the minimum resolution check. MozReview-Commit-ID: 7MiLpwjiq3s
81a987374ac7: Bug 1397141 - part2 : move resolution constraints to WMFVideoMFTManager r=jya
Alastor Wu <alwu@mozilla.com> - Wed, 13 Sep 2017 15:05:37 +0800 - rev 429994
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1397141 - part2 : move resolution constraints to WMFVideoMFTManager r=jya WMFDecoderModule should only focus on whether the mime type is supported or not. Let WMFVideoMFTManager do the checking. MozReview-Commit-ID: K6jPfrntu7s
6f22cc93383c: Bug 1397141 - part1 : update error description for getting more details r=jya
Alastor Wu <alwu@mozilla.com> - Wed, 13 Sep 2017 15:05:25 +0800 - rev 429993
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1397141 - part1 : update error description for getting more details r=jya We should report the more detailed error when creating the decoder failed, instead of just reporting "can not create decoder". MozReview-Commit-ID: 8vunP5c3zzI
9f16ba6aac28: Bug 1396547 - use Supports() in PDMFactory::CreateDecoder() instread of SupportsMimeType(). r=jya
Alastor Wu <alwu@mozilla.com> - Mon, 04 Sep 2017 18:50:23 +0800 - rev 428591
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1396547 - use Supports() in PDMFactory::CreateDecoder() instread of SupportsMimeType(). r=jya The TrackInfo [1] created in WMFDecoderModule::SupportsMimeType() doesn't contain valid image's width and height, because the TrackInfo is created without width and height [2] and the default width and height are both -1 [3]. Thesefore, we can't correctly check whether this resolution is supported by MFT [4]. We should use Supports() instead of SupportsMimeType(). [1] https://goo.gl/QV8Jgm [2] https://goo.gl/4siShn [3] https://goo.gl/BDoXYf [4] https://goo.gl/BZh4QA MozReview-Commit-ID: 4dIJ84eaytq
3ee078cdd897: Bug 1396428 - Fix end of lines. r=jya
Jean-Yves Avenard <jyavenard@mozilla.com> - Sun, 03 Sep 2017 21:28:52 +0200 - rev 427994
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1396428 - Fix end of lines. r=jya MozReview-Commit-ID: 3reYozTMKk0
19d7bf4e0708: Bug 1395014 - remove MediaSourceResource. r=jya
JW Wang <jwwang@mozilla.com> - Tue, 29 Aug 2017 14:03:19 +0800 - rev 427349
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1395014 - remove MediaSourceResource. r=jya MozReview-Commit-ID: D3W2u4MBize
59834f182e67: Bug 1393336 - Remove WaitForCDM state from MDSM. r=jwwang,jya
Kilik Kuo <kikuo@mozilla.com> - Fri, 25 Aug 2017 17:11:14 +0800 - rev 426876
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1393336 - Remove WaitForCDM state from MDSM. r=jwwang,jya MozReview-Commit-ID: 7XofvBZTerH
8514e013efd3: Bug 1394653 - remove MediaResource from the base class of MediaSourceResource. r=jya
JW Wang <jwwang@mozilla.com> - Fri, 25 Aug 2017 14:08:07 +0800 - rev 426854
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1394653 - remove MediaResource from the base class of MediaSourceResource. r=jya MozReview-Commit-ID: 9Pmp7K6zp13
1502253e5b92: Bug 1378070 - Update AudioConverter to use destination buffer. r=jya
Alex Chronopoulos <achronop@gmail.com> - Mon, 28 Aug 2017 17:15:46 +0300 - rev 426758
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1378070 - Update AudioConverter to use destination buffer. r=jya MozReview-Commit-ID: KCheY0DCjh4
ac74aef901ed: Bug 1392498 - Move TimeIntervals to dom::TimeRanges conversion into TimeRanges class. r=jya
Chris Pearce <cpearce@mozilla.com> - Mon, 21 Aug 2017 15:08:25 +1200 - rev 426628
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1392498 - Move TimeIntervals to dom::TimeRanges conversion into TimeRanges class. r=jya If TimeUnits.h includes mozilla/dom/TimeRanges.h, then the build ends up pulling in the Gecko DOM bindings, which pulls in a whole lot of JavaScript and DOM bindings code. That makes it trickier to import GeckoMedia into Servo, and makes Gecko's build slower, so move the code to convert TimeIntervals into dom::TimeRanges. Also remove an extraneous "virtual" and add "const" to some functions in TimeRanges. MozReview-Commit-ID: BLeehaf9gCE
11c32b6a01e8: Bug 1390406 - Remove unnecessary includes in dom/media/mediasource. r=jya
Chris Pearce <cpearce@mozilla.com> - Tue, 15 Aug 2017 16:13:00 +1200 - rev 424732
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1390406 - Remove unnecessary includes in dom/media/mediasource. r=jya MozReview-Commit-ID: 1aTncGfBicu
ccc818d08d1e: Bug 1390617 Fix casing of d3d11.h to fix MinGW build r=jya
Tom Ritter <tom@mozilla.com> - Tue, 15 Aug 2017 15:14:25 -0500 - rev 424676
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1390617 Fix casing of d3d11.h to fix MinGW build r=jya MozReview-Commit-ID: 8IHb4pYo77Y
44619504c9ee: Bug 1388288 - Make TrackBuffersManager compile in non-Unified mode. r=jya
Chris Pearce <cpearce@mozilla.com> - Tue, 08 Aug 2017 15:02:20 +0800 - rev 423109
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1388288 - Make TrackBuffersManager compile in non-Unified mode. r=jya TrackBuffersManager::SegmentParserLoop() fails to compile in non-Unified mode because some uses of SourceBufferAttributes::AppendState don't have the SourceBufferAttributes:: prefix. So just add a typedef to make that unnecessary. MozReview-Commit-ID: 1H3SIeUxNu7
4c166d879019: Bug 1388288 - Make Flac and Ogg decoders build in non-Unified mode. r=jya
Chris Pearce <cpearce@mozilla.com> - Tue, 08 Aug 2017 15:46:37 +0800 - rev 423106
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1388288 - Make Flac and Ogg decoders build in non-Unified mode. r=jya FLAC_MAX_CHANNELS is used in FlacDemuxer and in FrameParser, so move it, and all the other FLAC_{MIN,MAX}* #defines to FrameParser.h. The other defines don't technically need to be there, but it's nicer to have them all defined together. MozReview-Commit-ID: LVZzXvBSvMq
74a5b8b6715f: Bug 1388288 - Make dom/media/platforms build in non-unified mode. r=jya
Chris Pearce <cpearce@mozilla.com> - Tue, 08 Aug 2017 14:35:36 +0800 - rev 423105
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1388288 - Make dom/media/platforms build in non-unified mode. r=jya MozReview-Commit-ID: 43uEIDmfCMR
be090e170cc6: Bug 1386951 - have ChannelMediaDecoder implement CreateStateMachine() and fix includes. r=jya
JW Wang <jwwang@mozilla.com> - Thu, 03 Aug 2017 14:05:28 +0800 - rev 422325
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1386951 - have ChannelMediaDecoder implement CreateStateMachine() and fix includes. r=jya The implementation will be shared by most of the sub-classes except OggDecoder which needs to call demuxer->SetChainingEvents(). http://searchfox.org/mozilla-central/rev/f0e4ae5f8c40ba742214e89aba3f554da0b89a33/dom/media/ogg/OggDecoder.cpp#25 This helps reducing code changes whenever we add a field to MediaFormatReaderInit. MozReview-Commit-ID: 5K8NY1oxol4
d607b1cb6d58: Bug 1386478 - don't cap the return value of GetEndTime(). r=jya
JW Wang <jwwang@mozilla.com> - Wed, 02 Aug 2017 10:45:48 +0800 - rev 422034
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1386478 - don't cap the return value of GetEndTime(). r=jya A chained ogg file contains multiple streams and timestamps will not monolithically increase. The current code will cap AudioEndTime() to the end time of the 1st stream and currentTime won't be updated correctly when playback position reaches the 2nd stream or later ones. http://searchfox.org/mozilla-central/rev/bbc1c59e460a27b20929b56489e2e55438de81fa/dom/media/MediaDecoderStateMachine.cpp#3636-3637 MozReview-Commit-ID: DbIspYmEjVv
664aeccfc35b: Bug 1385305 - Change `CanBeCaptured` and friends so that they use explicit and self-explaining constants. r=jya
Paul Adenot <paul@paul.cx> - Fri, 28 Jul 2017 17:53:12 +0200 - rev 421918
Push 7761 by jlund@mozilla.com at 2017-09-15 00:19 +0000
Bug 1385305 - Change `CanBeCaptured` and friends so that they use explicit and self-explaining constants. r=jya MozReview-Commit-ID: Hpuxlc8Ib5j