45501 2010-09-09 16:15:03 -0700 AX: Mail crashes with searching for next misspelled word with VoiceOver 2010-09-10 09:39:27 -0700 1 1 1 Unclassified WebKit Accessibility 528+ (Nightly build) PC OS X 10.5 RESOLVED FIXED P2 Normal --- 1 cfleizach cfleizach bdakin ddkilzer oldest_to_newest 276825 0 cfleizach 2010-09-09 16:15:03 -0700 -Start VoiceOver -Open mail.app -Create a new message -Leave to/subject/etc black -Click on compose area -type "This is a tesst test" -Note that on every keypress VO will output "The next text marker just equaled the current text marker. This is a bug in WebKit. Please inform them" -Press VO-CMD-E to find the next misspelled word happens every time > 1 com.apple.WebCore 0x7fff9f2b8947 -[AccessibilityObjectWrapper accessibilityAttributeValue:forParameter:] + 0x5a7 2 com.apple.AppKit 0x7fffa092ea37 CopyParameterizedAttributeValue + 0x147 3 com.apple.HIServices 0x7fffa0521dbc _AXXMIGCopyParameterizedAttributeValue + 0x107 4 com.apple.HIServices 0x7fffa052c376 _XCopyParameterizedAttributeValue + 0x2f1 5 com.apple.HIServices 0x7fffa0508406 mshMIGPerform + 0x234 6 com.apple.CoreFoundation 0x7fffa31b047c __CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE1_PERFORM_FUNCTION__ + 0x2c 276827 1 67110 cfleizach 2010-09-09 16:17:35 -0700 Created attachment 67110 patch 277176 2 67110 ddkilzer 2010-09-10 07:28:52 -0700 Comment on attachment 67110 patch View in context: https://bugs.webkit.org/attachment.cgi?id=67110&action=prettypatch > WebCore/accessibility/mac/AccessibilityObjectWrapper.mm:1979 > + return YES; Is this fix related to this bug? If not, it should really be split into another bug report/commit. If you don't split it out, at least make a note that this fix is incidental and unrelated to the fix for the crash. 277247 3 cfleizach 2010-09-10 09:29:34 -0700 (In reply to comment #2) > (From update of attachment 67110 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=67110&action=prettypatch > > > WebCore/accessibility/mac/AccessibilityObjectWrapper.mm:1979 > > + return YES; > > Is this fix related to this bug? If not, it should really be split into another bug report/commit. > will separate into another bug. its always hard for me to pass up opportunity! 277255 4 cfleizach 2010-09-10 09:39:24 -0700 http://trac.webkit.org/changeset/67209 67110 2010-09-09 16:17:35 -0700 2010-09-10 07:28:52 -0700 patch patch.txt text/plain 5367 cfleizach SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiA2NzEyMikKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTggQEAKKzIwMTAtMDktMDkgIENocmlzIEZsZWl6YWNoICA8Y2ZsZWl6YWNoQGFw cGxlLmNvbT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAg ICBNYWlsIGNyYXNoZXMgd2l0aCBzZWFyY2hpbmcgZm9yIG5leHQgbWlzc3BlbGxlZCB3b3JkIHdp dGggVm9pY2VPdmVyCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNn aT9pZD00NTUwMQorCisgICAgICAgIFRlc3Q6IHBsYXRmb3JtL21hYy9hY2Nlc3NpYmlsaXR5L2Ny YXNoLWluLWVsZW1lbnQtZm9yLXRleHQtbWFya2VyLmh0bWwKKworICAgICAgICAqIGFjY2Vzc2li aWxpdHkvbWFjL0FjY2Vzc2liaWxpdHlPYmplY3RXcmFwcGVyLm1tOgorICAgICAgICAoLVtBY2Nl c3NpYmlsaXR5T2JqZWN0V3JhcHBlciBhY2Nlc3NpYmlsaXR5SXNJZ25vcmVkXSk6CisgICAgICAg ICAgICBUaGlzIHdhcyByZXR1cm5pbmcgbmlsIGluIGZhaWx1cmUgY2FzZXMgZXZlbiB0aG91Z2gg aXQgc2hvdWxkIHJldHVybiBhIEJPT0wuCisgICAgICAgICgtW0FjY2Vzc2liaWxpdHlPYmplY3RX cmFwcGVyIGFjY2Vzc2liaWxpdHlBdHRyaWJ1dGVWYWx1ZTpmb3JQYXJhbWV0ZXI6XSk6CisgICAg ICAgICAgICBDaGVjayB0aGF0IHRoZSBBWCBvYmplY3QgaXMgbm90IG5pbCBiZWZvcmUgYXNraW5n IGZvciB0aGUgd3JhcHBlcigpLgorCiAyMDEwLTA5LTA4ICBEYXJpbiBBZGxlciAgPGRhcmluQGFw cGxlLmNvbT4KIAogICAgICAgICBSZXZpZXdlZCBieSBBZGFtIEJhcnRoLgpJbmRleDogV2ViQ29y ZS9hY2Nlc3NpYmlsaXR5L21hYy9BY2Nlc3NpYmlsaXR5T2JqZWN0V3JhcHBlci5tbQo9PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09Ci0tLSBXZWJDb3JlL2FjY2Vzc2liaWxpdHkvbWFjL0FjY2Vzc2liaWxpdHlPYmplY3RXcmFw cGVyLm1tCShyZXZpc2lvbiA2NzExOCkKKysrIFdlYkNvcmUvYWNjZXNzaWJpbGl0eS9tYWMvQWNj ZXNzaWJpbGl0eU9iamVjdFdyYXBwZXIubW0JKHdvcmtpbmcgY29weSkKQEAgLTE5NzIsMTEgKzE5 NzIsMTEgQEAgLSAoQk9PTClhY2Nlc3NpYmlsaXR5SXNBdHRyaWJ1dGVTZXR0YWJsZQogLSAoQk9P TClhY2Nlc3NpYmlsaXR5SXNJZ25vcmVkCiB7CiAgICAgaWYgKCFtX29iamVjdCkKLSAgICAgICAg cmV0dXJuIG5pbDsKKyAgICAgICAgcmV0dXJuIFlFUzsKIAogICAgIG1fb2JqZWN0LT51cGRhdGVC YWNraW5nU3RvcmUoKTsKICAgICBpZiAoIW1fb2JqZWN0KQotICAgICAgICByZXR1cm4gbmlsOwor ICAgICAgICByZXR1cm4gWUVTOwogCiAgICAgaWYgKG1fb2JqZWN0LT5pc0F0dGFjaG1lbnQoKSkK ICAgICAgICAgcmV0dXJuIFtbc2VsZiBhdHRhY2htZW50Vmlld10gYWNjZXNzaWJpbGl0eUlzSWdu b3JlZF07CkBAIC0yMzU0LDcgKzIzNTQsMTAgQEAgLSAoaWQpYWNjZXNzaWJpbGl0eUF0dHJpYnV0 ZVZhbHVlOihOU1N0cgogICAgIC8vIGRpc3BhdGNoCiAgICAgaWYgKFthdHRyaWJ1dGUgaXNFcXVh bFRvU3RyaW5nOkAiQVhVSUVsZW1lbnRGb3JUZXh0TWFya2VyIl0pIHsKICAgICAgICAgVmlzaWJs ZVBvc2l0aW9uIHZpc2libGVQb3MgPSB2aXNpYmxlUG9zaXRpb25Gb3JUZXh0TWFya2VyKHRleHRN YXJrZXIpOwotICAgICAgICByZXR1cm4gbV9vYmplY3QtPmFjY2Vzc2liaWxpdHlPYmplY3RGb3JQ b3NpdGlvbih2aXNpYmxlUG9zKS0+d3JhcHBlcigpOworICAgICAgICBBY2Nlc3NpYmlsaXR5T2Jq ZWN0KiBheE9iamVjdCA9IG1fb2JqZWN0LT5hY2Nlc3NpYmlsaXR5T2JqZWN0Rm9yUG9zaXRpb24o dmlzaWJsZVBvcyk7CisgICAgICAgIGlmICghYXhPYmplY3QpCisgICAgICAgICAgICByZXR1cm4g bmlsOworICAgICAgICByZXR1cm4gYXhPYmplY3QtPndyYXBwZXIoKTsKICAgICB9CiAKICAgICBp ZiAoW2F0dHJpYnV0ZSBpc0VxdWFsVG9TdHJpbmc6QCJBWFRleHRNYXJrZXJSYW5nZUZvclVJRWxl bWVudCJdKSB7CkluZGV4OiBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5 b3V0VGVzdHMvQ2hhbmdlTG9nCShyZXZpc2lvbiA2NzEyMykKKysrIExheW91dFRlc3RzL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDEzIEBACisyMDEwLTA5LTA5ICBDaHJpcyBG bGVpemFjaCAgPGNmbGVpemFjaEBhcHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9C T0RZIChPT1BTISkuCisKKyAgICAgICAgTWFpbCBjcmFzaGVzIHdpdGggc2VhcmNoaW5nIGZvciBu ZXh0IG1pc3NwZWxsZWQgd29yZCB3aXRoIFZvaWNlT3ZlcgorICAgICAgICBodHRwczovL2J1Z3Mu d2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NDU1MDEKKworICAgICAgICAqIHBsYXRmb3JtL21h Yy9hY2Nlc3NpYmlsaXR5L2NyYXNoLWluLWVsZW1lbnQtZm9yLXRleHQtbWFya2VyLWV4cGVjdGVk LnR4dDogQWRkZWQuCisgICAgICAgICogcGxhdGZvcm0vbWFjL2FjY2Vzc2liaWxpdHkvY3Jhc2gt aW4tZWxlbWVudC1mb3ItdGV4dC1tYXJrZXIuaHRtbDogQWRkZWQuCisKIDIwMTAtMDktMDkgIE1p aGFpIFBhcnBhcml0YSAgPG1paGFpcEBjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgUmV2aWV3ZWQg YnkgVG9ueSBDaGFuZy4KSW5kZXg6IExheW91dFRlc3RzL3BsYXRmb3JtL21hYy9hY2Nlc3NpYmls aXR5L2NyYXNoLWluLWVsZW1lbnQtZm9yLXRleHQtbWFya2VyLWV4cGVjdGVkLnR4dAo9PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09Ci0tLSBMYXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMvYWNjZXNzaWJpbGl0eS9jcmFzaC1pbi1l bGVtZW50LWZvci10ZXh0LW1hcmtlci1leHBlY3RlZC50eHQJKHJldmlzaW9uIDApCisrKyBMYXlv dXRUZXN0cy9wbGF0Zm9ybS9tYWMvYWNjZXNzaWJpbGl0eS9jcmFzaC1pbi1lbGVtZW50LWZvci10 ZXh0LW1hcmtlci1leHBlY3RlZC50eHQJKHJldmlzaW9uIDApCkBAIC0wLDAgKzEsMTAgQEAKK2Fz ZGYgYXNkZgorVGhpcyBwcm90ZWN0cyBhZ2FpbnN0IGEgY3Jhc2ggd2hlbiBBWFVJRWxlbWVudEZv clRleHRNYXJrZXIgaXMgcXVlcmllZCB3aXRoIGFuIGVtcHR5IHRleHQgbWFya2VyLgorCitPbiBz dWNjZXNzLCB5b3Ugd2lsbCBzZWUgYSBzZXJpZXMgb2YgIlBBU1MiIG1lc3NhZ2VzLCBmb2xsb3dl ZCBieSAiVEVTVCBDT01QTEVURSIuCisKKworUEFTUyBzdWNjZXNzZnVsbHlQYXJzZWQgaXMgdHJ1 ZQorCitURVNUIENPTVBMRVRFCisKSW5kZXg6IExheW91dFRlc3RzL3BsYXRmb3JtL21hYy9hY2Nl c3NpYmlsaXR5L2NyYXNoLWluLWVsZW1lbnQtZm9yLXRleHQtbWFya2VyLmh0bWwKPT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PQotLS0gTGF5b3V0VGVzdHMvcGxhdGZvcm0vbWFjL2FjY2Vzc2liaWxpdHkvY3Jhc2gtaW4tZWxl bWVudC1mb3ItdGV4dC1tYXJrZXIuaHRtbAkocmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL3Bs YXRmb3JtL21hYy9hY2Nlc3NpYmlsaXR5L2NyYXNoLWluLWVsZW1lbnQtZm9yLXRleHQtbWFya2Vy Lmh0bWwJKHJldmlzaW9uIDApCkBAIC0wLDAgKzEsNDkgQEAKKzwhRE9DVFlQRSBIVE1MIFBVQkxJ QyAiLS8vSUVURi8vRFREIEhUTUwvL0VOIj4KKzxodG1sPgorPGhlYWQ+Cis8bGluayByZWw9InN0 eWxlc2hlZXQiIGhyZWY9Ii4uLy4uLy4uL2Zhc3QvanMvcmVzb3VyY2VzL2pzLXRlc3Qtc3R5bGUu Y3NzIj4KKzxzY3JpcHQ+Cit2YXIgc3VjY2Vzc2Z1bGx5UGFyc2VkID0gZmFsc2U7Cis8L3Njcmlw dD4KKzxzY3JpcHQgc3JjPSIuLi8uLi8uLi9mYXN0L2pzL3Jlc291cmNlcy9qcy10ZXN0LXByZS5q cyI+PC9zY3JpcHQ+Cis8L2hlYWQ+Cis8Ym9keSBpZD0iYm9keSI+CisKK2FzZGYKKworPGRpdiBp ZD0iY29udGVudCIgdGFiaW5kZXg9IjAiPnRlc3Q8L2Rpdj4KKworYXNkZgorCis8cCBpZD0iZGVz Y3JpcHRpb24iPjwvcD4KKzxkaXYgaWQ9ImNvbnNvbGUiPjwvZGl2PgorCis8c2NyaXB0PgorCisg ICAgZGVzY3JpcHRpb24oIlRoaXMgcHJvdGVjdHMgYWdhaW5zdCBhIGNyYXNoIHdoZW4gQVhVSUVs ZW1lbnRGb3JUZXh0TWFya2VyIGlzIHF1ZXJpZWQgd2l0aCBhbiBlbXB0eSB0ZXh0IG1hcmtlci4i KTsKKworICAgIGlmICh3aW5kb3cuYWNjZXNzaWJpbGl0eUNvbnRyb2xsZXIpIHsKKworICAgICAg ICAgIGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJjb250ZW50IikuZm9jdXMoKTsKKyAgICAgICAg ICB2YXIgY29udGVudCA9IGFjY2Vzc2liaWxpdHlDb250cm9sbGVyLmZvY3VzZWRFbGVtZW50Owor CisgICAgICAgICAgLy8gZ2V0IGEgbWFya2VyIHRoYXQgaXMgaW52YWxpZCAodGhlIG5leHQgb25l IGFmdGVyIHRoZSBlbmQgb2YgdGhlIGJvZHkpLgorICAgICAgICAgIHZhciB0ZXh0TWFya2VyUmFu Z2UgPSBjb250ZW50LnRleHRNYXJrZXJSYW5nZUZvckVsZW1lbnQoY29udGVudCk7CisgICAgICAg ICAgdmFyIGVuZE1hcmtlciA9IGNvbnRlbnQuZW5kVGV4dE1hcmtlckZvclRleHRNYXJrZXJSYW5n ZSh0ZXh0TWFya2VyUmFuZ2UpOworCisgICAgICAgICAgLy8gUmVtb3ZlICJjb250ZW50IiBzbyB0 aGF0IHRoZSB0ZXh0IG1hcmtlciBiZWNvbWVzIGludmFsaWQuCisgICAgICAgICAgZG9jdW1lbnQu Z2V0RWxlbWVudEJ5SWQoImJvZHkiKS5yZW1vdmVDaGlsZChkb2N1bWVudC5nZXRFbGVtZW50QnlJ ZCgiY29udGVudCIpKTsKKworICAgICAgICAgIGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJib2R5 IikuZm9jdXMoKTsKKyAgICAgICAgICB2YXIgYm9keSA9IGFjY2Vzc2liaWxpdHlDb250cm9sbGVy LmZvY3VzZWRFbGVtZW50OworCisgICAgICAgICAgLy8gQXNrIGZvciB0aGUgdWkgZWxlbWVudCBm b3IgdGhhdCBtYXJrZXIgKHNob3VsZCBub3QgY3Jhc2gpLgorICAgICAgICAgIGJvZHkuYWNjZXNz aWJpbGl0eUVsZW1lbnRGb3JUZXh0TWFya2VyKGVuZE1hcmtlcik7CisgICAgfQorCisgICAgc3Vj Y2Vzc2Z1bGx5UGFyc2VkID0gdHJ1ZTsKKzwvc2NyaXB0PgorCis8c2NyaXB0IHNyYz0iLi4vLi4v Li4vZmFzdC9qcy9yZXNvdXJjZXMvanMtdGVzdC1wb3N0LmpzIj48L3NjcmlwdD4KKzwvYm9keT4K KzwvaHRtbD4K