Find locks for successive searchs

[Frederico Caldeira Knabben]

Steps to Reproduce

1. Load sample01.html.
2. In the Find dialog search for "e".
3. Continuously click in the Find button to find all occurrences of "e".

It will successfully find the first two occurrences, in "some" and "sample". After that, it blocks on the last found position and don't continue through the remaining occurrences inside "text", "are" and "FCKeditor".

[Martin Kou]

It seems the way we're handling the highlight by document mutation is disrupting the logic for scanning the document. It's not likely the highlight can be done without document mutations due to the selection bug in IE. So the scanning logic will have to be changed.

[Martin Kou]

Turns out there's a really hideous bug in FCKDomRange::CreateBookmark2() which causes the returned bookmark value to be invalid when it encounters a set of DOM nodes like the following:

[Text node][Text node]...[Text node][Element node][Text node]

With the bold text node being one of the boundaries of the DOM range.

Fixing the bug in FCKDomRange::CreateBookmark2() fixes the issue with the Find/Replace dialog, as its scanning logic depends on CreaetBookmark2(). The scanning logic in the Find/Replace dialog is otherwise correct.

[Martin Kou]

Fixed with [1928].

Click here for more info about our SVN system.