6586 2006-01-16 09:10:31 -0800 REGRESSION: visible_text.cpp assertion failure in Mail.app 2006-02-03 06:54:38 -0800 1 1 1 Unclassified WebKit HTML Editing 420+ Mac OS X 10.4 RESOLVED FIXED InRadar, Regression P1 Normal --- 1 duncan justin.garcia alice.barraclough ap mrowe oldest_to_newest 28585 0 duncan 2006-01-16 09:10:31 -0800 Run Mail.app with TOT: run-webkit-app /Applications/Mail.app Reply to any message. Alt-downarrow, alt-uparrow. Mail.app fails with this assertion: .../WebKit/WebCore/khtml/editing/visible_text.cpp:748: failed assertion `!atEnd()' 28611 1 mrowe 2006-01-16 12:51:57 -0800 This triggers for me when doing Command-Down then Command-Up. Alt-Down, Alt-Up has no effect. 28612 2 mrowe 2006-01-16 12:55:23 -0800 This appears to happen any time the insertion point is at the end of the message and is then moved from this position. Clicking at the end of the message and then using the Up arrow has the same effect. 28615 3 mrowe 2006-01-16 13:01:32 -0800 Backtrace when the assertion failure occurs: #0 0x9004716c in kill () #1 0x90128b98 in abort () #2 0x946e7144 in __eprintf () #3 0x01937650 in WebCore::CharacterIterator::advance (this=0xbfffe26c, count=28) at .../WebCore/ khtml/editing/visible_text.cpp:748 #4 0x01a3b58c in MacFrame::markMisspellings (this=0xd991400, selection=@0xbfffe4a8) at .../ WebCore/bridge/mac/MacFrame.mm:3306 #5 0x01a3bb18 in MacFrame::respondToChangedSelection (this=0xd991400, oldSelection=@0xbfffe59c, closeTyping=true) at .../WebCore/bridge/mac/MacFrame.mm:3348 #6 0x01a1f8ac in Frame::setSelection (this=0xd991400, s=@0xbfffe63c, closeTyping=true, keepTypingStyle=false) at .../WebCore/page/Frame.cpp:1393 #7 0x01a6da40 in -[WebCoreFrameBridge alterCurrentSelection:direction:granularity:] (self=0x12724f00, _cmd=0x90a05c18, alteration=WebSelectByMoving, direction=WebBridgeSelectBackward, granularity=WebBridgeSelectToDocumentBoundary) at .../ WebCore/bridge/mac/WebCoreFrameBridge.mm:1906 #8 0x0038255c in -[WebHTMLView _alterCurrentSelection:direction:granularity:] (self=0x1273ac20, _cmd=0x909d5834, alteration=WebSelectByMoving, direction=WebBridgeSelectBackward, granularity=WebBridgeSelectToDocumentBoundary) at .../WebKit/WebView.subproj/WebHTMLView.m: 3360 #9 0x00382b44 in -[WebHTMLView moveToBeginningOfDocument:] (self=0x1273ac20, _cmd=0x4095ac, sender=0x0) at .../WebKit/WebView.subproj/WebHTMLView.m:3429 #10 0x0038c6e4 in -[WebHTMLView(WebNSTextInputSupport) doCommandBySelector:] (self=0x1273ac20, _cmd=0x90a102e0, aSelector=0x4095ac) at .../WebKit/WebView.subproj/ WebHTMLView.m:5166 #11 0x9375d3b0 in -[NSKeyBindingManager(NSKeyBindingManager_MultiClients) interpretEventAsCommand:forClient:] () #12 0x9375baac in -[NSTSMInputContext interpretKeyEvents:] () #13 0x9375ae8c in -[NSView interpretKeyEvents:] () #14 0x00381b3c in -[WebHTMLView _interceptEditingKeyEvent:] (self=0x1273ac20, _cmd=0x909e7c4c, event=0x12718f20) at .../WebKit/WebView.subproj/WebHTMLView.m:3253 #15 0x00381d18 in -[WebHTMLView keyDown:] (self=0x1273ac20, _cmd=0x90a1f8cc, event=0x12718f20) at .../WebKit/WebView.subproj/WebHTMLView.m:3274 #16 0x936d65c0 in -[NSWindow sendEvent:] () #17 0x00152260 in ?? () #18 0x9367eef4 in -[NSApplication sendEvent:] () #19 0x000e6290 in ?? () #20 0x93676330 in -[NSApplication run] () #21 0x93766e68 in NSApplicationMain () #22 0x00002888 in ?? () #23 0x000a4da0 in ?? () 29361 4 joost 2006-01-22 04:37:32 -0800 Adding Regression keyword. 29380 5 joost 2006-01-22 04:43:23 -0800 Upping to P1 because this is a regression. 30220 6 alice.barraclough 2006-01-26 17:19:38 -0800 <rdar://problem/4424035> 31116 7 justin.garcia 2006-02-02 17:22:21 -0800 Fixed. 31160 8 6218 ddkilzer 2006-02-03 06:54:38 -0800 Created attachment 6218 SVN revision r12538 6218 2006-02-03 06:54:38 -0800 2006-02-03 06:54:38 -0800 SVN revision r12538 bug-6586.diff text/plain 2494 ddkilzer SW5kZXg6IFdlYkNvcmUva2h0bWwvZWRpdGluZy92aXNpYmxlX3VuaXRzLmNwcAo9PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 Ci0tLSBXZWJDb3JlL2todG1sL2VkaXRpbmcvdmlzaWJsZV91bml0cy5jcHAJKHJldmlzaW9uIDEy NTM3KQorKysgV2ViQ29yZS9raHRtbC9lZGl0aW5nL3Zpc2libGVfdW5pdHMuY3BwCShyZXZpc2lv biAxMjUzOCkKQEAgLTY4LDExICs2OCwxOCBAQAogICAgICAgICBib3VuZGFyeSA9IGJvdW5kYXJ5 LT5wYXJlbnROb2RlKCk7CiAgICAgfQogCi0gICAgUmVmUHRyPFJhbmdlSW1wbD4gc2VhcmNoUmFu Z2UoZC0+Y3JlYXRlUmFuZ2UoKSk7CisgICAgUG9zaXRpb24gc3RhcnQgPSByYW5nZUNvbXBsaWFu dEVxdWl2YWxlbnQoUG9zaXRpb24oYm91bmRhcnksIDApKTsKKyAgICBQb3NpdGlvbiBlbmQgPSBy YW5nZUNvbXBsaWFudEVxdWl2YWxlbnQocG9zKTsKKyAgICBSZWZQdHI8UmFuZ2VJbXBsPiBzZWFy Y2hSYW5nZSA9IG5ldyBSYW5nZUltcGwoZCk7CisgICAgCiAgICAgaW50IGV4Y2VwdGlvbiA9IDA7 Ci0gICAgc2VhcmNoUmFuZ2UtPnNldFN0YXJ0QmVmb3JlKGJvdW5kYXJ5LCBleGNlcHRpb24pOwot ICAgIFBvc2l0aW9uIGVuZChyYW5nZUNvbXBsaWFudEVxdWl2YWxlbnQocG9zKSk7CisgICAgc2Vh cmNoUmFuZ2UtPnNldFN0YXJ0KHN0YXJ0Lm5vZGUoKSwgc3RhcnQub2Zmc2V0KCksIGV4Y2VwdGlv bik7CiAgICAgc2VhcmNoUmFuZ2UtPnNldEVuZChlbmQubm9kZSgpLCBlbmQub2Zmc2V0KCksIGV4 Y2VwdGlvbik7CisgICAgCisgICAgQVNTRVJUKCFleGNlcHRpb24pOworICAgIGlmIChleGNlcHRp b24pCisgICAgICAgIHJldHVybiBWaXNpYmxlUG9zaXRpb24oKTsKKyAgICAgICAgCiAgICAgU2lt cGxpZmllZEJhY2t3YXJkc1RleHRJdGVyYXRvciBpdChzZWFyY2hSYW5nZS5nZXQoKSk7CiAgICAg UVN0cmluZyBzdHJpbmc7CiAgICAgdW5zaWduZWQgbmV4dCA9IDA7CkluZGV4OiBXZWJDb3JlL2to dG1sL2VkaXRpbmcvdmlzaWJsZV90ZXh0LmgKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViQ29yZS9raHRtbC9l ZGl0aW5nL3Zpc2libGVfdGV4dC5oCShyZXZpc2lvbiAxMjUzNykKKysrIFdlYkNvcmUva2h0bWwv ZWRpdGluZy92aXNpYmxlX3RleHQuaAkocmV2aXNpb24gMTI1MzgpCkBAIC0yMjUsNyArMjI1LDcg QEAKICAgICAvLyBEaWQgd2UgaGF2ZSB0byBsb29rIGFoZWFkIGluIHRoZSB0ZXh0SXRlcmF0b3Ig dG8gY29uZmlybSB0aGUgY3VycmVudCBjaHVuaz8KICAgICBib29sIG1fZGlkTG9va0FoZWFkOwog Ci0gICAgUGFzc1JlZlB0cjxET006OlJhbmdlSW1wbD4gbV9yYW5nZTsKKyAgICBSZWZQdHI8RE9N OjpSYW5nZUltcGw+IG1fcmFuZ2U7CiAKICAgICBUZXh0SXRlcmF0b3IgbV90ZXh0SXRlcmF0b3I7 CiB9OwpJbmRleDogV2ViQ29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViQ29yZS9DaGFu Z2VMb2cJKHJldmlzaW9uIDEyNTM3KQorKysgV2ViQ29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDEy NTM4KQpAQCAtMSwzICsxLDIzIEBACisyMDA2LTAyLTAyICBKdXN0aW4gR2FyY2lhICA8anVzdGlu LmdhcmNpYUBhcHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgZGFyaW4sIGhhcnJpc29u CisgICAgICAgIAorICAgICAgICA8aHR0cDovL2J1Z3ppbGxhLm9wZW5kYXJ3aW4ub3JnL3Nob3df YnVnLmNnaT9pZD02NTg2PgorICAgICAgICBSRUdSRVNTSU9OOiB2aXNpYmxlX3RleHQuY3BwIGFz c2VydGlvbiBmYWlsdXJlIGluIE1haWwuYXBwCisKKyAgICAgICAgQ2FsbGluZyByYW5nZSgpIG9u IGEgV29yZEl0ZXJhdG9yIGRlc3Ryb3llZCBpdHMgUmFuZ2VJbXBsLCB1c2UKKyAgICAgICAgUmVm UHRyIGluc3RlYWQgb2YgUGFzc1JlZlB0ci4KKyAgICAgICAgVHVybmVkIG9uIGNvbnRpbnVvdXMg c3BlbGwgY2hlY2tpbmcgdG8gdGVzdCB0aGUgZml4LCBmb3VuZAorICAgICAgICBhIGNyYXNoZXIg Y2F1c2VkIGJ5IGludmFsaWQgcG9zaXRpb25zIGNyZWF0ZWQgaW4KKyAgICAgICAgcHJldmlvdXNC b3VuZGFyeS4KKworICAgICAgICBBZGRlZCBsYXlvdXQgdGVzdDoKKyAgICAgICAgKiBlZGl0aW5n L3NlbGVjdGlvbi9lbmQtb2YtZG9jdW1lbnQuaHRtbAorCisgICAgICAgICoga2h0bWwvZWRpdGlu Zy92aXNpYmxlX3RleHQuaDoKKyAgICAgICAgKiBraHRtbC9lZGl0aW5nL3Zpc2libGVfdW5pdHMu Y3BwOgorICAgICAgICAoa2h0bWw6OnByZXZpb3VzQm91bmRhcnkpOgorCiAyMDA2LTAyLTAyICBE YXZpZCBIeWF0dCAgPGh5YXR0QGFwcGxlLmNvbT4KIAogICAgICAgICBWZWN0b3JpemUgSW1hZ2Uu ICBPcHRpbWl6ZSBzbyB0aGF0IHdlIG5vIGxvbmdlciB0aHJvdyBhd2F5Cg==