97580 2012-09-25 11:16:31 -0700 [GTK] Don't kill automatically the web process when the ui process finishes 2012-09-25 11:33:43 -0700 1 1 1 Unclassified WebKit WebKit2 528+ (Nightly build) PC Linux RESOLVED FIXED Gtk P2 Normal --- 97583 1 cgarcia webkit-unassigned oldest_to_newest 727803 0 cgarcia 2012-09-25 11:16:31 -0700 We use prctl(PR_SET_PDEATHSIG, SIGKILL); in linux to make sure the web process is killed when the ui process finishes. This is not needed any more since now the web process stops the main loop when the connection with the ui process is closed in WebProcess::didClose(). This approach is better because it works for any platform and makes the web process finish normally from the main(). 727806 1 165643 cgarcia 2012-09-25 11:19:17 -0700 Created attachment 165643 Patch 727820 2 165643 mrobinson 2012-09-25 11:29:40 -0700 Comment on attachment 165643 Patch Nice. 727826 3 cgarcia 2012-09-25 11:33:43 -0700 Committed r129533: <http://trac.webkit.org/changeset/129533> 165643 2012-09-25 11:19:17 -0700 2012-09-25 11:29:40 -0700 Patch wk2-web-process-finish.diff text/plain 1884 cgarcia ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQyL0No YW5nZUxvZwppbmRleCBjZTA3MmEzLi5iZDFiY2MxIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0 Mi9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCkBAIC0xLDUgKzEsMjQg QEAKIDIwMTItMDktMjUgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29t PgogCisgICAgICAgIFtHVEtdIERvbid0IGtpbGwgYXV0b21hdGljYWxseSB0aGUgd2ViIHByb2Nl c3Mgd2hlbiB0aGUgdWkgcHJvY2VzcyBmaW5pc2hlcworICAgICAgICBodHRwczovL2J1Z3Mud2Vi a2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9OTc1ODAKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JP RFkgKE9PUFMhKS4KKworICAgICAgICBXZSB1c2UgcHJjdGwoUFJfU0VUX1BERUFUSFNJRywgU0lH S0lMTCk7IGluIGxpbnV4IHRvIG1ha2Ugc3VyZSB0aGUKKyAgICAgICAgd2ViIHByb2Nlc3MgaXMg a2lsbGVkIHdoZW4gdGhlIFVJIHByb2Nlc3MgZmluaXNoZXMuIFRoaXMgaXMgbm90CisgICAgICAg IG5lZWRlZCBhbnkgbW9yZSBzaW5jZSBub3cgdGhlIHdlYiBwcm9jZXNzIHN0b3BzIHRoZSBtYWlu IGxvb3Agd2hlbgorICAgICAgICB0aGUgY29ubmVjdGlvbiB3aXRoIHRoZSBVSSBwcm9jZXNzIGlz IGNsb3NlZCBpbgorICAgICAgICBXZWJQcm9jZXNzOjpkaWRDbG9zZSgpLiBUaGlzIGFwcHJvYWNo IGlzIGJldHRlciBiZWNhdXNlIGl0IHdvcmtzCisgICAgICAgIGZvciBhbnkgcGxhdGZvcm0gYW5k IG1ha2VzIHRoZSB3ZWIgcHJvY2VzcyBmaW5pc2ggbm9ybWFsbHkgZnJvbQorICAgICAgICBtYWlu KCkuCisKKyAgICAgICAgKiBVSVByb2Nlc3MvTGF1bmNoZXIvZ3RrL1Byb2Nlc3NMYXVuY2hlckd0 ay5jcHA6CisgICAgICAgIChXZWJLaXQ6OmNoaWxkU2V0dXBGdW5jdGlvbik6IERvbid0IGNhbGwg cHJjdGwoUFJfU0VUX1BERUFUSFNJRywKKyAgICAgICAgU0lHS0lMTCkgaW4gdGhlIGNoaWxkIHBy b2Nlc3MuCisKKzIwMTItMDktMjUgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2Fs aWEuY29tPgorCiAgICAgICAgIFtHVEtdIERvbid0IHVzZSB0aGUgQyBBUEkgaW50ZXJuYWxseSBp biBXZWJLaXRGaW5kQ29udHJvbGxlcgogICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9z aG93X2J1Zy5jZ2k/aWQ9OTY3NzUKIApkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdDIvVUlQcm9j ZXNzL0xhdW5jaGVyL2d0ay9Qcm9jZXNzTGF1bmNoZXJHdGsuY3BwIGIvU291cmNlL1dlYktpdDIv VUlQcm9jZXNzL0xhdW5jaGVyL2d0ay9Qcm9jZXNzTGF1bmNoZXJHdGsuY3BwCmluZGV4IDhmYjYx OWUuLmY5YjhlNWIgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9MYXVuY2hl ci9ndGsvUHJvY2Vzc0xhdW5jaGVyR3RrLmNwcAorKysgYi9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nl c3MvTGF1bmNoZXIvZ3RrL1Byb2Nlc3NMYXVuY2hlckd0ay5jcHAKQEAgLTYwLDExICs2MCw2IEBA IHN0YXRpYyB2b2lkIGNoaWxkU2V0dXBGdW5jdGlvbihncG9pbnRlciB1c2VyRGF0YSkKIAogICAg IC8vIE1ha2UgY2hpbGQgcHJvY2VzcyBpbmhlcml0IHBhcmVudCdzIGxvY2FsZS4KICAgICBnX3Nl dGVudigiTENfQUxMIiwgc2V0bG9jYWxlKExDX0FMTCwgMCksIFRSVUUpOwotCi0jaWYgT1MoTElO VVgpCi0gICAgLy8gS2lsbCBjaGlsZCBwcm9jZXNzIHdoZW4gcGFyZW50IGRpZXMuCi0gICAgcHJj dGwoUFJfU0VUX1BERUFUSFNJRywgU0lHS0lMTCk7Ci0jZW5kaWYKIH0KIAogc3RhdGljIHZvaWQg Y2hpbGRGaW5pc2hlZEZ1bmN0aW9uKEdQaWQsIGdpbnQgc3RhdHVzLCBncG9pbnRlciB1c2VyRGF0 YSkK