Bug 6586 – REGRESSION: visible_text.cpp assertion failure in Mail.app

• Status: RESOLVED
• Resolution: FIXED
• Component: HTML Editing

[Duncan Wilcox]

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()'

[Mark Rowe (bdash)]

This triggers for me when doing Command-Down then Command-Up.  Alt-Down, Alt-Up has no effect.

[Mark Rowe (bdash)]

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.

[Mark Rowe (bdash)]

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 ?? ()

[Joost de Valk (AlthA)]

Adding Regression keyword.

[Joost de Valk (AlthA)]

Upping to P1 because this is a regression.

[Alice Liu]

<rdar://problem/4424035>

[Justin Garcia]

Fixed.

[David Kilzer (:ddkilzer)]

Created attachment 6218 [details]
SVN revision r12538