#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.prototype
in 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 |
---|
Related MS Connect issue: https://connect.microsoft.com/IE/feedback/details/1577449/edge-browser-crashes-when-using-ckeditor-toggling-from-wysiwyg-to-source-mode