Simplify DocumentCursor::setPosition

Seems like we don't care for validity in this case, so just simply assign without checking for isValid(). The function does show up a lot in profiles that I did for checking why anchorInternal() was taking so much time. Also, removed the pass by ref as that is just unnecessary indirection.