176031 2017-08-28 12:08:38 -0700 [Win] The test http/tests/misc/policy-delegate-called-twice.html is crashing. 2017-08-28 13:14:12 -0700 1 1 1 Unclassified WebKit WebKit Misc. WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 pvollan pvollan achristensen bfulgham webkit-bug-importer oldest_to_newest 1342880 0 pvollan 2017-08-28 12:08:38 -0700 FAULTING_IP: WebKit!WebFrameLoaderClient::dispatchDecidePolicyForResponse+e1 [c:\cygwin\home\buildbot\slave\win-release\build\source\webkitlegacy\win\webcoresupport\webframeloaderclient.cpp @ 530] 6b660a31 8b01 mov eax,dword ptr [ecx] EXCEPTION_RECORD: ffffffffffffffff -- (.exr 0xffffffffffffffff) .exr 0xffffffffffffffff ExceptionAddress: 000000006b660a31 (WebKit!WTF::Function<void __cdecl(enum WebCore::PolicyAction)>::operator()+0x0000000000000007) ExceptionCode: c0000005 (Access violation) ExceptionFlags: 00000000 NumberParameters: 2 Parameter[0]: 0000000000000000 Parameter[1]: 0000000000000000 Attempt to read from address 0000000000000000 STACK_TEXT: 001ce5e4 6b725b0b 0912e7e0 0912e718 001ce654 WebKit!WebFrameLoaderClient::dispatchDecidePolicyForResponse+0xe1 001ce5fc 6b71d605 0912e7e0 001ce654 091b1b70 WebKit!WebCore::FrameLoader::checkContentPolicy+0x2b 001ce6ec 6b71d12b 091b1b70 001ce72c 6bca713a WebKit!WebCore::DocumentLoader::responseReceived+0x435 001ce6f8 6bca713a 091b19f8 091b1b70 71ce6ed0 WebKit!WebCore::DocumentLoader::responseReceived+0xb 001ce72c 6b7336ec 001ceacc 05e0c4a0 091ae290 WebKit!WebCore::CachedRawResource::responseReceived+0x6a 001ceaa4 6b72ebf6 001ceacc 05e0c4a0 001cecf8 WebKit!WebCore::SubresourceLoader::didReceiveResponse+0x30c 001ceab4 6bcb4b8a 091ae870 001ceacc 05de4c60 WebKit!WebCore::ResourceLoader::didReceiveResponse+0x26 001cecf8 6bcb4441 05dcb458 05e0c4a0 001ced2c WebKit!WebCore::SynchronousResourceHandleCFURLConnectionDelegate::didReceiveResponse+0x9a 001ced08 7150c7d4 05dcb458 05e0c4a0 091ae8b8 WebKit!WebCore::ResourceHandleCFURLConnectionDelegate::didReceiveResponseCallback+0x11 WARNING: Stack unwind information not available. Following frames may be wrong. 001ced2c 7150b382 05e0c4a0 001ced58 d1a9a868 CFNetwork!CFHTTPCookieCreateWithResponseHeaderFields+0xb804 001ceeac 7150b4b7 001ceee4 00000003 d1a9abe8 CFNetwork!CFHTTPCookieCreateWithResponseHeaderFields+0xa3b2 001cf014 7150a470 05df5d54 00000003 05de8950 CFNetwork!CFHTTPCookieCreateWithResponseHeaderFields+0xa4e7 001cf038 713c15e2 d1a9b5d8 057824f0 00000000 CFNetwork!CFHTTPCookieCreateWithResponseHeaderFields+0x94a0 001cf0f0 713c10ec 057824fc 001cf128 71621268 CFNetwork+0x115e2 001cf0fc 71621268 05de8950 00662c28 00662c28 CFNetwork+0x110ec 001cf128 716248f5 00662c28 00000001 00662c80 CoreFoundation!CFDateFormatterGetTimeStyle+0x6e03f 001cf18c 71624e11 00662c80 00000000 00000000 CoreFoundation!CFDateFormatterGetTimeStyle+0x716cc 001cf1e0 71624f60 00662c28 7171cfa0 00000000 CoreFoundation!CFDateFormatterGetTimeStyle+0x71be8 001cf328 71d79a7a 001cf340 001e6df0 00552300 CoreFoundation!CFDateFormatterGetTimeStyle+0x71d37 001cfba0 71d79c4e 00000002 00552300 001cfe78 DumpRenderTreeLib!main+0x43a 001cfbb0 001d16c9 00000002 00552300 001e6dec DumpRenderTreeLib!dllLauncherEntryPoint+0xe 001cfe78 001d32ba 00000002 00552300 00555b10 DumpRenderTree!main+0x469 001cfec4 7699336a 7efde000 001cff10 77d79902 DumpRenderTree!__scrt_common_main_seh+0xff 001cfed0 77d79902 7efde000 30e64911 00000000 KERNEL32!BaseThreadInitThunk+0x12 001cff10 77d798d5 001d3337 7efde000 00000000 ntdll_77d40000!RtlInitializeExceptionChain+0x63 001cff28 00000000 001d3337 7efde000 00000000 ntdll_77d40000!RtlInitializeExceptionChain+0x36 1342883 1 319189 pvollan 2017-08-28 12:16:06 -0700 Created attachment 319189 Patch 1342901 2 319189 bfulgham 2017-08-28 12:59:12 -0700 Comment on attachment 319189 Patch r=me. Consider omitting the comments, which I don't think are really necessary. The ChangeLog explains things well, and the code isn't very confusing. 1342903 3 pvollan 2017-08-28 13:03:57 -0700 (In reply to Brent Fulgham from comment #2) > Comment on attachment 319189 [details] > Patch > > r=me. Consider omitting the comments, which I don't think are really > necessary. The ChangeLog explains things well, and the code isn't very > confusing. Thanks for reviewing! I will remove the comments before landing. 1342909 4 pvollan 2017-08-28 13:13:15 -0700 Committed r221268: <https://trac.webkit.org/changeset/221268/webkit> 1342910 5 webkit-bug-importer 2017-08-28 13:14:12 -0700 <rdar://problem/34116974> 319189 2017-08-28 12:16:06 -0700 2017-08-28 12:59:12 -0700 Patch bug-176031-20170828211606.patch text/plain 3251 pvollan SW5kZXg6IFNvdXJjZS9XZWJLaXRMZWdhY3kvd2luL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBT b3VyY2UvV2ViS2l0TGVnYWN5L3dpbi9DaGFuZ2VMb2cJKHJldmlzaW9uIDIyMTI2NCkKKysrIFNv dXJjZS9XZWJLaXRMZWdhY3kvd2luL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsx LDE5IEBACisyMDE3LTA4LTI4ICBQZXIgQXJuZSBWb2xsYW4gIDxwdm9sbGFuQGFwcGxlLmNvbT4K KworICAgICAgICBbV2luXSBUaGUgdGVzdCBodHRwL3Rlc3RzL21pc2MvcG9saWN5LWRlbGVnYXRl LWNhbGxlZC10d2ljZS5odG1sIGlzIGNyYXNoaW5nLgorICAgICAgICBodHRwczovL2J1Z3Mud2Vi a2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTc2MDMxCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9C T0RZIChPT1BTISkuCisKKyAgICAgICAgV2hlbiB0aGUgZnVuY3Rpb24gaW4gdGhlIEZyYW1lUG9s aWN5RnVuY3Rpb24gcGFyYW1ldGVyIGlzIGNhbGxlZCwgaXQgaGFzIGFscmVhZHkgYmVlbiBtb3Zl ZAorICAgICAgICB0byBhbiBpbnRlcm5hbCBtZW1iZXIgYnkgdGhlIHNldFVwUG9saWN5TGlzdGVu ZXIgbWV0aG9kLCBhbmQgaXMgbm8gbG9uZ2VyIHZhbGlkLiBJbnZva2UgdGhlCisgICAgICAgIGZ1 bmN0aW9uIGJ5IHVzaW5nIHRoZSBzdG9yZWQgbWVtYmVyIGluc3RlYWQuCisKKyAgICAgICAgKiBX ZWJDb3JlU3VwcG9ydC9XZWJGcmFtZUxvYWRlckNsaWVudC5jcHA6CisgICAgICAgIChXZWJGcmFt ZUxvYWRlckNsaWVudDo6ZGlzcGF0Y2hEZWNpZGVQb2xpY3lGb3JSZXNwb25zZSk6CisgICAgICAg IChXZWJGcmFtZUxvYWRlckNsaWVudDo6ZGlzcGF0Y2hEZWNpZGVQb2xpY3lGb3JOZXdXaW5kb3dB Y3Rpb24pOgorICAgICAgICAoV2ViRnJhbWVMb2FkZXJDbGllbnQ6OmRpc3BhdGNoRGVjaWRlUG9s aWN5Rm9yTmF2aWdhdGlvbkFjdGlvbik6CisKIDIwMTctMDgtMjIgIFdlbnNvbiBIc2llaCAgPHdl bnNvbl9oc2llaEBhcHBsZS5jb20+CiAKICAgICAgICAgVW5yZXZpZXdlZCwgdHJ5IHRvIGZpeCB0 aGUgV2luZG93cyBidWlsZCBhZnRlciByMjIxMDY0CkluZGV4OiBTb3VyY2UvV2ViS2l0TGVnYWN5 L3dpbi9XZWJDb3JlU3VwcG9ydC9XZWJGcmFtZUxvYWRlckNsaWVudC5jcHAKPT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQot LS0gU291cmNlL1dlYktpdExlZ2FjeS93aW4vV2ViQ29yZVN1cHBvcnQvV2ViRnJhbWVMb2FkZXJD bGllbnQuY3BwCShyZXZpc2lvbiAyMjEyMDApCisrKyBTb3VyY2UvV2ViS2l0TGVnYWN5L3dpbi9X ZWJDb3JlU3VwcG9ydC9XZWJGcmFtZUxvYWRlckNsaWVudC5jcHAJKHdvcmtpbmcgY29weSkKQEAg LTUyNyw3ICs1MjcsOCBAQCB2b2lkIFdlYkZyYW1lTG9hZGVyQ2xpZW50OjpkaXNwYXRjaERlY2lk CiAgICAgaWYgKFNVQ0NFRURFRChwb2xpY3lEZWxlZ2F0ZS0+ZGVjaWRlUG9saWN5Rm9yTUlNRVR5 cGUod2ViVmlldywgQlN0cmluZyhyZXNwb25zZS5taW1lVHlwZSgpKSwgdXJsUmVxdWVzdC5nZXQo KSwgbV93ZWJGcmFtZSwgc2V0VXBQb2xpY3lMaXN0ZW5lcihXVEZNb3ZlKGZ1bmN0aW9uKSkuZ2V0 KCkpKSkKICAgICAgICAgcmV0dXJuOwogCi0gICAgZnVuY3Rpb24oUG9saWN5VXNlKTsKKyAgICAv LyBUaGUgJ2Z1bmN0aW9uJyBwYXJhbWV0ZXIgaGFzIGJlZW4gbW92ZWQgdG8gJ21fcG9saWN5TGlz dGVuZXJQcml2YXRlLT5tX3BvbGljeUZ1bmN0aW9uJy4KKyAgICBtX3BvbGljeUxpc3RlbmVyUHJp dmF0ZS0+bV9wb2xpY3lGdW5jdGlvbihQb2xpY3lVc2UpOwogfQogCiB2b2lkIFdlYkZyYW1lTG9h ZGVyQ2xpZW50OjpkaXNwYXRjaERlY2lkZVBvbGljeUZvck5ld1dpbmRvd0FjdGlvbihjb25zdCBO YXZpZ2F0aW9uQWN0aW9uJiBhY3Rpb24sIGNvbnN0IFJlc291cmNlUmVxdWVzdCYgcmVxdWVzdCwg Rm9ybVN0YXRlKiBmb3JtU3RhdGUsIGNvbnN0IFN0cmluZyYgZnJhbWVOYW1lLCBGcmFtZVBvbGlj eUZ1bmN0aW9uJiYgZnVuY3Rpb24pCkBAIC01NDYsNyArNTQ3LDggQEAgdm9pZCBXZWJGcmFtZUxv YWRlckNsaWVudDo6ZGlzcGF0Y2hEZWNpZAogICAgIGlmIChTVUNDRUVERUQocG9saWN5RGVsZWdh dGUtPmRlY2lkZVBvbGljeUZvck5ld1dpbmRvd0FjdGlvbih3ZWJWaWV3LCBhY3Rpb25JbmZvcm1h dGlvbi5nZXQoKSwgdXJsUmVxdWVzdC5nZXQoKSwgQlN0cmluZyhmcmFtZU5hbWUpLCBzZXRVcFBv bGljeUxpc3RlbmVyKFdURk1vdmUoZnVuY3Rpb24pKS5nZXQoKSkpKQogICAgICAgICByZXR1cm47 CiAKLSAgICBmdW5jdGlvbihQb2xpY3lVc2UpOworICAgIC8vIFRoZSAnZnVuY3Rpb24nIHBhcmFt ZXRlciBoYXMgYmVlbiBtb3ZlZCB0byAnbV9wb2xpY3lMaXN0ZW5lclByaXZhdGUtPm1fcG9saWN5 RnVuY3Rpb24nLgorICAgIG1fcG9saWN5TGlzdGVuZXJQcml2YXRlLT5tX3BvbGljeUZ1bmN0aW9u KFBvbGljeVVzZSk7CiB9CiAKIHZvaWQgV2ViRnJhbWVMb2FkZXJDbGllbnQ6OmRpc3BhdGNoRGVj aWRlUG9saWN5Rm9yTmF2aWdhdGlvbkFjdGlvbihjb25zdCBOYXZpZ2F0aW9uQWN0aW9uJiBhY3Rp b24sIGNvbnN0IFJlc291cmNlUmVxdWVzdCYgcmVxdWVzdCwgYm9vbCwgRm9ybVN0YXRlKiBmb3Jt U3RhdGUsIEZyYW1lUG9saWN5RnVuY3Rpb24mJiBmdW5jdGlvbikKQEAgLTU2NSw3ICs1NjcsOCBA QCB2b2lkIFdlYkZyYW1lTG9hZGVyQ2xpZW50OjpkaXNwYXRjaERlY2lkCiAgICAgaWYgKFNVQ0NF RURFRChwb2xpY3lEZWxlZ2F0ZS0+ZGVjaWRlUG9saWN5Rm9yTmF2aWdhdGlvbkFjdGlvbih3ZWJW aWV3LCBhY3Rpb25JbmZvcm1hdGlvbi5nZXQoKSwgdXJsUmVxdWVzdC5nZXQoKSwgbV93ZWJGcmFt ZSwgc2V0VXBQb2xpY3lMaXN0ZW5lcihXVEZNb3ZlKGZ1bmN0aW9uKSkuZ2V0KCkpKSkKICAgICAg ICAgcmV0dXJuOwogCi0gICAgZnVuY3Rpb24oUG9saWN5VXNlKTsKKyAgICAvLyBUaGUgJ2Z1bmN0 aW9uJyBwYXJhbWV0ZXIgaGFzIGJlZW4gbW92ZWQgdG8gJ21fcG9saWN5TGlzdGVuZXJQcml2YXRl LT5tX3BvbGljeUZ1bmN0aW9uJy4KKyAgICBtX3BvbGljeUxpc3RlbmVyUHJpdmF0ZS0+bV9wb2xp Y3lGdW5jdGlvbihQb2xpY3lVc2UpOwogfQogCiB2b2lkIFdlYkZyYW1lTG9hZGVyQ2xpZW50Ojpk aXNwYXRjaFVuYWJsZVRvSW1wbGVtZW50UG9saWN5KGNvbnN0IFJlc291cmNlRXJyb3ImIGVycm9y KQo=