#13919 closed Bug (fixed)
[Edge] CKEDITOR.dom.node#isReadOnly crashes window when in the context of an input
| Reported by: | Marek Lewandowski | Owned by: | Marek Lewandowski |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 4.5.5 |
| Component: | General | Version: | 4.5.5 |
| Keywords: | Cc: |
Description (last modified by )
Recent update for Edge brought a regression that if you check .isContentEditable property on an input element it will crash.
It happens only in built CKE distribution as it includes bidi plugin, which injects .isReadOnly() check to setAttribute method.
This is very similar to #13609.
Change History (9)
comment:1 Changed 10 years ago by
| Description: | modified (diff) |
|---|
comment:2 Changed 10 years ago by
| Owner: | set to Marek Lewandowski |
|---|---|
| Status: | new → assigned |
comment:3 Changed 10 years ago by
comment:5 Changed 10 years ago by
| Status: | review → review_failed |
|---|
The fix looks ok, but it needs tests (for textarea and input) around https://github.com/cksource/ckeditor-dev/blob/400e2a5d126bc6bbb16ee0d2fa3f682694b5c345/tests/core/dom/node.js#L189-L255 because
- The issue should be visible in dev, not only when BiDi is loaded and overrides
CKEDITOR.dom.element.prototypein build. - #13609 implemented no tests :(
- We should never meet this kind of issue again ;)
comment:6 Changed 10 years ago by
| Status: | review_failed → review |
|---|
Added test, I've used whole DTD so that we catch next regressions in the future.
Pushed to t/13919 at dev.
comment:7 Changed 10 years ago by
| Status: | review → review_passed |
|---|
comment:8 Changed 10 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Merged git:400376f.
comment:9 Changed 10 years ago by
| Summary: | [Edge] Failing tests → [Edge] CKEDITOR.dom.node#isReadOnly crashes window when in the context of an input |
|---|
Note: See
TracTickets for help on using
tickets.
Related MS Connect issue: https://connect.microsoft.com/IE/feedback/details/1577449/edge-browser-crashes-when-using-ckeditor-toggling-from-wysiwyg-to-source-mode