Changes between Initial Version and Version 1 of Ticket #6340


Ignore:
Timestamp:
Sep 23, 2010, 3:29:08 PM (14 years ago)
Author:
Sa'ar Zac Elias
Comment:

Works fine with Safari.

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #6340

    • Property Keywords Chrome added
    • Property Status changed from new to confirmed
  • Ticket #6340 – Description

    initial v1  
    11Steps to reproduce:[[BR]]
    221. From Google Chrome open the CKEditor demo page (http://ckeditor.com/demo) and go to the Source view.[[BR]]
    3 2. Paste the following malformed HTML:[[BR]]
    4 <span id="sample" overflow="hidden" ;"="" style="font-size:8pt; font-weight:normal; font-style:normal; color:#808080; background:transparent">Text</span>[[BR]]
    5 3. Switch to Standard view, so the HTML gets parsed.[[BR]]
     32. Paste the following malformed HTML:
     4{{{
     5<span id="sample" overflow="hidden" ;"="" style="font-size:8pt; font-weight:normal; font-style:normal; color:#808080; background:transparent">Text</span>
     6}}}
     73. Switch to WYSIWYG mode, so the HTML gets parsed.[[BR]]
    68[[BR]]
    79After following these steps, the browser tab should hang.[[BR]]
     
    1214[[BR]]
    1315I debugged the problem and found out that the problem is in this regular expression that is used to parse the HTML:[[BR]]
    14 [[BR]]
    15 htmlPartsRegex:new RegExp("<(?:(?:\\/([^>]+)>)|(?:!--([\\S|\\s]*?)-->)|(?:([^\\s>]+)\\s*((?:(?:[^\"'>]+)|(?:\"[^\"]*\")|(?:'[^']*'))*)\\/?>))",'g')[[BR]]
    16 [[BR]]
     16{{{
     17htmlPartsRegex:new RegExp("<(?:(?:\\/([^>]+)>)|(?:!--([\\S|\\s]*?)-->)|(?:([^\\s>]+)\\s*((?:(?:[^\"'>]+)|(?:\"[^\"]*\")|(?:'[^']*'))*)\\/?>))",'g')
     18}}}
    1719Thanks in advance.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy