db3ac4b81bc7ba601354f332573ed4746ed0507d: Bug 1599026, part 2 - Don't send media notifications to discarded top BCs. r=alwu
draft
Andrew McCreight <continuation@gmail.com> - Mon, 25 Nov 2019 11:00:28 -0800 - rev 2496070
Push
455279 by amccreight@mozilla.com at Mon, 25 Nov 2019 19:56:04 +0000
Bug 1599026, part 2 - Don't send media notifications to discarded top BCs. r=alwu
This patch centralizes the checking for discarded browsing contexts
for the media IPC messages. It also adds a check that the top BC is
not discarded, which seems to be causing crashes. A BC can be
non-discard when its parent is discarded because discarding is not
atomic.
This changes the BC we log the id of, but hopefully that is okay.
The patch also fixes the 'Browing' typo in the method name.
505817db5977035f35932db5913bf4c8f08d597e: Bug 1599026, part 1 - De-static methods in MediaControlUtils.h. r=alwu
draft
Andrew McCreight <continuation@gmail.com> - Mon, 25 Nov 2019 09:15:59 -0800 - rev 2496069
Push
455279 by amccreight@mozilla.com at Mon, 25 Nov 2019 19:56:04 +0000
Bug 1599026, part 1 - De-static methods in MediaControlUtils.h. r=alwu
I expect that the intention is to indicate that these are helper
methods and to produce a warning if the helper methods are no longer
used, but making functions static in a header creates a new copy of
the function in every compilation unit that includes it.
7f996d69bb08843ba8e51d72ae694148376d839a: Bug 1599180 - Fix some trivial "browing" typos. r=ehsan
draft
Andrew McCreight <continuation@gmail.com> - Mon, 25 Nov 2019 10:19:37 -0800 - rev 2496068
Push
455279 by amccreight@mozilla.com at Mon, 25 Nov 2019 19:56:04 +0000
Bug 1599180 - Fix some trivial "browing" typos. r=ehsan
Differential Revision:
https://phabricator.services.mozilla.com/D54570
a958af7762aba767148a8b25478ebc7e936dfe05: Bug 1599175, part 2 - Remove DOS line endings in PiP's browser.ini. r=mconley
draft
Andrew McCreight <continuation@gmail.com> - Mon, 25 Nov 2019 09:46:09 -0800 - rev 2496067
Push
455279 by amccreight@mozilla.com at Mon, 25 Nov 2019 19:56:04 +0000
Bug 1599175, part 2 - Remove DOS line endings in PiP's browser.ini. r=mconley
Differential Revision:
https://phabricator.services.mozilla.com/D54566
23cc28364da191e3faad6fac0f878b508578f3f6: Bug 1599175, part 1 - Remove unused 'browingContextId' message field in PictureInPictureChild. r=mconley
draft
Andrew McCreight <continuation@gmail.com> - Mon, 25 Nov 2019 09:31:26 -0800 - rev 2496066
Push
455279 by amccreight@mozilla.com at Mon, 25 Nov 2019 19:56:04 +0000
Bug 1599175, part 1 - Remove unused 'browingContextId' message field in PictureInPictureChild. r=mconley
This field is misspelled. Don't worry though, it is also unused.
Differential Revision:
https://phabricator.services.mozilla.com/D54565
93519b2a071cc49d26c34fc51fa36b3cdd833fa8: Bug 1598787 - Rename XRE_ChildProcessTypeToString to XRE_GeckoProcessTypeToString. r=froydnj
draft
Andrew McCreight <continuation@gmail.com> - Fri, 22 Nov 2019 14:35:33 -0800 - rev 2496065
Push
455279 by amccreight@mozilla.com at Mon, 25 Nov 2019 19:56:04 +0000
Bug 1598787 - Rename XRE_ChildProcessTypeToString to XRE_GeckoProcessTypeToString. r=froydnj
This function works on all GeckoProcessTypes, not just those for child
processes.
Differential Revision:
https://phabricator.services.mozilla.com/D54375
cd53a79a4c21e355a9d8c8fb984161e709c43583: Bug 1598173 - Make NoteIntentionalCrash() work in the main process. r=froydnj
draft
Andrew McCreight <continuation@gmail.com> - Fri, 22 Nov 2019 14:16:12 -0800 - rev 2496064
Push
455279 by amccreight@mozilla.com at Mon, 25 Nov 2019 19:56:04 +0000
Bug 1598173 - Make NoteIntentionalCrash() work in the main process. r=froydnj
For processes besides the main process, the leak log is derived from
the value of XPCOM_MEM_BLOAT_LOG, but for the main process, it is
simply whatever XPCOM_MEM_BLOAT_LOG is. This patch adds code to handle
the latter case.
It would be nice to share this code with nsTraceRefcnt.cpp, but
NoteIntentionalCrash() is used in the test plugins so I don't think it
can actually call into libxul. The logic isn't all that complex.
Differential Revision:
https://phabricator.services.mozilla.com/D54374
263e5c4688672c30c3a78216f57729b8e441eb24: Revert "trying and failing to make a crashing readable streams test"
draft
Andrew McCreight <continuation@gmail.com> - Mon, 25 Nov 2019 11:49:55 -0800 - rev 2496063
Push
455279 by amccreight@mozilla.com at Mon, 25 Nov 2019 19:56:04 +0000
Revert "trying and failing to make a crashing readable streams test"
This reverts commit 7664a294a0761c323bedec77917fb027d821de89.
0d9e7834e5e0cea4ae2522b96735056b977f2213: trying and failing to make a crashing readable streams test
draft
Andrew McCreight <continuation@gmail.com> - Fri, 22 Nov 2019 14:13:47 -0800 - rev 2496062
Push
455279 by amccreight@mozilla.com at Mon, 25 Nov 2019 19:56:04 +0000
trying and failing to make a crashing readable streams test
f1ce595c1334744af3d42b06db8bc4015f921402: Bug 1598075 - Reuse Fission content processes in reader mode tests. r=Gijs
draft
Andrew McCreight <continuation@gmail.com> - Wed, 20 Nov 2019 11:09:13 -0800 - rev 2496061
Push
455279 by amccreight@mozilla.com at Mon, 25 Nov 2019 19:56:04 +0000
Bug 1598075 - Reuse Fission content processes in reader mode tests. r=Gijs
There are frequent shutdown hangs in this directory. Making us reuse
content processes when Fission is enabled has papered over the
shutdown hangs from reader mode tests in other directories, so
hopefully it will help here, too.
Differential Revision:
https://phabricator.services.mozilla.com/D54012
9f355e9d2c4f59f33f5207a07b7df5d68cd26aee: Bug 1597036 - Add fission.autostart as an optional locked pref in the troubleshooting schema.
draft
Andrew McCreight <continuation@gmail.com> - Tue, 19 Nov 2019 14:54:11 -0800 - rev 2496060
Push
455279 by amccreight@mozilla.com at Mon, 25 Nov 2019 19:56:04 +0000
Bug 1597036 - Add fission.autostart as an optional locked pref in the troubleshooting schema.
I think that
Bug 1572925 made it an error to have any locked prefs
reported in about:support, because they had a pref that accidentally
was getting reported, and I guess we don't have any by
default. fission.autostart is locked on beta and release, but not on
Nightly. This patch makes it so it is okay, but not necessary, to have
fission.autostart reported as a locked preference, so that the test
will pass in either configuration.
Differential Revision:
https://phabricator.services.mozilla.com/D53888
43f53851231874d8af3b7e5e8d93d18a9d719f14: Bug 1597804 - Fix "pricnipal" typos in some comments. r=ehsan
draft
Andrew McCreight <continuation@gmail.com> - Tue, 19 Nov 2019 13:43:53 -0800 - rev 2496059
Push
455279 by amccreight@mozilla.com at Mon, 25 Nov 2019 19:56:04 +0000
Bug 1597804 - Fix "pricnipal" typos in some comments. r=ehsan
Differential Revision:
https://phabricator.services.mozilla.com/D53882
f57bbf975cc436516f7dfaf747ebeb90a8cf56f2: Bug 1596254 - process selectors should deal with null processes. r=mconley
draft
Andrew McCreight <continuation@gmail.com> - Tue, 19 Nov 2019 09:09:30 -0800 - rev 2496058
Push
455279 by amccreight@mozilla.com at Mon, 25 Nov 2019 19:56:04 +0000
Bug 1596254 - process selectors should deal with null processes. r=mconley
If a process is in the middle of shutting down, it will show up as
null in the processes array.
Differential Revision:
https://phabricator.services.mozilla.com/D53839
8c66b0d86e9750b328840fcfe7f58a6c3ebbc03d: Bug 1596609 - Make more tests reuse Fission content processes. r=kmag
draft
Andrew McCreight <continuation@gmail.com> - Thu, 14 Nov 2019 15:54:11 -0800 - rev 2496057
Push
455279 by amccreight@mozilla.com at Mon, 25 Nov 2019 19:56:04 +0000
Bug 1596609 - Make more tests reuse Fission content processes. r=kmag
These tests are frequently leaking a single BackstagePass and its
XPCWN. To paper over the leak, this patch makes us reuse Fission
content processes in a few places. I filed
bug 1596611 about fixing
the leak and reverting the change.
Differential Revision:
https://phabricator.services.mozilla.com/D53108
9397a92019d87b3117bbf8520e882e957ea00073: Revert "Shutdown DMD scan"
draft
Andrew McCreight <continuation@gmail.com> - Fri, 15 Nov 2019 08:30:30 -0800 - rev 2496056
Push
455279 by amccreight@mozilla.com at Mon, 25 Nov 2019 19:56:04 +0000
Revert "Shutdown DMD scan"
This reverts commit ca8f5d90c3d44715a29fdb6c376d8c2161cfb8dd.
cbf27a98f095d9a2c295d21fa2f42addf9e470e2: Shutdown DMD scan
draft
Andreas Pehrson <apehrson@mozilla.com> - Wed, 13 Nov 2019 15:49:09 -0800 - rev 2496055
Push
455279 by amccreight@mozilla.com at Mon, 25 Nov 2019 19:56:04 +0000
Shutdown DMD scan
30640f965593270f300713fafee32b57f2a71ac9: Fuzzy query=mochitest-media
draft
Dan Minor <dminor@mozilla.com> - Mon, 25 Nov 2019 14:55:33 -0500 - rev 2496054
Push
455278 by dminor@mozilla.com at Mon, 25 Nov 2019 19:55:49 +0000
Fuzzy query=mochitest-media
Pushed via `mach try fuzzy`
d3b32e6aa6c816f41124195c39c9781795c61931: Bug 1581074 - Remove mStartBlocking; r=pehrsons
draft
Dan Minor <dminor@mozilla.com> - Fri, 22 Nov 2019 15:58:29 -0500 - rev 2496053
Push
455278 by dminor@mozilla.com at Mon, 25 Nov 2019 19:55:49 +0000
Bug 1581074 - Remove mStartBlocking; r=pehrsons
03cbabba0155d5d0eb35a2f9d53f799007148628: Bug 1581074 - Remove AdvanceTimeVaryingValuesToCurrentTime; r=pehrsons
draft
Dan Minor <dminor@mozilla.com> - Fri, 22 Nov 2019 15:30:18 -0500 - rev 2496052
Push
455278 by dminor@mozilla.com at Mon, 25 Nov 2019 19:55:49 +0000
Bug 1581074 - Remove AdvanceTimeVaryingValuesToCurrentTime; r=pehrsons
991655c9cc76b21557384bdb3c4318494ef2e07e: Bug 1581074 - Remove flags from ProcessInput; r=pehrsons
draft
Dan Minor <dminor@mozilla.com> - Fri, 22 Nov 2019 14:50:28 -0500 - rev 2496051
Push
455278 by dminor@mozilla.com at Mon, 25 Nov 2019 19:55:49 +0000
Bug 1581074 - Remove flags from ProcessInput; r=pehrsons