127387 2014-01-21 17:55:10 -0800 AX: Crash in WebKit::WebPageProxy::registerWebProcessAccessibilityToken 2014-02-28 14:58:33 -0800 1 1 1 Unclassified WebKit Accessibility 528+ (Nightly build) All All RESOLVED FIXED InRadar P2 Normal --- 1 cfleizach cfleizach commit-queue webkit-bug-importer oldest_to_newest 970501 0 cfleizach 2014-01-21 17:55:10 -0800 37 com.apple.WebKit2: WebKit::WebPageProxy::registerWebProcessAccessibilityToken + 11 <== 37 com.apple.WebKit2: WebKit::WebPageProxy::didReceiveMessage + 7257 37 com.apple.WebKit2: CoreIPC::MessageReceiverMap::dispatchMessage + 125 37 com.apple.WebKit2: WebKit::ChildProcessProxy::dispatchMessage + 13 37 com.apple.WebKit2: WebKit::WebProcessProxy::didReceiveMessage + 24 37 com.apple.WebKit2: CoreIPC::Connection::dispatchMessage + 101 37 com.apple.WebKit2: CoreIPC::Connection::dispatchOneMessage + 106 28 com.apple.WebCore: WebCore::RunLoop::performWork + 270 | 28 com.apple.WebCore: WebCore::RunLoop::performWork + 34 | 28 com.apple.CoreFoundation: __CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION__ + 17 | 28 com.apple.CoreFoundation: __CFRunLoopDoSources0 + 242 | 28 com.apple.CoreFoundation: __CFRunLoopRun + 831 | 28 com.apple.CoreFoundation: CFRunLoopRunSpecific + 309 | 28 com.apple.HIToolbox: RunCurrentEventLoopInMode + 226 | 28 com.apple.HIToolbox: ReceiveNextEventCommon + 479 | 28 com.apple.HIToolbox: _BlockUntilNextEventMatchingListInModeWithFilter + 65 | 28 com.apple.AppKit: _DPSNextEvent + 1434 | 28 com.apple.AppKit: -[NSApplication nextEventMatchingMask:untilDate:inMode:dequeue:] + 122 | 28 com.apple.AppKit: -[NSApplication run] + 553 | 28 com.apple.quicklook.ui.helper: main | 28 libdyld.dylib: start + 1 9 com.apple.WebCore: WebCore::RunLoop::performWork + 557 9 com.apple.WebCore: WebCore::RunLoop::performWork + 34 9 com.apple.CoreFoundation: __CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION__ + 17 9 com.apple.CoreFoundation: __CFRunLoopDoSources0 + 242 9 com.apple.CoreFoundation: __CFRunLoopRun + 831 9 com.apple.CoreFoundation: CFRunLoopRunSpecific + 309 9 com.apple.HIToolbox: RunCurrentEventLoopInMode + 226 9 com.apple.HIToolbox: ReceiveNextEventCommon + 479 9 com.apple.HIToolbox: _BlockUntilNextEventMatchingListInModeWithFilter + 65 9 com.apple.AppKit: _DPSNextEvent + 1434 9 com.apple.AppKit: -[NSApplication nextEventMatchingMask:untilDate:inMode:dequeue:] + 122 9 com.apple.AppKit: -[NSApplication run] + 553 9 com.apple.quicklook.ui.helper: main 9 libdyld.dylib: start + 1 <rdar://problem/15723355> <rdar://probl;e 970502 1 221815 cfleizach 2014-01-21 17:56:54 -0800 Created attachment 221815 patch 970731 2 221815 ap 2014-01-22 09:59:27 -0800 Comment on attachment 221815 patch View in context: https://bugs.webkit.org/attachment.cgi?id=221815&action=review > Source/WebKit2/UIProcess/mac/WebPageProxyMac.mm:390 > + if (!isValid()) > + return; Is it semantically correct to ignore this call, or should the data be queued and sent once the process is (re)started? 970806 3 cfleizach 2014-01-22 11:50:41 -0800 (In reply to comment #2) > (From update of attachment 221815 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=221815&action=review > > > Source/WebKit2/UIProcess/mac/WebPageProxyMac.mm:390 > > + if (!isValid()) > > + return; > > Is it semantically correct to ignore this call, or should the data be queued and sent once the process is (re)started? Good question. I should play with this some more 985727 4 221815 commit-queue 2014-02-28 14:58:31 -0800 Comment on attachment 221815 patch Clearing flags on attachment: 221815 Committed r164887: <http://trac.webkit.org/changeset/164887> 985728 5 commit-queue 2014-02-28 14:58:33 -0800 All reviewed patches have been landed. Closing bug. 221815 2014-01-21 17:56:54 -0800 2014-02-28 14:58:31 -0800 patch patch text/plain 1305 cfleizach SW5kZXg6IFNvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi S2l0Mi9DaGFuZ2VMb2cJKHJldmlzaW9uIDE2MjQ4OSkKKysrIFNvdXJjZS9XZWJLaXQyL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE1IEBACisyMDE0LTAxLTIxICBDaHJpcyBG bGVpemFjaCAgPGNmbGVpemFjaEBhcHBsZS5jb20+CisKKyAgICAgICAgQVg6IENyYXNoIGluIFdl YktpdDo6V2ViUGFnZVByb3h5OjpyZWdpc3RlcldlYlByb2Nlc3NBY2Nlc3NpYmlsaXR5VG9rZW4K KyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEyNzM4Nwor CisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFNwZWN1bGF0 aXZlIGZpeCBmb3IgY3Jhc2hlci4gV2Ugc2hvdWxkIGNoZWNrIGlmIHRoZSBjb25uZWN0aW9uIGlz IHN0aWxsIHZhbGlkIGJlZm9yZSBwcm9jZWVkaW5nLgorCisgICAgICAgICogVUlQcm9jZXNzL21h Yy9XZWJQYWdlUHJveHlNYWMubW06CisgICAgICAgIChXZWJLaXQ6OldlYlBhZ2VQcm94eTo6cmVn aXN0ZXJXZWJQcm9jZXNzQWNjZXNzaWJpbGl0eVRva2VuKToKKwogMjAxNC0wMS0yMSAgVGltIEhv cnRvbiAgPHRpbW90aHlfaG9ydG9uQGFwcGxlLmNvbT4KIAogICAgICAgICBEb24ndCBjb3B5IFZp ZXdHZXN0dXJlR2VvbWV0cnlDb2xsZWN0b3IubWVzc2FnZXMuaW4gaW50byB0aGUgV2ViUHJvY2Vz cyBidW5kbGUuCkluZGV4OiBTb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvbWFjL1dlYlBhZ2VQcm94 eU1hYy5tbQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvbWFjL1dlYlBh Z2VQcm94eU1hYy5tbQkocmV2aXNpb24gMTYyMjA1KQorKysgU291cmNlL1dlYktpdDIvVUlQcm9j ZXNzL21hYy9XZWJQYWdlUHJveHlNYWMubW0JKHdvcmtpbmcgY29weSkKQEAgLTM4Niw2ICszODYs OSBAQAogICAgIAogdm9pZCBXZWJQYWdlUHJveHk6OnJlZ2lzdGVyV2ViUHJvY2Vzc0FjY2Vzc2li aWxpdHlUb2tlbihjb25zdCBJUEM6OkRhdGFSZWZlcmVuY2UmIGRhdGEpCiB7CisgICAgaWYgKCFp c1ZhbGlkKCkpCisgICAgICAgIHJldHVybjsKKyAgICAKICAgICBtX3BhZ2VDbGllbnQuYWNjZXNz aWJpbGl0eVdlYlByb2Nlc3NUb2tlblJlY2VpdmVkKGRhdGEpOwogfSAgICAKICAgICAK