V3: Move Range.CreateBookmark2 from V2

[Frederico Caldeira Knabben]

There are a few features depending on the CreateBookmark2 and MoveToBookmark2 present on V2. These functions must be moved to V3.

[Frederico Caldeira Knabben]

The proposed patch is much simpler than the implementation present on V2. So, an accurate review is needed.

[Martin Kou]

There's still a case where the bookmarks that came out of createBookmark2() is non-normalized.

Let's say after running a certain command, a text node in the following paragraph is broken up to two pieces:

<p>Part 1 Part 2</p>

And the selection range looks like this, which means it selects "Part 2" only:

startContainer : <p>
startOffset : 1

When we call createBookmark2() with this range:

1. The normalization logic in createBookmark2() isn't executed, because startContainer is not a text node.
2. The normalization logic in getAddress() is also not executed, for the same reason.

But the index path in the bookmark is certainly not normalized.

[Martin Kou]

Oops, forgot to delete the ? symbol.

[Frederico Caldeira Knabben]

This new patch fixes and include a test for Martin's case. It also makes the normalization optional, and makes the moveToBookmark function work for both kinds of bookmarks.

[Frederico Caldeira Knabben]

I've attached a new patch as the previous one was not considering the normalization option when movingToBookmark.

[Frederico Caldeira Knabben]

Fixed with [3065]. Click here for more info about our SVN system.