BSONIteratorSorted traverses the provided BSONObj to collect all field names. But when storing it just uses char*.

This leads to the comparator having to re-calculate the field name size over and over again during the sorting stage.

Later when a user is extracting BSONElements in sorted order the field name sizes are calculated yet again.

We should be able to just calculate it once and then reuse.