Bug 32903

Summary: REGRESSION (r52494): Assertion failure in Frame::caretBlinkTimerFired() (selection()->isCaret())
Product: WebKit Reporter: mitz
Component: HTML EditingAssignee: Darin Adler <darin>
Status: RESOLVED FIXED    
Severity: Normal CC: darin, levin
Priority: P1 Keywords: InRadar, Regression
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   

Description mitz 2009-12-23 10:14:19 PST
ASSERTION FAILED: selection()->isCaret()
(WebCore/page/Frame.cpp:697 void WebCore::Frame::caretBlinkTimerFired(WebCore::Timer<WebCore::Frame>*))

<http://trac.webkit.org/changeset/52494> delayed stopping the caret blink timer, and it looks like in this case it fires after the selection becomes a non-caret but before selectionLayoutChanged() is called to stop the timer.

<rdar://problem/7496082>
Comment 1 David Levin 2009-12-23 17:16:28 PST
Rolled out r52494 with r52535. I'll re-open the original bug.
Comment 2 Darin Adler 2009-12-26 17:30:49 PST
Do you have steps to reproduce? I fixed this, but want to test.
Comment 3 mitz 2009-12-26 17:31:45 PST
(In reply to comment #2)
> Do you have steps to reproduce? I fixed this, but want to test.

I do not.
Comment 4 David Levin 2009-12-31 10:51:49 PST
(In reply to comment #2)
> Do you have steps to reproduce? I fixed this, but want to test.

I have no repro steps either. I encountered while running layout test in chromium on osx with the patch.