Bug 89082 - CompositeEditCommand::insertNodeAt triggers an assertion
Summary: CompositeEditCommand::insertNodeAt triggers an assertion
Status: REOPENED
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:
 
Reported: 2012-06-14 02:21 PDT by Shinya Kawanaka
Modified: 2012-06-26 13:40 PDT (History)
4 users (show)

See Also:

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 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-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.
Comment 1 Shinya Kawanaka 2012-06-20 17:45:38 PDT 
According to the spec, surprisingly this is correct behavior...
So let's make this INVALID.
Comment 2 Shinya Kawanaka 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.
Comment 3 Shinya Kawanaka 2012-06-21 15:18:56 PDT 
Try JustifyFull and JustifyNone. Sometimes only one pressing the button won't cause a crash.
Comment 4 Shinya Kawanaka 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.
Comment 5 Shinya Kawanaka 2012-06-22 10:42:05 PDT 
Created attachment 149054 [details]
Repro
Comment 6 Shinya Kawanaka 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.
Comment 7 Shinya Kawanaka 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.