Opened 11 years ago
Closed 11 years ago
#11178 closed Bug (fixed)
[IE] Error thrown in htmlDataProcessor test
Reported by: | Piotrek Koszuliński | Owned by: | Marek Lewandowski |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 4.3.1 |
Component: | General | Version: | |
Keywords: | Cc: |
Description
Attachments (2)
Change History (10)
comment:1 Changed 11 years ago by
Status: | new → confirmed |
---|
comment:2 Changed 11 years ago by
Owner: | set to Piotr Jasiun |
---|---|
Status: | confirmed → assigned |
comment:3 Changed 11 years ago by
Owner: | changed from Piotr Jasiun to Marek Lewandowski |
---|
comment:5 Changed 11 years ago by
From what I see that doc#selectionchange
event during which selection is broken is fired after editable is destroy, so using editable#attachListener
helps. However, I'd be good to know exact moment when it happens - what code causes selectionchange.
comment:6 Changed 11 years ago by
Status: | assigned → review |
---|
I've applied in t/11178 fix suggested by PK. It prevents from exception, and still provides quickfix originally implemented by changed code. After some testing I was not able to precisely locate real source of this issue.
I've separated essential code which leads to exception in a TC but i dont think if it's needed since it can't be fit into an assertion (so CKTester can't display it as failure), since it happens in event listener. But still i will attach this TC to the ticket.
comment:7 Changed 11 years ago by
Status: | review → review_passed |
---|
I gave up investigating the origin of that bug since it's just a waste of time. The only conclusion is that debugger in IE is useless.
I attached another sample to reproduce the issue without tester.
comment:8 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Yea it was pretty time consuming. Fixed with git:73b32732d at dev.
So far i got following informations for this issue:
its caused by test avoid having inline event handlers executed TC.
The reason leading to an exception is following (is quite nice sequence):
enterMode = CKEDITOR.ENTER_BR
this.editor.focus()
editor.setData
Error is originated from wysiwygarea\plugin.js:168 which is invoked selectionchange
affected browsers: ie10, ie11 not affected: ie9