Bug 88492

Summary: [Shadow][Editing] Crashes document.execCommand('InsertLineBreak') in Shadow DOM
Product: WebKit Reporter: Shinya Kawanaka <shinyak>
Component: HTML EditingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED INVALID    
Severity: Normal CC: hayato
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 82697    
Attachments:
Description Flags
Repro none

Shinya Kawanaka
Reported 2012-06-06 21:13:02 PDT
HTML <div id="host" contenteditable></div> Shadow <div>BEFORE</div><content></content><div>AFTER</div> Select from BEFORE to AFTER, then document.execCommand('InsertLineBreak'); ASSERTION FAILED: isStartOfParagraph(startOfParagraphToMove) CompositeEditCommand.cpp(1120) : moveParagraph
Attachments
Repro (1.25 KB, text/html)
2012-06-13 22:44 PDT, Shinya Kawanaka 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Hayato Ito
Comment 1 2012-06-10 22:14:09 PDT
I can not reproduce it. :(
Shinya Kawanaka
Comment 2 2012-06-11 22:41:26 PDT
Hmm... I cannot reproduce on ToT. I could reproduce it in a bit older revision. So this might be fixed by other patches. make INVALID for now. Please feel free to reopen.
Shinya Kawanaka
Comment 3 2012-06-11 22:46:13 PDT
Sorry, I could reproduce it on ToT. Reopened.
Shinya Kawanaka
Comment 4 2012-06-11 22:47:02 PDT
I could reproduce this in nestedShadowDOM.
Shinya Kawanaka
Comment 5 2012-06-13 22:17:25 PDT
Actually, deleting them causes a crash. Pressing 'delete' key will also causes a crash.
Shinya Kawanaka
Comment 6 2012-06-13 22:42:59 PDT
The root cause seems the same as Bug 88484.
Shinya Kawanaka
Comment 7 2012-06-13 22:44:33 PDT
Created attachment 147489 [details] Repro Please put it in LayoutTest/editing/shadow
Shinya Kawanaka
Comment 8 2012-06-18 15:17:00 PDT
Patch for Bug 88502 will also fix this issue, maybe.
Shinya Kawanaka
Comment 9 2012-06-22 10:04:11 PDT
Yay! I confirmed that this bug disappeared by Patch for Bug 88502. Let's close this as INVALID!
Note You need to log in before you can comment on or make changes to this bug.