searching for reviewer(cpearce)
8579d0e08d4a: Bug 1522923 - Turn off blocking autoplay on Beta. r=cpearce
Alastor Wu <alwu@mozilla.com> - Tue, 19 Feb 2019 20:28:25 +0000 - rev 460301
Push 35590 by rgurzau@mozilla.com at Fri, 22 Feb 2019 05:26:22 +0000
Bug 1522923 - Turn off blocking autoplay on Beta. r=cpearce We would enable blocking autoplay feature via roll-out (Normandy), not just fliping on the pref. Therefore, we would turn off the pref on Beta on Feb 25th, which gives us 4 weeks on and 3 weeks off on Beta. And then use Normandy to turn the feature on step by step. Differential Revision: https://phabricator.services.mozilla.com/D20329
204a7782dc0b: Bug 1516598 - remove 'feature-policy' checking for autoplay policy. r=cpearce,baku
Alastor Wu <alwu@mozilla.com> - Tue, 12 Feb 2019 18:21:58 +0000 - rev 458775
Push 35548 by opoprus@mozilla.com at Wed, 13 Feb 2019 09:48:26 +0000
Bug 1516598 - remove 'feature-policy' checking for autoplay policy. r=cpearce,baku From the following table, we can know the feature policy have no any effect for the autoplay result. That is, if the document has been gesture activated, we want to playback, regardless of the status of the feature policy. If the site has denied autoplay permission via feature policy, if the user gesture activates (clicks play) we still want to be able to play. Therefore, we can remove `feature-policy` checking. | gesture activated | autoplay feature policy status | allowed to play | |-------------------|--------------------------------|-----------------| | activated | allowed | true | | not activated | not allowed | false | | activated | not allowed | true | | not activated | allowed | false | Differential Revision: https://phabricator.services.mozilla.com/D18098
9452ba2d9657: Bug 1523013 - Clear SamplesWaitingForKey::mProxy in Shutdown(). r=cpearce
Andrew McCreight <continuation@gmail.com> - Wed, 30 Jan 2019 22:56:21 +0000 - rev 456151
Push 35472 by dvarga@mozilla.com at Thu, 31 Jan 2019 04:28:45 +0000
Bug 1523013 - Clear SamplesWaitingForKey::mProxy in Shutdown(). r=cpearce There's a strong cycle of references between SamplesWaitingForKey and CDMProxy that does not get cleared unless keys actually come in. This causes a leak. One way to avoid that leak is to clear the proxy reference when the things holding the SamplesWaitingForKey is shut down. Differential Revision: https://phabricator.services.mozilla.com/D17960
349af469066d: Bug 1487844 - turn on the blocking-autoplay by default for all branches. r=cpearce
alwu <alwu@mozilla.com> - Wed, 23 Jan 2019 01:55:17 +0000 - rev 454935
Push 35419 by dluca@mozilla.com at Wed, 23 Jan 2019 16:31:25 +0000
Bug 1487844 - turn on the blocking-autoplay by default for all branches. r=cpearce Differential Revision: https://phabricator.services.mozilla.com/D17315
89875cd4d942: Bug 1520361 - Add Telemetry to know how people add site to the exception list. r=smaug,cpearce,janerik
Alastor Wu <alwu@mozilla.com> - Tue, 22 Jan 2019 18:22:44 +0000 - rev 454835
Push 35417 by rgurzau@mozilla.com at Tue, 22 Jan 2019 21:53:49 +0000
Bug 1520361 - Add Telemetry to know how people add site to the exception list. r=smaug,cpearce,janerik Add Telemetry to know the number of times user adds site to allow list, remove site from allow list, add site to block list and remove site from block list. Differential Revision: https://phabricator.services.mozilla.com/D16642
1c1ef238dc17: Bug 1520663 - enable blocking autoplay with user-gesture-activation model on Fennec. r=cpearce
alwu <alwu@mozilla.com> - Thu, 17 Jan 2019 22:24:37 +0000 - rev 454380
Push 35395 by ebalazs@mozilla.com at Fri, 18 Jan 2019 09:38:33 +0000
Bug 1520663 - enable blocking autoplay with user-gesture-activation model on Fennec. r=cpearce Differential Revision: https://phabricator.services.mozilla.com/D16773
177103817494: Bug 1520457 - Adjust the message written in the console when the auto-play policy block an AudioContext. r=flod,cpearce
Paul Adenot <paul@paul.cx> - Wed, 16 Jan 2019 22:14:08 +0000 - rev 454254
Push 35390 by shindli@mozilla.com at Thu, 17 Jan 2019 16:13:20 +0000
Bug 1520457 - Adjust the message written in the console when the auto-play policy block an AudioContext. r=flod,cpearce Differential Revision: https://phabricator.services.mozilla.com/D16692
5a384547b8b3: Bug 1520088 - part5 - add pref to control whether blacklist can override the default setting. r=cpearce
Alastor Wu <alwu@mozilla.com> - Wed, 16 Jan 2019 03:57:11 +0000 - rev 454150
Push 35387 by aiakab@mozilla.com at Thu, 17 Jan 2019 04:19:37 +0000
Bug 1520088 - part5 - add pref to control whether blacklist can override the default setting. r=cpearce Differential Revision: https://phabricator.services.mozilla.com/D16640
5e0eb1c00744: Bug 1520088 - part4 : modify test. r=cpearce
Alastor Wu <alwu@mozilla.com> - Wed, 16 Jan 2019 02:17:56 +0000 - rev 454149
Push 35387 by aiakab@mozilla.com at Thu, 17 Jan 2019 04:19:37 +0000
Bug 1520088 - part4 : modify test. r=cpearce Differential Revision: https://phabricator.services.mozilla.com/D16581
2ce9b482cda2: Bug 1520088 - part3 : implement different phases autoplay checking. r=cpearce
Alastor Wu <alwu@mozilla.com> - Wed, 16 Jan 2019 18:21:39 +0000 - rev 454148
Push 35387 by aiakab@mozilla.com at Thu, 17 Jan 2019 04:19:37 +0000
Bug 1520088 - part3 : implement different phases autoplay checking. r=cpearce The autoplay checking for media element has 4 different phases, 1. check whether media element itself meets the autoplay condition 2. check whethr the site is in the autoplay whitelist 3. check global autoplay setting and check wether the site is in the autoplay blacklist. 4. check whether media is allowed under current blocking model (click-to-play or user-gesture-activation) Differential Revision: https://phabricator.services.mozilla.com/D16525
1d2fcf021b32: Bug 1520088 - part2 : ensure logging result can always be executed. r=cpearce
Alastor Wu <alwu@mozilla.com> - Wed, 16 Jan 2019 01:04:40 +0000 - rev 454147
Push 35387 by aiakab@mozilla.com at Thu, 17 Jan 2019 04:19:37 +0000
Bug 1520088 - part2 : ensure logging result can always be executed. r=cpearce Ensure we can always see the debug log for the autoplay result. Differential Revision: https://phabricator.services.mozilla.com/D16524
eb086e6c019f: Bug 1520088 - part1 : let 'IsMediaElementAllowedToPlay()' only check media element's attributes r=cpearce
Alastor Wu <alwu@mozilla.com> - Wed, 16 Jan 2019 01:04:27 +0000 - rev 454146
Push 35387 by aiakab@mozilla.com at Thu, 17 Jan 2019 04:19:37 +0000
Bug 1520088 - part1 : let 'IsMediaElementAllowedToPlay()' only check media element's attributes r=cpearce We would like to have different phases checking for autoplay, the first phase is to check media element itself, so we need to move other non-related checkings out from 'IsMediaElementAllowedToPlay()'. Differential Revision: https://phabricator.services.mozilla.com/D16523
3420cf8f79a9: Bug 1506290 - part2 : add test. r=cpearce
Alastor Wu <alwu@mozilla.com> - Mon, 14 Jan 2019 23:47:35 +0000 - rev 453891
Push 35378 by btara@mozilla.com at Tue, 15 Jan 2019 16:08:45 +0000
Bug 1506290 - part2 : add test. r=cpearce Differential Revision: https://phabricator.services.mozilla.com/D15451
63c3828351f0: Bug 1506290 - part1 : implement 'document.autoplayPolicy'. r=baku,cpearce
alwu <alwu@mozilla.com> - Tue, 15 Jan 2019 00:13:34 +0000 - rev 453890
Push 35378 by btara@mozilla.com at Tue, 15 Jan 2019 16:08:45 +0000
Bug 1506290 - part1 : implement 'document.autoplayPolicy'. r=baku,cpearce `document.autoplayPolicy` returns a enum string that can change overtime based on user session activity: - “allowed” if autoplay is currently allowed. - “allowed-muted” if muted video autoplay is currently allowed. - “disallowed” is autoplay is not current allowed. Differential Revision: https://phabricator.services.mozilla.com/D11543
36ec37a68035: Bug 1516660 - Add pattern encryption params to GMP IPDL and plumb to the CDM. r=cpearce
Bryce Van Dyk <bvandyk@mozilla.com> - Mon, 14 Jan 2019 20:24:57 +0000 - rev 453808
Push 35374 by cbrindusan@mozilla.com at Tue, 15 Jan 2019 04:37:11 +0000
Bug 1516660 - Add pattern encryption params to GMP IPDL and plumb to the CDM. r=cpearce Note, we only pass the relevant IV across the IPC boundry. I.e. if the crypto scheme is cenc we do not pass a constant IV (this is only used by cbcs), and only pass per sample IVs. For cbcs we do the converse. This means in the CDM child we're only receiving one IV, which should be appropriate for whatever scheme (this is similar to how Chromium handle IVs being passed to the CDM). The CDM child side now writes pattern information to samples it's preparing for CDM. With these changes we should be passing all the information required to handle cbcs to the CDM. Differential Revision: https://phabricator.services.mozilla.com/D16459
041c4580aa6f: Bug 1516669 - Update fake-cdm to use CDM10 interface. r=cpearce
Bryce Van Dyk <bvandyk@mozilla.com> - Mon, 14 Jan 2019 16:41:55 +0000 - rev 453764
Push 35372 by cbrindusan@mozilla.com at Mon, 14 Jan 2019 21:49:33 +0000
Bug 1516669 - Update fake-cdm to use CDM10 interface. r=cpearce Differential Revision: https://phabricator.services.mozilla.com/D15520
bd1e99a6325a: Bug 1516669 - Check init value of clearkey WMF decoder for failure. r=cpearce
Bryce Van Dyk <bvandyk@mozilla.com> - Mon, 14 Jan 2019 16:40:29 +0000 - rev 453763
Push 35372 by cbrindusan@mozilla.com at Mon, 14 Jan 2019 21:49:33 +0000
Bug 1516669 - Check init value of clearkey WMF decoder for failure. r=cpearce We weren't checking this value and it was causing compiler warnings. Instead this change means we check the value and log an error if init has failed. Differential Revision: https://phabricator.services.mozilla.com/D15519
b430ac03ce29: Bug 1516669 - Update gmp-clearkey to use CDM10 interface. r=cpearce
Bryce Van Dyk <bvandyk@mozilla.com> - Mon, 14 Jan 2019 16:40:04 +0000 - rev 453762
Push 35372 by cbrindusan@mozilla.com at Mon, 14 Jan 2019 21:49:33 +0000
Bug 1516669 - Update gmp-clearkey to use CDM10 interface. r=cpearce Not much changed in the interface: - CDM initialization takes a use hardware codecs arg, we pipe this through, but don't do anything more with it in the clearkey CDM. - The CDM should call the hosts OnInitialized() when it's initialized. This is done is now done in the clearkey CDM once the session management is ready, as that is the last thing done during init. Aside from the above, it's just updating interface usage. While the new CDM interface adds support for different encryption schemes, this is not handled here and will be done in follow up bug 1516673. Differential Revision: https://phabricator.services.mozilla.com/D15518
c34e287f2f7c: Bug 1513733 - part2 : try to start AudioContext when media element which is as a source for web audio starts r=cpearce,karlt
alwu <alwu@mozilla.com> - Fri, 11 Jan 2019 20:43:01 +0000 - rev 453532
Push 35360 by nbeleuzu@mozilla.com at Sat, 12 Jan 2019 09:39:47 +0000
Bug 1513733 - part2 : try to start AudioContext when media element which is as a source for web audio starts r=cpearce,karlt If media element is used as a source for AudioContext, we would try to start AudioContext which was not allowed to start when media element starts playing. Differential Revision: https://phabricator.services.mozilla.com/D14593
df38df5ce11e: Bug 1513039 - part11 : remove pref 'media.autoplay.ask-permission'. r=cpearce
alwu <alwu@mozilla.com> - Mon, 07 Jan 2019 18:41:58 +0000 - rev 452778
Push 35330 by opoprus@mozilla.com at Tue, 08 Jan 2019 09:41:12 +0000
Bug 1513039 - part11 : remove pref 'media.autoplay.ask-permission'. r=cpearce Depends on D14335 Differential Revision: https://phabricator.services.mozilla.com/D14336
2e48b6769911: Bug 1513039 - part10 : remove nsIAutoplay.PROMPT r=daleharvey,cpearce
alwu <alwu@mozilla.com> - Mon, 07 Jan 2019 18:41:03 +0000 - rev 452777
Push 35330 by opoprus@mozilla.com at Tue, 08 Jan 2019 09:41:12 +0000
Bug 1513039 - part10 : remove nsIAutoplay.PROMPT r=daleharvey,cpearce Depends on D14334 Differential Revision: https://phabricator.services.mozilla.com/D14335
54817f010dde: Bug 1513039 - part6 : remove AutoplayPermissionManager and AutoplayPermissionRequest. r=cpearce,smaug
alwu <alwu@mozilla.com> - Mon, 07 Jan 2019 18:35:30 +0000 - rev 452773
Push 35330 by opoprus@mozilla.com at Tue, 08 Jan 2019 09:41:12 +0000
Bug 1513039 - part6 : remove AutoplayPermissionManager and AutoplayPermissionRequest. r=cpearce,smaug Depends on D14330 Differential Revision: https://phabricator.services.mozilla.com/D14331
0e229bbdaae5: Bug 1513039 - part5 : log warning when autoplay is blocked. r=cpearce,karlt
alwu <alwu@mozilla.com> - Mon, 07 Jan 2019 19:57:48 +0000 - rev 452772
Push 35330 by opoprus@mozilla.com at Tue, 08 Jan 2019 09:41:12 +0000
Bug 1513039 - part5 : log warning when autoplay is blocked. r=cpearce,karlt Wrap 'nsContentUtils::ReportToConsole()' to reduce necessary input parameters and call it when we need to log error or warning message. Show the warning when autoplay is blocked. For web audio, this restores the console messages removed in part4 and also reports the same message when the AudioContext is blocked in the constructor. Differential Revision: https://phabricator.services.mozilla.com/D14330
5d5121ecc181: Bug 1513039 - part3 : remove permission request codes in MediaElement. r=cpearce
alwu <alwu@mozilla.com> - Mon, 07 Jan 2019 18:32:42 +0000 - rev 452770
Push 35330 by opoprus@mozilla.com at Tue, 08 Jan 2019 09:41:12 +0000
Bug 1513039 - part3 : remove permission request codes in MediaElement. r=cpearce Depends on D14326 Differential Revision: https://phabricator.services.mozilla.com/D14327
9cb5b5f77ed8: Bug 1513681 - part1 : dispatch 'GloballyAutoplayBlocked' event when site is blocked r=cpearce,smaug
Alastor Wu <alwu@mozilla.com> - Fri, 04 Jan 2019 19:05:52 +0000 - rev 452621
Push 35317 by rgurzau@mozilla.com at Sat, 05 Jan 2019 12:34:37 +0000
Bug 1513681 - part1 : dispatch 'GloballyAutoplayBlocked' event when site is blocked r=cpearce,smaug This event is used to notify tab that this site is blocked and we should show the blocking icon for it. Patch2 will handle following details. Differential Revision: https://phabricator.services.mozilla.com/D14794
7ab6eb45e6b8: Bug 1513733 - part2 : try to start AudioContext when media element which is as a source for web audio starts r=cpearce,karlt
Alastor Wu <alwu@mozilla.com> - Wed, 02 Jan 2019 23:19:55 +0000 - rev 452362
Push 35304 by dvarga@mozilla.com at Thu, 03 Jan 2019 16:24:35 +0000
Bug 1513733 - part2 : try to start AudioContext when media element which is as a source for web audio starts r=cpearce,karlt If media element is used as a source for AudioContext, we would try to start AudioContext which was not allowed to start when media element starts playing. Differential Revision: https://phabricator.services.mozilla.com/D14593
23b5a58e3bcc: Bug 1513681 - part1 : dispatch 'GloballyAutoplayBlocked' event when site is permanent blocked. r=cpearce,smaug
alwu <alwu@mozilla.com> - Tue, 18 Dec 2018 18:11:56 +0000 - rev 452087
Push 35283 by dvarga@mozilla.com at Sat, 29 Dec 2018 09:34:06 +0000
Bug 1513681 - part1 : dispatch 'GloballyAutoplayBlocked' event when site is permanent blocked. r=cpearce,smaug This event is used to notify tab that this site is permanently blocked and we should show the blocking icon for it. Patch2 will handle following details. Differential Revision: https://phabricator.services.mozilla.com/D14794
dc450ef62331: Bug 1512362 - hook pref "media.autoplay.allow-muted" to all inaudible autoplay media policy r=cpearce
Alastor Wu <alwu@mozilla.com> - Thu, 13 Dec 2018 21:31:42 +0000 - rev 450469
Push 35202 by btara@mozilla.com at Fri, 14 Dec 2018 05:43:22 +0000
Bug 1512362 - hook pref "media.autoplay.allow-muted" to all inaudible autoplay media policy r=cpearce When talking about autoplay, video without audio track should also be controlled by the pref "media.autoplay.allow-muted". Differential Revision: https://phabricator.services.mozilla.com/D14480
8442297c6818: Bug 1495053 - p3: implement MediaDrmCDMProxy::SetServerCertificate(). r=cpearce
John Lin <jolin@mozilla.com> - Fri, 07 Dec 2018 14:44:34 +0000 - rev 449590
Push 35173 by nerli@mozilla.com at Fri, 07 Dec 2018 21:39:01 +0000
Bug 1495053 - p3: implement MediaDrmCDMProxy::SetServerCertificate(). r=cpearce Differential Revision: https://phabricator.services.mozilla.com/D7792
f21136ecba05: Bug 1495053 - p2: add setServerCertificate() to Java/JNI proxy classes. r=cpearce
John Lin <jolin@mozilla.com> - Fri, 07 Dec 2018 14:44:10 +0000 - rev 449589
Push 35173 by nerli@mozilla.com at Fri, 07 Dec 2018 21:39:01 +0000
Bug 1495053 - p2: add setServerCertificate() to Java/JNI proxy classes. r=cpearce Differential Revision: https://phabricator.services.mozilla.com/D7791
c51f74f5fcb8: Bug 1495053 - p1: support setting server certificate in MediaDrm bridge. r=cpearce
John Lin <jolin@mozilla.com> - Fri, 07 Dec 2018 14:43:33 +0000 - rev 449588
Push 35173 by nerli@mozilla.com at Fri, 07 Dec 2018 21:39:01 +0000
Bug 1495053 - p1: support setting server certificate in MediaDrm bridge. r=cpearce Differential Revision: https://phabricator.services.mozilla.com/D7790
3f5a132b7e4f: Bug 1512283 - Turn off doorhanger in Nightly. r=cpearce
alwu <alwu@mozilla.com> - Wed, 05 Dec 2018 20:28:52 +0000 - rev 449468
Push 35164 by shindli@mozilla.com at Thu, 06 Dec 2018 04:02:49 +0000
Bug 1512283 - Turn off doorhanger in Nightly. r=cpearce Now We decide to ship block-autoplay without doorhanger, so we would turn the pref off in Nightly. Differential Revision: https://phabricator.services.mozilla.com/D13853
c8e66d503efd: Bug 1509277: Handle ERROR_INVALID_PARAMETER from OpenProcess differently from other errors in launcher process startup; r=cpearce
Aaron Klotz <aklotz@mozilla.com> - Thu, 22 Nov 2018 22:37:04 +0000 - rev 447765
Push 35088 by shindli@mozilla.com at Fri, 23 Nov 2018 09:37:59 +0000
Bug 1509277: Handle ERROR_INVALID_PARAMETER from OpenProcess differently from other errors in launcher process startup; r=cpearce If the launcher process's parent process was ephemeral, the parent may have terminated before the launcher has a chance to open a handle to it. If so, OpenProcess will fail with ERROR_INVALID_PARAMETER, as the pid is now invalid. This is a common case, so instead of erroring out, IsSameBinaryAsParentProcess should treat this case as, "No, our parent was not a Firefox binary" and just return false. This eliminated the intermittent xpcshell failures on try. Differential Revision: https://phabricator.services.mozilla.com/D12714
345d44957fee: Bug 1491475 - part3 : add test to ensure doorhanger would not dismiss when inaudible media starts. r=cpearce
alwu <alwu@mozilla.com> - Mon, 19 Nov 2018 17:55:11 +0000 - rev 447507
Push 35081 by ccoroiu@mozilla.com at Wed, 21 Nov 2018 21:41:51 +0000
Bug 1491475 - part3 : add test to ensure doorhanger would not dismiss when inaudible media starts. r=cpearce Differential Revision: https://phabricator.services.mozilla.com/D12178
b42cd879741d: Bug 1508532 - Provide more verbose description of cdm add-on. r=cpearce,flod
Bryce Van Dyk <bvandyk@mozilla.com> - Wed, 21 Nov 2018 09:43:47 +0000 - rev 447493
Push 35081 by ccoroiu@mozilla.com at Wed, 21 Nov 2018 21:41:51 +0000
Bug 1508532 - Provide more verbose description of cdm add-on. r=cpearce,flod Differential Revision: https://phabricator.services.mozilla.com/D12383
65987ae1dfca: Bug 1505494 - remove redundant codes. r=cpearce
alwu <alwu@mozilla.com> - Mon, 12 Nov 2018 17:39:53 +0000 - rev 445899
Push 35028 by csabou@mozilla.com at Mon, 12 Nov 2018 21:54:15 +0000
Bug 1505494 - remove redundant codes. r=cpearce There are two exactly same codes here, remove one of them. Differential Revision: https://phabricator.services.mozilla.com/D11502
fe1869606e94: Bug 1505494 - not allow autoplay if we don't have a document. r=cpearce
alwu <alwu@mozilla.com> - Mon, 12 Nov 2018 17:39:51 +0000 - rev 445898
Push 35028 by csabou@mozilla.com at Mon, 12 Nov 2018 21:54:15 +0000
Bug 1505494 - not allow autoplay if we don't have a document. r=cpearce In order to avoid the nullptr crash when using `approver`. Differential Revision: https://phabricator.services.mozilla.com/D11501
6f08cc82afe3: Bug 1505910 - P2. Remove use of AsTaskQueue(). r=cpearce
Jean-Yves Avenard <jyavenard@mozilla.com> - Mon, 12 Nov 2018 01:09:03 +0000 - rev 445850
Push 35027 by rmaries@mozilla.com at Mon, 12 Nov 2018 17:18:42 +0000
Bug 1505910 - P2. Remove use of AsTaskQueue(). r=cpearce Make IsCurrentThreadIn() const. Depends on D11491 Differential Revision: https://phabricator.services.mozilla.com/D11492
f574c731fcb0: Bug 1497034 - FeaturePolicy: autoplay, r=cpearce
Andrea Marchesini <amarchesini@mozilla.com> - Tue, 09 Oct 2018 14:22:19 +0200 - rev 440236
Push 34810 by shindli@mozilla.com at Tue, 09 Oct 2018 16:24:56 +0000
Bug 1497034 - FeaturePolicy: autoplay, r=cpearce
2cdf1848b6a7: Bug 1496501 - Do not mark CDM input as unencrypted even if it has no encrypted bytes. r=cpearce
Bryce Van Dyk <bvandyk@mozilla.com> - Mon, 08 Oct 2018 22:34:32 +0000 - rev 440119
Push 34807 by shindli@mozilla.com at Tue, 09 Oct 2018 09:27:05 +0000
Bug 1496501 - Do not mark CDM input as unencrypted even if it has no encrypted bytes. r=cpearce Bug 1494178 added code to mark samples with 0 encrypted ranges as unencrypted before they were fed to the CDM. This was to catch issues where we could mark such unencrypted samples as encrypted. However, the CDM expects certain samples that are clear to still be marked as encrypted. Specifically, WebM samples should be marked as encrypted if they are from an encrypted track and have the signal byte's encryption bit set (a marker for if the packet is encrypted), even if they have no encrypted ranges. The WebM demuxer is already doing this. Further inspection and testing of the mp4 demuxer shows it is behaving in line with Chromium's current mp4 parser, which we can expect prepares its data sensibly for Widevine. As the code removed here was added as a safety fallback, but is causing issues, and as the demuxers already appear to be doing the right thing, the fallback code can be removed. Differential Revision: https://phabricator.services.mozilla.com/D8024
b88586cb4fdd: Bug 1495025 - P5. Add Windows P010 and P016 support for software decoder r=cpearce
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 04 Oct 2018 09:41:58 +0000 - rev 439965
Push 34801 by toros@mozilla.com at Mon, 08 Oct 2018 16:19:23 +0000
Bug 1495025 - P5. Add Windows P010 and P016 support for software decoder r=cpearce As we do not have an IMF nor D3D11 NV12 image, we always require a full copy of the data that will deinterleave the chroma channels. Depends on D7316 Differential Revision: https://phabricator.services.mozilla.com/D7318
5e7ea76f73f3: Bug 1495025 - P3. Store original IMFMediaType's subtype in D3D11SharedHandleImage. r=cpearce
Jean-Yves Avenard <jyavenard@mozilla.com> - Mon, 08 Oct 2018 13:26:00 +0200 - rev 439963
Push 34801 by toros@mozilla.com at Mon, 08 Oct 2018 16:19:23 +0000
Bug 1495025 - P3. Store original IMFMediaType's subtype in D3D11SharedHandleImage. r=cpearce This allows more easily the creation of the MFT required to convert to a RGBA32 image when doing a readback. Depends on D7295 Differential Revision: https://phabricator.services.mozilla.com/D7296
2851d7aead76: Bug 1495025 - P2. Use lambda for callback. r=cpearce
Jean-Yves Avenard <jyavenard@mozilla.com> - Mon, 08 Oct 2018 13:24:19 +0200 - rev 439962
Push 34801 by toros@mozilla.com at Mon, 08 Oct 2018 16:19:23 +0000
Bug 1495025 - P2. Use lambda for callback. r=cpearce I find it easier to read than a function pointer making you search elsewhere to see what it's about Depends on D7294 Differential Revision: https://phabricator.services.mozilla.com/D7295
107add19c310: Bug 1495025 - P1. Search for alternative pixel format when stream change. r=cpearce
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 04 Oct 2018 09:39:50 +0000 - rev 439961
Push 34801 by toros@mozilla.com at Mon, 08 Oct 2018 16:19:23 +0000
Bug 1495025 - P1. Search for alternative pixel format when stream change. r=cpearce When decoding a vp9 profile 2 (10 bits), the MF_E_TRANSFORM_STREAM_CHANGE message is returned. We need to look for alternative format type other than NV12: 10/16 bits. When using those formats, we can no longer assume that the D3D11ShareHandleImage can use NV12. So we will convert to RGBA32 on the fly via a MFT. Differential Revision: https://phabricator.services.mozilla.com/D7294
68efa7588ba8: Bug 1495025 - P5. Add Windows P010 and P016 support for software decoder r=cpearce
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 04 Oct 2018 09:41:58 +0000 - rev 439921
Push 34796 by btara@mozilla.com at Sun, 07 Oct 2018 09:50:12 +0000
Bug 1495025 - P5. Add Windows P010 and P016 support for software decoder r=cpearce As we do not have an IMF nor D3D11 NV12 image, we always require a full copy of the data that will deinterleave the chroma channels. Depends on D7316 Differential Revision: https://phabricator.services.mozilla.com/D7318
7fd1f6103294: Bug 1495025 - P3. Store original IMFMediaType's subtype in D3D11SharedHandleImage. r=cpearce
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 04 Oct 2018 09:40:51 +0000 - rev 439919
Push 34796 by btara@mozilla.com at Sun, 07 Oct 2018 09:50:12 +0000
Bug 1495025 - P3. Store original IMFMediaType's subtype in D3D11SharedHandleImage. r=cpearce This allows more easily the creation of the MFT required to convert to a RGBA32 image when doing a readback. Depends on D7295 Differential Revision: https://phabricator.services.mozilla.com/D7296
f1afe7e2a9e3: Bug 1495025 - P2. Use lambda for callback. r=cpearce
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 04 Oct 2018 09:40:18 +0000 - rev 439918
Push 34796 by btara@mozilla.com at Sun, 07 Oct 2018 09:50:12 +0000
Bug 1495025 - P2. Use lambda for callback. r=cpearce I find it easier to read than a function pointer making you search elsewhere to see what it's about Depends on D7294 Differential Revision: https://phabricator.services.mozilla.com/D7295
c62823871aca: Bug 1495025 - P1. Search for alternative pixel format when stream change. r=cpearce
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 04 Oct 2018 09:39:50 +0000 - rev 439917
Push 34796 by btara@mozilla.com at Sun, 07 Oct 2018 09:50:12 +0000
Bug 1495025 - P1. Search for alternative pixel format when stream change. r=cpearce When decoding a vp9 profile 2 (10 bits), the MF_E_TRANSFORM_STREAM_CHANGE message is returned. We need to look for alternative format type other than NV12: 10/16 bits. When using those formats, we can no longer assume that the D3D11ShareHandleImage can use NV12. So we will convert to RGBA32 on the fly via a MFT. Differential Revision: https://phabricator.services.mozilla.com/D7294
528dbc463c22: Bug 1495025 - P5. Add Windows P010 and P016 support for software decoder r=cpearce
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 04 Oct 2018 09:41:58 +0000 - rev 439546
Push 34778 by nbeleuzu@mozilla.com at Thu, 04 Oct 2018 15:22:02 +0000
Bug 1495025 - P5. Add Windows P010 and P016 support for software decoder r=cpearce As we do not have an IMF nor D3D11 NV12 image, we always require a full copy of the data that will deinterleave the chroma channels. Depends on D7316 Differential Revision: https://phabricator.services.mozilla.com/D7318
c3b43ee1092e: Bug 1495025 - P3. Store original IMFMediaType's subtype in D3D11SharedHandleImage. r=cpearce
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 04 Oct 2018 09:40:51 +0000 - rev 439544
Push 34778 by nbeleuzu@mozilla.com at Thu, 04 Oct 2018 15:22:02 +0000
Bug 1495025 - P3. Store original IMFMediaType's subtype in D3D11SharedHandleImage. r=cpearce This allows more easily the creation of the MFT required to convert to a RGBA32 image when doing a readback. Depends on D7295 Differential Revision: https://phabricator.services.mozilla.com/D7296