Bug 88492 - [Shadow][Editing] Crashes document.execCommand('InsertLineBreak') in Shadow DOM
Summary: [Shadow][Editing] Crashes document.execCommand('InsertLineBreak') in Shadow DOM
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 82697
  Show dependency tree / graph
 
Reported: 2012-06-06 21:13 PDT by Shinya Kawanaka
Modified: 2012-06-22 10:04 PDT (History)
1 user (show)

See Also:

Attachments
Repro (1.25 KB, text/html)
2012-06-13 22:44 PDT, Shinya Kawanaka 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Shinya Kawanaka 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
Comment 1 Hayato Ito 2012-06-10 22:14:09 PDT 
I can not reproduce it. :(
Comment 2 Shinya Kawanaka 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.
Comment 3 Shinya Kawanaka 2012-06-11 22:46:13 PDT 
Sorry, I could reproduce it on ToT. Reopened.
Comment 4 Shinya Kawanaka 2012-06-11 22:47:02 PDT 
I could reproduce this in nestedShadowDOM.
Comment 5 Shinya Kawanaka 2012-06-13 22:17:25 PDT 
Actually, deleting them causes a crash. Pressing 'delete' key will also causes a crash.
Comment 6 Shinya Kawanaka 2012-06-13 22:42:59 PDT 
The root cause seems the same as Bug 88484.
Comment 7 Shinya Kawanaka 2012-06-13 22:44:33 PDT 
Created attachment 147489 [details]
Repro

Please put it in LayoutTest/editing/shadow
Comment 8 Shinya Kawanaka 2012-06-18 15:17:00 PDT 
Patch for Bug 88502 will also fix this issue, maybe.
Comment 9 Shinya Kawanaka 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!