84463 2012-04-20 09:52:32 -0700 Crashes in WebProcess at AccessibilityRenderObject::isAttachment 2012-04-26 00:36:43 -0700 1 1 1 Unclassified WebKit Accessibility 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 cfleizach cfleizach aboxhall adele bdakin enrica rniwa webkit.review.bot oldest_to_newest 606653 0 cfleizach 2012-04-20 09:52:32 -0700 * BACKTRACE ('>' indicates stack frame used for CrashTracer aggregation) 1 page zero 000000000000000000 0 + 0 > 2 com.apple.WebCore 0x7fff86940c9d WebCore::AccessibilityRenderObject::isAttachment() const + 0x1d 3 com.apple.WebCore 0x7fff86ddc092 WebCore::AccessibilityObject::detachFromParent() + 0x12 4 com.apple.WebCore 0x7fff869475c1 WebCore::AccessibilityRenderObject::clearChildren() + 0x31 5 com.apple.WebCore 0x7fff86947565 WebCore::AccessibilityRenderObject::detach() + 0x15 6 com.apple.WebCore 0x7fff86947378 WebCore::AXObjectCache::~AXObjectCache() + 0x78 7 com.apple.WebCore 0x7fff868f9002 WebCore::Document::detach() + 0x122 8 com.apple.WebCore 0x7fff8689a3b8 WebCore::Frame::setView(WTF::PassRefPtr<WebCore::FrameView>) + 0x48 9 com.apple.WebCore 0x7fff86fe3571 WebCore::Frame::createView(WebCore::IntSize const&, WebCore::Color const&, bool, WebCore::IntSize const&, bool, WebCore::ScrollbarMode, bool, WebCore::ScrollbarMode, bool) + 0x61 10 com.apple.WebKit2 0x7fff890f4eef WebKit::WebFrameLoaderClient::transitionToCommittedForNewPage() + 0xb9 11 com.apple.WebCore 0x7fff86899a3c WebCore::FrameLoader::transitionToCommitted(WTF::PassRefPtr<WebCore::CachedPage>) + 0x29c 12 com.apple.WebCore 0x7fff86899018 WebCore::FrameLoader::commitProvisionalLoad() + 0x158 13 com.apple.WebCore 0x7fff8690ac05 606654 1 cfleizach 2012-04-20 09:53:23 -0700 It looks like the problem is that it's possible to ask for the focused ui element through WK2 without actually enabling accessibility. 606672 2 138111 cfleizach 2012-04-20 10:17:54 -0700 Created attachment 138111 patch 607483 3 rniwa 2012-04-22 23:05:37 -0700 Not sure if we want to enable accessibility on demand like this. 607485 4 cfleizach 2012-04-22 23:06:44 -0700 (In reply to comment #3) > Not sure if we want to enable accessibility on demand like this. yes we do. it's an oversight in WK2 that it's not done here 610406 5 138111 webkit.review.bot 2012-04-26 00:36:38 -0700 Comment on attachment 138111 patch Clearing flags on attachment: 138111 Committed r115295: <http://trac.webkit.org/changeset/115295> 610407 6 webkit.review.bot 2012-04-26 00:36:43 -0700 All reviewed patches have been landed. Closing bug. 138111 2012-04-20 10:17:54 -0700 2012-04-26 00:36:38 -0700 patch patch.txt text/plain 3347 cfleizach SW5kZXg6IFNvdXJjZS9XZWJLaXQyL1dlYlByb2Nlc3MvV2ViUGFnZS9tYWMvV0tBY2Nlc3NpYmls aXR5V2ViUGFnZU9iamVjdC5tbQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViS2l0Mi9XZWJQcm9j ZXNzL1dlYlBhZ2UvbWFjL1dLQWNjZXNzaWJpbGl0eVdlYlBhZ2VPYmplY3QubW0JKHJldmlzaW9u IDExNDcyMSkKKysrIFNvdXJjZS9XZWJLaXQyL1dlYlByb2Nlc3MvV2ViUGFnZS9tYWMvV0tBY2Nl c3NpYmlsaXR5V2ViUGFnZU9iamVjdC5tbQkod29ya2luZyBjb3B5KQpAQCAtNDMsNiArNDMsOSBA QAogCiAtIChpZClhY2Nlc3NpYmlsaXR5Um9vdE9iamVjdFdyYXBwZXIKIHsKKyAgICBpZiAoIVdl YkNvcmU6OkFYT2JqZWN0Q2FjaGU6OmFjY2Vzc2liaWxpdHlFbmFibGVkKCkpCisgICAgICAgIFdl YkNvcmU6OkFYT2JqZWN0Q2FjaGU6OmVuYWJsZUFjY2Vzc2liaWxpdHkoKTsKKwogICAgIFdlYkNv cmU6OlBhZ2UqIHBhZ2UgPSBtX3BhZ2UtPmNvcmVQYWdlKCk7CiAgICAgaWYgKCFwYWdlKQogICAg ICAgICByZXR1cm4gbmlsOwpJbmRleDogU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCj09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT0KLS0tIFNvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZwkocmV2aXNpb24gMTE0NzU3KQorKysgU291 cmNlL1dlYktpdDIvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTMgQEAKKzIw MTItMDQtMjAgIENocmlzIEZsZWl6YWNoMiAgPGNmbGVpemFjaEBhcHBsZS5jb20+CisKKyAgICAg ICAgQ3Jhc2hUcmFjZXI6IFtVU0VSXSAxNTcgY3Jhc2hlcyBpbiBXZWJQcm9jZXNzIGF0IGNvbS5h cHBsZS5XZWJDb3JlOiBXZWJDb3JlOjpBY2Nlc3NpYmlsaXR5UmVuZGVyT2JqZWN0Ojppc0F0dGFj aG1lbnQgY29uc3QgKyAyOQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1 Zy5jZ2k/aWQ9ODQ0NjMKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKwor ICAgICAgICAqIFdlYlByb2Nlc3MvV2ViUGFnZS9tYWMvV0tBY2Nlc3NpYmlsaXR5V2ViUGFnZU9i amVjdC5tbToKKyAgICAgICAgKFdlYktpdCk6CisKIDIwMTItMDQtMTkgIENhcmxvcyBHYXJjaWEg Q2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29tPgogCiAgICAgICAgIFtHVEtdIElnbm9yZSByZXNv dXJjZXMgd2hpbGUgcmVwbGFjaW5nIGNvbnRlbnQgaW4gV2ViS2l0MiBHVEsrIEFQSQpJbmRleDog U291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJDb3JlL0No YW5nZUxvZwkocmV2aXNpb24gMTE0NzU3KQorKysgU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCSh3 b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTkgQEAKKzIwMTItMDQtMjAgIENocmlzIEZsZWl6YWNo MiAgPGNmbGVpemFjaEBhcHBsZS5jb20+CisKKyAgICAgICAgQ3Jhc2hUcmFjZXI6IFtVU0VSXSAx NTcgY3Jhc2hlcyBpbiBXZWJQcm9jZXNzIGF0IGNvbS5hcHBsZS5XZWJDb3JlOiBXZWJDb3JlOjpB Y2Nlc3NpYmlsaXR5UmVuZGVyT2JqZWN0Ojppc0F0dGFjaG1lbnQgY29uc3QgKyAyOQorICAgICAg ICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9ODQ0NjMKKworICAgICAg ICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBBY2Nlc3NpYmlsaXR5IHdh cyBub3QgYmVpbmcgZW5hYmxlZCB3aGVuIFdLMiB3YXMgYXNraW5nIG9ubHkgZm9yIHRoZSBmb2N1 c2VkIFVJIGVsZW1lbnQuCisKKyAgICAgICAgTm8gbGF5b3V0IHRlc3QgY291bGQgYmUgd3JpdHRl biBiZWNhdXNlIHRoZSBXS1Rlc3RSdW5uZXIgbWVjaGFuaXNtIHdvcmtzIGRpZmZlcmVudGx5IHdo ZW4gYXNraW5nIGZvciB0aGlzLgorCisgICAgICAgICogYWNjZXNzaWJpbGl0eS9BWE9iamVjdENh Y2hlLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OkFYT2JqZWN0Q2FjaGU6OmZvY3VzZWRVSUVsZW1l bnRGb3JQYWdlKToKKyAgICAgICAgKFdlYkNvcmU6OkFYT2JqZWN0Q2FjaGU6OnJvb3RPYmplY3Qp OgorICAgICAgICAoV2ViQ29yZTo6QVhPYmplY3RDYWNoZTo6cm9vdE9iamVjdEZvckZyYW1lKToK KwogMjAxMi0wNC0yMCAgQWxsYW4gU2FuZGZlbGQgSmVuc2VuICA8YWxsYW4uamVuc2VuQG5va2lh LmNvbT4KIAogICAgICAgICBbUXRdIFBsYXRmb3JtTW91c2VFdmVudFF0LmNwcCBzaG91bGQgYmUg ZGVsZXRlZC4KSW5kZXg6IFNvdXJjZS9XZWJDb3JlL2FjY2Vzc2liaWxpdHkvQVhPYmplY3RDYWNo ZS5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PQotLS0gU291cmNlL1dlYkNvcmUvYWNjZXNzaWJpbGl0eS9BWE9iamVj dENhY2hlLmNwcAkocmV2aXNpb24gMTE0NzIxKQorKysgU291cmNlL1dlYkNvcmUvYWNjZXNzaWJp bGl0eS9BWE9iamVjdENhY2hlLmNwcAkod29ya2luZyBjb3B5KQpAQCAtMTI5LDYgKzEyOSw5IEBA CiAgICAgCiBBY2Nlc3NpYmlsaXR5T2JqZWN0KiBBWE9iamVjdENhY2hlOjpmb2N1c2VkVUlFbGVt ZW50Rm9yUGFnZShjb25zdCBQYWdlKiBwYWdlKQogeworICAgIGlmICghZ0FjY2Vzc2liaWxpdHlF bmFibGVkKQorICAgICAgICByZXR1cm4gMDsKKwogICAgIC8vIGdldCB0aGUgZm9jdXNlZCBub2Rl IGluIHRoZSBwYWdlCiAgICAgRG9jdW1lbnQqIGZvY3VzZWREb2N1bWVudCA9IHBhZ2UtPmZvY3Vz Q29udHJvbGxlcigpLT5mb2N1c2VkT3JNYWluRnJhbWUoKS0+ZG9jdW1lbnQoKTsKICAgICBOb2Rl KiBmb2N1c2VkTm9kZSA9IGZvY3VzZWREb2N1bWVudC0+Zm9jdXNlZE5vZGUoKTsKQEAgLTI4OCwx MSArMjkxLDE3IEBACiAgICAgCiBBY2Nlc3NpYmlsaXR5T2JqZWN0KiBBWE9iamVjdENhY2hlOjpy b290T2JqZWN0KCkKIHsKKyAgICBpZiAoIWdBY2Nlc3NpYmlsaXR5RW5hYmxlZCkKKyAgICAgICAg cmV0dXJuIDA7CisgICAgCiAgICAgcmV0dXJuIGdldE9yQ3JlYXRlKG1fZG9jdW1lbnQtPnZpZXco KSk7CiB9CiAKIEFjY2Vzc2liaWxpdHlPYmplY3QqIEFYT2JqZWN0Q2FjaGU6OnJvb3RPYmplY3RG b3JGcmFtZShGcmFtZSogZnJhbWUpCiB7CisgICAgaWYgKCFnQWNjZXNzaWJpbGl0eUVuYWJsZWQp CisgICAgICAgIHJldHVybiAwOworCiAgICAgaWYgKCFmcmFtZSkKICAgICAgICAgcmV0dXJuIDA7 CiAgICAgcmV0dXJuIGdldE9yQ3JlYXRlKGZyYW1lLT52aWV3KCkpOwo=