Changeset 2247

Timestamp:

2008-07-21 14:28:17 (20 months ago)

Author:

fredck

Message:

Fixed #2363 : There were some sporadic "Permission Denied" errors with IE on some situations.

Location:

FCKeditor/trunk

Files:

• editor/_source/internals/fckdocumentprocessor.js (modified) (2 diffs)
• _whatsnew.html (modified) (1 diff)

FCKeditor/trunk/editor/_source/internals/fckdocumentprocessor.js

@@ -143 +143 @@
         }
 
+        var processElementsByName = function( elementName, doc )
+        {
+                var aObjects = doc.getElementsByTagName( elementName );
+                for ( var i = aObjects.length - 1 ; i >= 0 ; i-- )
+                        processElement( aObjects[i] ) ;
+        }
+
+        var processObjectAndEmbed = function( doc )
+        {
+                processElementsByName( 'object', doc );
+                processElementsByName( 'embed', doc );
+        }
+
         return FCKTools.Merge( FCKDocumentProcessor.AppendNew(),
                        {
@@ -149 +162 @@
                                         // Firefox 3 would sometimes throw an unknown exception while accessing EMBEDs and OBJECTs
                                         // without the setTimeout().
-                                        FCKTools.RunFunction( function()
-                                                {
-                                                        // Process OBJECTs first, since EMBEDs can sometimes go inside OBJECTS (e.g. Flash).
-                                                        var aObjects = doc.getElementsByTagName( 'object' );
-                                                        for ( var i = aObjects.length - 1 ; i >= 0 ; i-- )
-                                                                processElement( aObjects[i] ) ;
-
-                                                        // Now process any EMBEDs left.
-                                                        var aEmbeds = doc.getElementsByTagName( 'embed' ) ;
-                                                        for ( var i = aEmbeds.length - 1 ; i >= 0 ; i-- )
-                                                                processElement( aEmbeds[i] ) ;
-                                                } ) ;
+                                        if ( FCKBrowserInfo.IsGecko )
+                                                FCKTools.RunFunction( processObjectAndEmbed, this, [ doc ] ) ;
+                                        else
+                                                processObjectAndEmbed( doc ) ;
                                 },
 

FCKeditor/trunk/_whatsnew.html

@@ -94 +94 @@
                         multiple contiguous paragraphs to Formatted will now be merged into a single 
                         &lt;PRE&gt; block.</li>
+                <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2363">#2363</a>] There
+                        were some sporadic "Permission Denied" errors with IE on some situations.</li>
         </ul>
         <p>