Bug 1598075 - Reuse Fission content processes in reader mode tests. r=Gijs draft
authorAndrew McCreight <continuation@gmail.com>
Wed, 20 Nov 2019 11:09:13 -0800
changeset 2496061 f1ce595c1334744af3d42b06db8bc4015f921402
parent 2496060 9f355e9d2c4f59f33f5207a07b7df5d68cd26aee
child 2496062 0d9e7834e5e0cea4ae2522b96735056b977f2213
push id455279
push useramccreight@mozilla.com
push dateMon, 25 Nov 2019 19:56:04 +0000
treeherdertry@59b13b57567a [default view] [failures only]
reviewersGijs
bugs1598075
milestone72.0a1
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
toolkit/components/reader/test/browser.ini
--- a/toolkit/components/reader/test/browser.ini
+++ b/toolkit/components/reader/test/browser.ini
@@ -1,9 +1,14 @@
 [DEFAULT]
+prefs =
+    # With Fission enabled, this directory spawns many child processes, which
+    # somehow results in frequent shutdown leaks. Reuse the content processes
+    # to avoid this. See bug 1594572.
+    dom.ipc.keepProcessesAlive.webIsolated.perOrigin=1
 support-files = head.js
 [browser_readerMode.js]
 support-files =
   readerModeNonArticle.html
   readerModeArticle.html
   readerModeArticleHiddenNodes.html
 skip-if = fission && os == 'win' # Bug 1586139
 [browser_readerMode_hidden_nodes.js]