Shared secondary process is the default in the GTK+ port, but we should run the tests using the network process to ensure everything works with the network process. In the future, the shared secondary process model might be a special case of multiple processes with 1 as maximum number of processes, but always using the network process.
Created attachment 241031 [details] Patch
Committed r175690: <http://trac.webkit.org/changeset/175690>
Re-opened since this is blocked by bug 138460
Let's try again now :-)
Committed r175776: <http://trac.webkit.org/changeset/175776>
Ok, no more crashes, but there are still some more failures, most of them are http tests. I'll look at those in detail and will try to fix them without rolling this out again.
(In reply to comment #6) > Ok, no more crashes, but there are still some more failures, most of them > are http tests. I'll look at those in detail and will try to fix them > without rolling this out again. This (r175776) has caused +60 new failures: r175775 -> https://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release%20%28Tests%29/builds/4067 r175776 -> https://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release%20%28Tests%29/builds/4064
One big reason for these failures is that shared workers are disabled in multiple web process mode. Another one that I'm particularly interested in is http/tests/xmlhttprequest/cross-origin-cookie-storage.html. It looks like I independently broke this test on Mac a few revisions later. We'll need to see what's going on there.
> Another one that I'm particularly interested in is > http/tests/xmlhttprequest/cross-origin-cookie-storage.html. Turns out that it's simply because testRunner.setAlwaysAcceptCookies is not implemented with NetworkProcess. The test used to pass on Mac by mistake, due to having a cookie leak from a previous test. I disabled it on Mac.
(In reply to comment #7) > (In reply to comment #6) > > Ok, no more crashes, but there are still some more failures, most of them > > are http tests. I'll look at those in detail and will try to fix them > > without rolling this out again. > > This (r175776) has caused +60 new failures: I know, and I said that I'll look at those in detail and I'll try to fix them without rolling it out. If I can't figure out how to fix the failures, I'll roll this out, don't worry. > r175775 -> > https://build.webkit.org/builders/GTK%20Linux%2064- > bit%20Release%20%28Tests%29/builds/4067 > > r175776 -> > https://build.webkit.org/builders/GTK%20Linux%2064- > bit%20Release%20%28Tests%29/builds/4064
(In reply to comment #8) > One big reason for these failures is that shared workers are disabled in > multiple web process mode. Thanks, I've just skipped the shared workers tests in r176019