JS error in Find & Replace dialog

[Tobiasz Cudnik]

TC

1. Use following content:

   <p>
   	^foo<br />
   </p>
   <p>
   	foo<br />
   </p>
   

2. Hit find button 2 or more times
3. Result:

   find.js (line 216)
   endNode is null
   [Break on this error] while ( endNode.type != CKEDITOR.NODE_TEXT )
   

4. Stack trace

   updateFromDomRange()
   removeHighlight()
   find()
   onClick()
   listenerFirer()
   (?)()
   (?)()
   listenerFirer()
   (?)()
   (?)()
   

Reproduced using r3500 in FF3, IE8.

[Garry Yao]

I didn't got the error besides the symptom described in #3450, or is it random?

[Tobiasz Cudnik]

It happens all the time in my case but whole thing is very weird. Specially L219: endNode = boundaryNodes.endNode; ...after which endNode becomes null, while boundaryNodes.endNode is not.

I can't reproduce it in Opera and Chrome. Although i can reproduce it on firebug-less FF, so that's not a debugger fault. It also happens in IE8.

Best way to reproduce it is just hitting Find, Find, Find.

Possible workaround would be securing loop conditions and assignments at L220-L222.

Can someone else please try to reproduce it ?

[Tobiasz Cudnik]

Sorry for no line break, pasting mentioned line again:

endNode = boundaryNodes.endNode;

[Garry Yao]

Running minor fixings.

[Garry Yao]

Fixed with [3517].