Ticket #2363: 2363.patch

_whatsnew.html

@@ -82 +82 @@
                         in the Paste dialog.</li>
                 <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/1907">#1907</a>] Fixed sporadic
                         "FCKeditorAPI is not defined" errors in Firefox 3.</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>
                 <a href="_whatsnew_history.html">See previous versions history</a></p>

editor/_source/internals/fckdocumentprocessor.js

@@ -142 +142 @@
                 el.parentNode.replaceChild( replaceElement, el ) ;
         }
 
+        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(),
                        {
                                 ProcessDocument : function( doc )
                                 {
                                         // 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 ) ;
                                 },
 
                                 RefreshView : function( placeHolder, original )