searching for reviewer(cpearce)
062f54735111: Bug 1522547 - Have MediaKeys observe if the owning document becomes inactive so it can shutdown. r=cpearce
Bryce Van Dyk <bvandyk@mozilla.com> - Tue, 05 Mar 2019 16:21:40 +0000 - rev 520333
Push 10862 by ffxbld-merge at Mon, 11 Mar 2019 13:01:11 +0000
Bug 1522547 - Have MediaKeys observe if the owning document becomes inactive so it can shutdown. r=cpearce MediaKeys objects are typically created and associated with an HTMLMediaElement, however it is possible to create a MediaKeys object and not associate it with an HTMLMediaElement. This resulted in an issue where these MediaKeys would keep alive other components that would assert during bowrser shutdown (see bug 1522547). We anticipated that MediaKeys associated with an HTMLMediaElement would need to be shutdown if their owning document became inactive, but were not handling the case where the keys never became associated with an element. This patch has the MediaKeys listen directly to their owning document for activity change. The MediaKeys will shutdown if their document becomes inactive. This avoids MediaKeys not associated with HTMLMediaElements keeping other objects alive during browser shutdown. Differential Revision: https://phabricator.services.mozilla.com/D21983
153245ec86c2: Bug 1522547 - Log in MediaKeySession dtor. r=cpearce
Bryce Van Dyk <bvandyk@mozilla.com> - Tue, 05 Mar 2019 16:21:30 +0000 - rev 520332
Push 10862 by ffxbld-merge at Mon, 11 Mar 2019 13:01:11 +0000
Bug 1522547 - Log in MediaKeySession dtor. r=cpearce Differential Revision: https://phabricator.services.mozilla.com/D21982
c273b07bc5f0: Bug 1522547 - Add more logs to MediaKeys, use PRIu32 for printing where appropriate. r=cpearce
Bryce Van Dyk <bvandyk@mozilla.com> - Tue, 05 Mar 2019 16:21:28 +0000 - rev 520331
Push 10862 by ffxbld-merge at Mon, 11 Mar 2019 13:01:11 +0000
Bug 1522547 - Add more logs to MediaKeys, use PRIu32 for printing where appropriate. r=cpearce - Add more logging to MediaKeys. - Replace the instances of the %d formatter with PRIu32 when printing the PromiseId type (which has the underlying uint32_t type). Differential Revision: https://phabricator.services.mozilla.com/D21981
c092112f933e: Bug 1522547 - Log `this` in ChromiumCDMProxy logs, use PRIu32 where appropriate. r=cpearce
Bryce Van Dyk <bvandyk@mozilla.com> - Tue, 05 Mar 2019 16:21:24 +0000 - rev 520330
Push 10862 by ffxbld-merge at Mon, 11 Mar 2019 13:01:11 +0000
Bug 1522547 - Log `this` in ChromiumCDMProxy logs, use PRIu32 where appropriate. r=cpearce Because multiple ChromiumCDMProxy object may exist during a browser lifetime, logging the value of `this` in their logs is useful for disambiguating log statements, as well as matching ChromiumCDMProxy objects to pointers held by other objects. Update formatter usage in logs to prefer PRIu32 over %u when the underlying type is uint32_t. Differential Revision: https://phabricator.services.mozilla.com/D21980
7e199486530b: Bug 1522547 - Differentiate GMPServiceParent and GMPServiceChild __CLASS__ macro. r=cpearce
Bryce Van Dyk <bvandyk@mozilla.com> - Tue, 05 Mar 2019 16:21:20 +0000 - rev 520329
Push 10862 by ffxbld-merge at Mon, 11 Mar 2019 13:01:11 +0000
Bug 1522547 - Differentiate GMPServiceParent and GMPServiceChild __CLASS__ macro. r=cpearce The macros in these classes are used to output class names in logs. Differentiating them helps make logs clearer. Differential Revision: https://phabricator.services.mozilla.com/D21979
6f4b04b69f80: Bug 1522547 - Add logging to GMPContentParent. r=cpearce
Bryce Van Dyk <bvandyk@mozilla.com> - Tue, 05 Mar 2019 16:19:43 +0000 - rev 520328
Push 10862 by ffxbld-merge at Mon, 11 Mar 2019 13:01:11 +0000
Bug 1522547 - Add logging to GMPContentParent. r=cpearce Use GMPLog.h in GMPContentPareant. Add logging to most functions. This logging was added to aid in diagnosing a shutdown crash, but should be generally useful to have. Driveby touchup of arg name to ChromiumCDMDestroyed to match header. Differential Revision: https://phabricator.services.mozilla.com/D21978
0bd2a637efa3: Bug 1532354 - Remove virtual keyword on GMPServiceChild|GMPServiceParent::GetContentParent. r=cpearce
Bryce Van Dyk <bvandyk@mozilla.com> - Tue, 05 Mar 2019 00:32:07 +0000 - rev 520248
Push 10862 by ffxbld-merge at Mon, 11 Mar 2019 13:01:11 +0000
Bug 1532354 - Remove virtual keyword on GMPServiceChild|GMPServiceParent::GetContentParent. r=cpearce These functions already specify the override keyword. This change also makes them more consistent with the overloaded versions of the same functions in these classes. Depends on D21968 Differential Revision: https://phabricator.services.mozilla.com/D21975
65fc1cf8774f: Bug 1532354 - Remove unused DeleteGMPServiceParent class. r=cpearce
Bryce Van Dyk <bvandyk@mozilla.com> - Tue, 05 Mar 2019 00:31:27 +0000 - rev 520247
Push 10862 by ffxbld-merge at Mon, 11 Mar 2019 13:01:11 +0000
Bug 1532354 - Remove unused DeleteGMPServiceParent class. r=cpearce Bug 1352924 removed the usage of this class, so we can safely remove the dead code. Differential Revision: https://phabricator.services.mozilla.com/D21968
c1e801670307: Bug 1532354 - Remove GMPServiceParent::ProcessPossiblePlugin decl as it is unimplemented. r=cpearce
Bryce Van Dyk <bvandyk@mozilla.com> - Tue, 05 Mar 2019 00:30:13 +0000 - rev 520246
Push 10862 by ffxbld-merge at Mon, 11 Mar 2019 13:01:11 +0000
Bug 1532354 - Remove GMPServiceParent::ProcessPossiblePlugin decl as it is unimplemented. r=cpearce Differential Revision: https://phabricator.services.mozilla.com/D21967
07c49703b70c: Bug 1530220 - part4 : separate tesing web audio with gum to another test. r=cpearce
Alastor Wu <alwu@mozilla.com> - Tue, 05 Mar 2019 02:13:55 +0000 - rev 520209
Push 10862 by ffxbld-merge at Mon, 11 Mar 2019 13:01:11 +0000
Bug 1530220 - part4 : separate tesing web audio with gum to another test. r=cpearce Testing web audio with GUM is not related with the original purpose of this test. In order to reduce the complexity of this test, separate this part as another new test. Also modify the naming of functions, because Mozilla coding style is to use the camelCase for the name. Differential Revision: https://phabricator.services.mozilla.com/D21257
f706441a86b5: Bug 1530220 - part2 : modify test in order to test different kinds of keycode. r=cpearce
Alastor Wu <alwu@mozilla.com> - Tue, 05 Mar 2019 02:09:50 +0000 - rev 520207
Push 10862 by ffxbld-merge at Mon, 11 Mar 2019 13:01:11 +0000
Bug 1530220 - part2 : modify test in order to test different kinds of keycode. r=cpearce Except printable keys, we would treat `carriage return` and `space` as supported user gesture inputs for activating document. Other keys won't activate document. Differential Revision: https://phabricator.services.mozilla.com/D21255
7c1b053e161b: Bug 1530220 - part1 : allow some non-printalble keys as supported user gesture inputs to activate document. r=masayuki,cpearce
Alastor Wu <alwu@mozilla.com> - Tue, 05 Mar 2019 02:06:22 +0000 - rev 520206
Push 10862 by ffxbld-merge at Mon, 11 Mar 2019 13:01:11 +0000
Bug 1530220 - part1 : allow some non-printalble keys as supported user gesture inputs to activate document. r=masayuki,cpearce `carriage return` and `space` are common keys which user might use to start media, so we should take account them as supported user gesture inputs. As their pseudo char code are zero, we have to check their key code in order to distinguish them from other controls keys such as shift, alt... Differential Revision: https://phabricator.services.mozilla.com/D21253
6e3eb5df50ff: Bug 1526355 - part1 : notify the front end side to show the blocking icon if the site is still being blocked after it's back from the bfcache. r=cpearce
Alastor Wu <alwu@mozilla.com> - Mon, 04 Mar 2019 17:25:52 +0000 - rev 520123
Push 10862 by ffxbld-merge at Mon, 11 Mar 2019 13:01:11 +0000
Bug 1526355 - part1 : notify the front end side to show the blocking icon if the site is still being blocked after it's back from the bfcache. r=cpearce In order to display blocking icon when the document comes back from the bfcache, we have to notify front end what's the current blocking status. As the front end side would clear blocking autoplay information when nagivation occurs, and the media might not invoke the play again when they comes back from the bfcache. Therefore, we should notify front end side that the site is still being blocked, and we should show blocking icon for it. Differential Revision: https://phabricator.services.mozilla.com/D21582
62d4e16fbb2b: Bug 1531500 - Fix HE-AAC with WMF decoder. r=cpearce
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 01 Mar 2019 00:54:32 +0000 - rev 519723
Push 10862 by ffxbld-merge at Mon, 11 Mar 2019 13:01:11 +0000
Bug 1531500 - Fix HE-AAC with WMF decoder. r=cpearce The rate changes after decoding the first sample to what was indicated in the container. The code to handle that case was incorrectly removed in bug 1530234 Differential Revision: https://phabricator.services.mozilla.com/D21618
0a003a54ff45: Bug 1525156 - part2 : Add Telemetry to know how many autoplay media, which has been resumed from blocked, played exactly 7 secocnds or more, or less than 7 seconds. r=cpearce,janerik
Alastor Wu <alwu@mozilla.com> - Tue, 26 Feb 2019 04:44:19 +0000 - rev 519109
Push 10862 by ffxbld-merge at Mon, 11 Mar 2019 13:01:11 +0000
Bug 1525156 - part2 : Add Telemetry to know how many autoplay media, which has been resumed from blocked, played exactly 7 secocnds or more, or less than 7 seconds. r=cpearce,janerik By adding the Telemetry to measure the number of video/audio which played exactly 7 seconds or more, or less than 7 seconds, after those media has been resumed from blocked state, we can know how many media would meet the Chrome's MEI condition, which could help us to know more about the whole landscape of autoplay media. In addition, it could help us know how many media are played 'by users intention' because we assume that users are more likely to stop the media if autoplay media is unblocked by accident. Differential Revision: https://phabricator.services.mozilla.com/D18628
adb554a755e5: Bug 1525156 - part1 : not dispatch event when play is rejected by AudioChannelAgent. r=cpearce
Alastor Wu <alwu@mozilla.com> - Thu, 21 Feb 2019 22:52:57 +0000 - rev 519108
Push 10862 by ffxbld-merge at Mon, 11 Mar 2019 13:01:11 +0000
Bug 1525156 - part1 : not dispatch event when play is rejected by AudioChannelAgent. r=cpearce "blocked" event is used for blocking autoplay. The `AudioChannelAgentBlockedPlay()` returns true when we lost audio focus on Android, so actually we don't need to dispatch "blocked" event. Differential Revision: https://phabricator.services.mozilla.com/D18627
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 518183
Push 10862 by ffxbld-merge at Mon, 11 Mar 2019 13:01:11 +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 516657
Push 10862 by ffxbld-merge at Mon, 11 Mar 2019 13:01:11 +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 514033
Push 10862 by ffxbld-merge at Mon, 11 Mar 2019 13:01:11 +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
cd093abd9a5c: Bug 1526355 - part1 : notify the front end side to show the blocking icon if the site is still being blocked after it's back from the bfcache. r=cpearce a=lizzard
Alastor Wu <alwu@mozilla.com> - Mon, 04 Mar 2019 17:25:52 +0000 - rev 513412
Push 10839 by archaeopteryx@coole-files.de at Thu, 07 Mar 2019 09:52:32 +0000
Bug 1526355 - part1 : notify the front end side to show the blocking icon if the site is still being blocked after it's back from the bfcache. r=cpearce a=lizzard In order to display blocking icon when the document comes back from the bfcache, we have to notify front end what's the current blocking status. As the front end side would clear blocking autoplay information when nagivation occurs, and the media might not invoke the play again when they comes back from the bfcache. Therefore, we should notify front end side that the site is still being blocked, and we should show blocking icon for it. Differential Revision: https://phabricator.services.mozilla.com/D21582
3b0bc8cad66f: Bug 1530220 - part4 : separate tesing web audio with gum to another test. r=cpearce a=lizzard
Alastor Wu <alwu@mozilla.com> - Tue, 05 Mar 2019 02:13:55 +0000 - rev 513407
Push 10838 by ccoroiu@mozilla.com at Thu, 07 Mar 2019 00:22:54 +0000
Bug 1530220 - part4 : separate tesing web audio with gum to another test. r=cpearce a=lizzard Testing web audio with GUM is not related with the original purpose of this test. In order to reduce the complexity of this test, separate this part as another new test. Also modify the naming of functions, because Mozilla coding style is to use the camelCase for the name. Differential Revision: https://phabricator.services.mozilla.com/D21257
4b05bf628c61: Bug 1530220 - part2 : modify test in order to test different kinds of keycode. r=cpearce a=lizzard
Alastor Wu <alwu@mozilla.com> - Tue, 05 Mar 2019 02:09:50 +0000 - rev 513405
Push 10838 by ccoroiu@mozilla.com at Thu, 07 Mar 2019 00:22:54 +0000
Bug 1530220 - part2 : modify test in order to test different kinds of keycode. r=cpearce a=lizzard Except printable keys, we would treat `carriage return` and `space` as supported user gesture inputs for activating document. Other keys won't activate document. Differential Revision: https://phabricator.services.mozilla.com/D21255
dc39216dfad3: Bug 1530220 - part1 : allow some non-printalble keys as supported user gesture inputs to activate document. r=masayuki,cpearce a=lizzard
Alastor Wu <alwu@mozilla.com> - Tue, 05 Mar 2019 02:06:22 +0000 - rev 513404
Push 10838 by ccoroiu@mozilla.com at Thu, 07 Mar 2019 00:22:54 +0000
Bug 1530220 - part1 : allow some non-printalble keys as supported user gesture inputs to activate document. r=masayuki,cpearce a=lizzard `carriage return` and `space` are common keys which user might use to start media, so we should take account them as supported user gesture inputs. As their pseudo char code are zero, we have to check their key code in order to distinguish them from other controls keys such as shift, alt... Differential Revision: https://phabricator.services.mozilla.com/D21253
d520fdd91a48: Bug 1522923 - Turn off blocking autoplay on Beta. r=cpearce a=lizzard
Alastor Wu <alwu@mozilla.com> - Tue, 19 Feb 2019 20:28:25 +0000 - rev 513285
Push 10789 by rmaries@mozilla.com at Wed, 27 Feb 2019 20:23:22 +0000
Bug 1522923 - Turn off blocking autoplay on Beta. r=cpearce a=lizzard 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
1079d285dd40: Bug 1523013 - Clear SamplesWaitingForKey::mProxy in Shutdown(). r=cpearce a=lizzard
Andrew McCreight <continuation@gmail.com> - Wed, 30 Jan 2019 22:56:21 +0000 - rev 512876
Push 10630 by nbeleuzu@mozilla.com at Wed, 06 Feb 2019 10:44:39 +0000
Bug 1523013 - Clear SamplesWaitingForKey::mProxy in Shutdown(). r=cpearce a=lizzard 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 512065
Push 10566 by archaeopteryx@coole-files.de at Mon, 28 Jan 2019 12:41:12 +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 511972
Push 10566 by archaeopteryx@coole-files.de at Mon, 28 Jan 2019 12:41:12 +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 511483
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 511357
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 511253
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 511252
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 511251
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 511250
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 511249
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 510994
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 510993
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 510911
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 510867
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 510866
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 510865
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 510635
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 509881
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 509880
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 509876
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 509875
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 509873
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 509724
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 509465
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 509190
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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 507572
Push 10547 by ffxbld-merge at Mon, 21 Jan 2019 13:03:58 +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