Opened 13 years ago
Closed 13 years ago
#9384 closed Bug (fixed)
[Webkit] Filler char should be treated as bogus node
| Reported by: | Garry Yao | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.6.5 |
| Component: | General | Version: | 3.6.5 |
| Keywords: | Cc: |
Description
The filler char we created for selecting an empty paragraph in webkit, is corrupting range::checkStartOfBlock/range::checkBoundaryOfElement, it can be verified by the ticket tcs.
Attachments (2)
Change History (9)
Changed 13 years ago by
| Attachment: | 9384.patch added |
|---|
comment:1 Changed 13 years ago by
| Owner: | set to Garry Yao |
|---|---|
| Status: | new → review |
comment:2 Changed 13 years ago by
I've created a standalone ticket TC to justify the bug. http://v3.ckeditor.t/tt/9384/1.html
comment:3 Changed 13 years ago by
| Status: | review → assigned |
|---|
Changed 13 years ago by
| Attachment: | 9384_2.patch added |
|---|
comment:4 Changed 13 years ago by
| Status: | assigned → review |
|---|
It looks like a more appropriate fix, is instead to treat the filler like a whitespace, so it has a wider impact range.
comment:5 Changed 13 years ago by
| Status: | review → review_failed |
|---|
The new patch sounded very nice to me as well, but these are the test results I'm having now, with t/9384@tests (Chrome):
CKTester Fort (8 failed / 727 passed) - 01m 26s 225ms - 100%
comment:7 Changed 13 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_failed → closed |
Fixed with [7625].
Note: See
TracTickets for help on using
tickets.
Opened t/9384 on test to justify the ticket issue.