Bug 89082

Summary: CompositeEditCommand::insertNodeAt triggers an assertion
Product: WebKit Reporter: Shinya Kawanaka <shinyak>
Component: HTML EditingAssignee: Nobody <webkit-unassigned>
Status: REOPENED    
Severity: Normal CC: darin, enrica, rniwa, shinyak
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Repro
none
Repro none

Shinya Kawanaka
Reported 2012-06-14 02:21:57 PDT
Created attachment 147527 [details] Repro ASSERTION FAILED: isEditablePosition(editingPosition) Select from "AFTER HOST" to "hoge 1", and press JustifyNone. This will trigger ASSERTION FAILURE.
Attachments
Repro (3.76 KB, text/html)
2012-06-14 02:21 PDT, Shinya Kawanaka 		no flags
Details
Repro (907 bytes, text/html)
2012-06-22 10:42 PDT, Shinya Kawanaka 		no flags
Details
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Shinya Kawanaka
Comment 1 2012-06-20 17:45:38 PDT
According to the spec, surprisingly this is correct behavior... So let's make this INVALID.
Shinya Kawanaka
Comment 2 2012-06-21 08:11:21 PDT
(In reply to comment #1) > According to the spec, surprisingly this is correct behavior... > So let's make this INVALID. Sorry I commented on the wrong place. Reopened.
Shinya Kawanaka
Comment 3 2012-06-21 15:18:56 PDT
Try JustifyFull and JustifyNone. Sometimes only one pressing the button won't cause a crash.
Shinya Kawanaka
Comment 4 2012-06-22 10:36:21 PDT
I found that this actually does not related to Shadow DOM. This happens event if no Shadow DOM is attached.
Shinya Kawanaka
Comment 5 2012-06-22 10:42:05 PDT
Created attachment 149054 [details] Repro
Shinya Kawanaka
Comment 6 2012-06-22 10:43:09 PDT
(In reply to comment #4) > I found that this actually does not related to Shadow DOM. > This happens event if no Shadow DOM is attached. Oops too bad English... I found that this bug actually is not related to Shadow DOM. This happens even if Shadow DOM is not used.
Shinya Kawanaka
Comment 7 2012-06-22 10:51:35 PDT
Since this is not related to Shadow DOM, I remove dependency from Bug 82697. But we should fix this anyway.
Note You need to log in before you can comment on or make changes to this bug.