Ticket #7639 (closed Bug: fixed)

Opened 4 years ago

Last modified 3 years ago

IE9: Browser crashes when inserting image links with different targets

Reported by: jamescun Owned by: Saare
Priority: Normal Milestone: CKEditor 3.6.1
Component: General Version: 3.3.2
Keywords: IE9 IBM Cc: damo, satya, tmonahan

Description

Steps to reproduce the defect:

  1. Open the Ajax sample in IE9.
  1. Open the image dialog.
  1. On the Image Info tab, Paste the following image into the URL field. http://a.cksource.com/e/1/img/logo-ckeditor-h100.png
  1. Click on the Link tab & paste the same URL into the URL field.
  1. Set the target to New Window (_blank) & click OK.
  1. See that the image is inserted into the editor.
  1. Right click on the image & open the image properties dialog.
  1. Click on the Link tab & change the target to Topmost Window (_top) & click OK.
  1. Click on Remove Editor.

Result: A message saying Internet Explorer has stopped working is displayed and the page closes

Attachments

7639.patch (731 bytes) - added by Saare 4 years ago.
7639_2.patch (639 bytes) - added by garry.yao 3 years ago.
7639_3.patch (602 bytes) - added by Saare 3 years ago.

Change History

comment:1 Changed 4 years ago by j.swiderski

  • Status changed from new to confirmed
  • Version changed from 3.5.4 (SVN - trunk) to 3.3.2

comment:2 Changed 4 years ago by garry.yao

It looks to be related to the image selection state when remove editor (doesn't happen when first remove image selection from editor).

comment:3 Changed 4 years ago by fredck

  • Milestone set to CKEditor 3.6.1

Changed 4 years ago by Saare

comment:4 Changed 4 years ago by Saare

  • Owner set to Saare
  • Status changed from confirmed to review

Seems like cleaning the selection is enough to resolve the bug.

Changed 3 years ago by garry.yao

comment:5 Changed 3 years ago by garry.yao

  • Status changed from review to review_failed

Let's keep the hack right now inside of the selection plugin.

comment:6 Changed 3 years ago by garry.yao

  • Status changed from review_failed to review
  • Owner changed from Saare to garry.yao

comment:7 Changed 3 years ago by Saare

  • Status changed from review to review_failed

The new patch doesn't seem to help to resolve the bug.

Changed 3 years ago by Saare

comment:8 Changed 3 years ago by Saare

  • Status changed from review_failed to review
  • Owner changed from garry.yao to Saare

comment:9 Changed 3 years ago by garry.yao

  • Status changed from review to review_passed

Perhaps I get some luck with my previous patch but ok...

comment:10 Changed 3 years ago by Saare

  • Status changed from review_passed to closed
  • Resolution set to fixed

Fixed with [6993].

Note: See TracTickets for help on using tickets.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy