#9731 closed defect (fixed)
1.3.2: Editor sometimes triggers a secure/insecure warning in IE6
Reported by: | Jared Jurkiewicz | Owned by: | Jared Jurkiewicz |
---|---|---|---|
Priority: | high | Milestone: | 1.4 |
Component: | Editor | Version: | 1.3.2 |
Keywords: | 1.3.3 | Cc: | |
Blocked By: | Blocking: |
Description
1.3.2: Editor sometimes triggers a secure/insecure warning in IE6
This problem occurs due to a fix we put in to handle corrupted charsets in IE, which occurred when we changed the editor over from being loaded from blank.html to dynamic JS string creation. The blank.html change was needed to fix an IE iframes/back button issue so going back to that isn't satisfactory.
The problem occurs (I believe), when the charset of the main page doesn't match the charset the iframe is initially created as and then loaded. When the iframe encounters the charset declaration, it triggers a reparse. If that reparse occurs while the frame is still in the middle of doing something, it then throws the secure/insecure warning.
This, I suspect, is very similar to another very well known bug in IE6 where if you remove a <DIV> that had a background image from the page while the image was loading it threw a secure/insecure warning. Neither of these situations have anything to do with secure/insecure, but however IE6 handles the case seems to trip their hander for such.
In any event, Bill Keese made changes to 1.4 that got rid of the need to set the charset meta tag and still obtained a correct encoding for the editor content. I backported that change and verified with a co-worker that the backport eliminated the warning he was getting. I'll be attaching the patch for it here. After some quick testing, I'll also commit the change.
Attachments (1)
Change History (7)
comment:1 Changed 11 years ago by
Owner: | set to Jared Jurkiewicz |
---|
Changed 11 years ago by
Attachment: | 1.3.2_Editor_secure_insecure.patch added |
---|
comment:2 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:3 Changed 11 years ago by
Milestone: | tbd → 1.3.3 |
---|
Backport of the changesets related to removing the need for charset decl on IE. Also added in the UT test for charset corruptions