138500 2014-11-07 02:29:45 -0800 WTR crashes after running a test when NetworkProcess enabled 2014-11-08 02:47:13 -0800 1 1 1 Unclassified WebKit Tools / Tests 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 138428 1 cgarcia webkit-unassigned ap buildbot rniwa oldest_to_newest 1047053 0 cgarcia 2014-11-07 02:29:45 -0800 Program received signal SIGSEGV, Segmentation fault. 0x00007ffff53b0b38 in WebKit::WebNotificationManagerProxy::initializeProvider(WKNotificationProviderBase const*) () from WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37 (gdb) bt #0 0x00007ffff53b0b38 in WebKit::WebNotificationManagerProxy::initializeProvider(WKNotificationProviderBase const*) () from WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37 #1 0x0000000000418476 in WTR::WebNotificationProvider::~WebNotificationProvider() () #2 0x000000000040fc33 in WTR::TestController::~TestController() () #3 0x000000000040c250 in main () I don't know why it only happens with the network process enabled, though. The thing is that WebNotificationProvider destructor is calling WKNotificationManagerSetProvider, but m_notificationManager is null because removeNotificationManager was already called. 1047054 1 241170 cgarcia 2014-11-07 02:33:01 -0800 Created attachment 241170 Patch 1047249 2 241170 ap 2014-11-07 16:48:08 -0800 Comment on attachment 241170 Patch This seems OK to do. The difference is that on Mac, WebContext is destroyed after WebNotificationProvider, so removeNotificationManager is never called. 1047277 3 cgarcia 2014-11-08 02:47:13 -0800 Committed r175774: <http://trac.webkit.org/changeset/175774> 241170 2014-11-07 02:33:01 -0800 2014-11-07 16:48:08 -0800 Patch wtr-crash.diff text/plain 1556 cgarcia ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCBjMjBk Mjc3Li41MzJmOGNlIDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh bmdlTG9nCkBAIC0xLDMgKzEsMTggQEAKKzIwMTQtMTEtMDcgIENhcmxvcyBHYXJjaWEgQ2FtcG9z ICA8Y2dhcmNpYUBpZ2FsaWEuY29tPgorCisgICAgICAgIFdUUiBjcmFzaGVzIGFmdGVyIHJ1bm5p bmcgYSB0ZXN0IHdoZW4gTmV0d29ya1Byb2Nlc3MgZW5hYmxlZAorICAgICAgICBodHRwczovL2J1 Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTM4NTAwCisKKyAgICAgICAgUmV2aWV3ZWQg YnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgV2ViTm90aWZpY2F0aW9uUHJvdmlkZXIgZGVz dHJ1Y3RvciBpcyBjYWxsaW5nCisgICAgICAgIFdLTm90aWZpY2F0aW9uTWFuYWdlclNldFByb3Zp ZGVyLCBidXQgbV9ub3RpZmljYXRpb25NYW5hZ2VyIGlzCisgICAgICAgIG51bGwgYmVjYXVzZSBy ZW1vdmVOb3RpZmljYXRpb25NYW5hZ2VyIHdhcyBhbHJlYWR5IGNhbGxlZC4KKworICAgICAgICAq IFdlYktpdFRlc3RSdW5uZXIvV2ViTm90aWZpY2F0aW9uUHJvdmlkZXIuY3BwOgorICAgICAgICAo V1RSOjpXZWJOb3RpZmljYXRpb25Qcm92aWRlcjo6fldlYk5vdGlmaWNhdGlvblByb3ZpZGVyKTog Q2hlY2sKKyAgICAgICAgbV9ub3RpZmljYXRpb25NYW5hZ2VyIGlzIG5vdCBudWxsIGJlZm9yZSBj YWxsaW5nIFdLTm90aWZpY2F0aW9uTWFuYWdlclNldFByb3ZpZGVyLgorCiAyMDE0LTExLTA2ICBK YWtlIE5pZWxzZW4gIDxqYWNvYl9uaWVsc2VuQGFwcGxlLmNvbT4KIAogICAgICAgICBSZW1vdmUg ZHVwbGljYXRlIGNvZGUgZnJvbSBQYXRjaEFuYWx5c2lzVGFzay5fdGVzdF9wYXRjaCBhbmQgZml4 IGJ1ZwpkaWZmIC0tZ2l0IGEvVG9vbHMvV2ViS2l0VGVzdFJ1bm5lci9XZWJOb3RpZmljYXRpb25Q cm92aWRlci5jcHAgYi9Ub29scy9XZWJLaXRUZXN0UnVubmVyL1dlYk5vdGlmaWNhdGlvblByb3Zp ZGVyLmNwcAppbmRleCBkNDNiMzI5Li42OWM3MTc1IDEwMDY0NAotLS0gYS9Ub29scy9XZWJLaXRU ZXN0UnVubmVyL1dlYk5vdGlmaWNhdGlvblByb3ZpZGVyLmNwcAorKysgYi9Ub29scy9XZWJLaXRU ZXN0UnVubmVyL1dlYk5vdGlmaWNhdGlvblByb3ZpZGVyLmNwcApAQCAtNjUsNyArNjUsOCBAQCBX ZWJOb3RpZmljYXRpb25Qcm92aWRlcjo6V2ViTm90aWZpY2F0aW9uUHJvdmlkZXIoKQogCiBXZWJO b3RpZmljYXRpb25Qcm92aWRlcjo6fldlYk5vdGlmaWNhdGlvblByb3ZpZGVyKCkKIHsKLSAgICBX S05vdGlmaWNhdGlvbk1hbmFnZXJTZXRQcm92aWRlcihtX25vdGlmaWNhdGlvbk1hbmFnZXIuZ2V0 KCksIDApOworICAgIGlmIChtX25vdGlmaWNhdGlvbk1hbmFnZXIpCisgICAgICAgIFdLTm90aWZp Y2F0aW9uTWFuYWdlclNldFByb3ZpZGVyKG1fbm90aWZpY2F0aW9uTWFuYWdlci5nZXQoKSwgbnVs bHB0cik7CiB9CiAKIFdLTm90aWZpY2F0aW9uUHJvdmlkZXJWMCBXZWJOb3RpZmljYXRpb25Qcm92 aWRlcjo6cHJvdmlkZXIoKQo=