Ticket #1792: 1792.patch

_whatsnew.html

@@ -107 +107 @@
                 <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/1832">#1832</a>] Calling
                         FCK.InsertHtml() in non-IE browsers would now activate the document processor
                         as expected.</li>
+                <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/1792">#1792</a>] In IE,
+                        the browser was able to enter in an infinite loop when working with multiple editors
+                        in the same page. </li>
         </ul>
         <h3>
                 Version 2.6 Beta 1</h3>

editor/_source/classes/fckeditingarea.js

@@ -300 +300 @@
         {
                 if ( this.Mode == FCK_EDITMODE_WYSIWYG )
                 {
-                        // The following check is important to avoid IE entering in a focus loop. Ref:
-                        // http://sourceforge.net/tracker/index.php?func=detail&aid=1567060&group_id=75348&atid=543653
-                        if ( FCKBrowserInfo.IsIE && this.Document.hasFocus() )
-                                this._EnsureFocusIE() ;
-
-                        this.Window.focus() ;
-
-                        // In IE it can happen that the document is in theory focused but the active element is outside it
                         if ( FCKBrowserInfo.IsIE )
-                                this._EnsureFocusIE() ;
+                                this._FocusIE() ;
+                        else
+                                this.Window.focus() ;
                 }
                 else
                 {
@@ -323 +317 @@
         catch(e) {}
 }
 
-FCKEditingArea.prototype._EnsureFocusIE = function()
+FCKEditingArea.prototype._FocusIE = function()
 {
-        // In IE it can happen that the document is in theory focused but the active element is outside it
+        // In IE it can happen that the document is in theory focused but the
+        // active element is outside of it.
         this.Document.body.setActive() ;
 
+        this.Window.focus() ;
+
         // Kludge for #141... yet more code to workaround IE bugs
         var range = this.Document.selection.createRange() ;
 

editor/_source/internals/fck.js

@@ -1145 +1145 @@
 
 function FCKFocusManager_Win_OnFocus_Area()
 {
+        // Check if we are already focusing the editor (to avoid loops).
+        if ( FCKFocusManager._IsFocusing )
+                return ;
+
+        FCKFocusManager._IsFocusing = true ;
+
         FCK.Focus() ;
         FCKFocusManager_Win_OnFocus() ;
+
+        // The above FCK.Focus() call may trigger other focus related functions.
+        // So, to avoid a loop, we delay the focusing mark removal, so it get
+        // executed after all othre functions have been run.
+        FCKTools.RunFunction( function()
+                {
+                        delete FCKFocusManager._IsFocusing ;
+                } ) ;
 }
 
 function FCKFocusManager_Win_OnFocus()