Make the findMetricsForCharacter SVGTextQuery-helper return an iterator

Make the findMetricsForCharacter SVGTextQuery-helper return an iterator

Convert the helper to return an iterator into the metrics vector. Then
use that new property in the calculateGlyphRange helper to avoid O(n^2)
runtime.

Also remove modifyStartEndPositionsRespectingLigatures, because all
users of it (indirectly through
mapStartEndPositionsIntoFragmentCoordinates) now uses the pre-computed
text metrics - which should account for the same thing automatically.
Also fix up the TODO in endPositionOfCharacterCallback by doing what it
says.

Committed: https://crrev.com/bd5d36e41e8ed061b646157465c91b4530b117d1
Cr-Commit-Position: refs/heads/master@{#382536}

[fs, 2016-03-21 13:32:20]

Description was changed from

==========
Make the findMetricsForCharacter SVGTextQuery-helper return an iterator

Convert the helper to return an iterator into the metrics vector. Then
use that new property in the calculateGlyphRange helper to avoid O(N^2)
runtime.

Also remove modifyStartEndPositionsRespectingLigatures, because all
users of it (indirectly through
mapStartEndPositionsIntoFragmentCoordinates) now uses the pre-computed
text metrics - which should account for the same thing automatically.
Also fix up the TODO in endPositionOfCharacterCallback by doing what it
says.
==========

to

==========
Make the findMetricsForCharacter SVGTextQuery-helper return an iterator

Convert the helper to return an iterator into the metrics vector. Then
use that new property in the calculateGlyphRange helper to avoid O(n^2)
runtime.

Also remove modifyStartEndPositionsRespectingLigatures, because all
users of it (indirectly through
mapStartEndPositionsIntoFragmentCoordinates) now uses the pre-computed
text metrics - which should account for the same thing automatically.
Also fix up the TODO in endPositionOfCharacterCallback by doing what it
says.
==========

[fs, 2016-03-21 13:32:31]

fs@opera.com changed reviewers:
+ pdr@chromium.org

[pdr., 2016-03-21 23:00:26]

On 2016/03/21 at 13:32:31, fs wrote:
> 

LGTM, so much cleaner

[fs, 2016-03-22 08:44:49]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-03-22 08:45:01]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1822673002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1822673002/1

[commit-bot: I haz the power, 2016-03-22 09:12:04]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-22 09:13:35]

Description was changed from

==========
Make the findMetricsForCharacter SVGTextQuery-helper return an iterator

Convert the helper to return an iterator into the metrics vector. Then
use that new property in the calculateGlyphRange helper to avoid O(n^2)
runtime.

Also remove modifyStartEndPositionsRespectingLigatures, because all
users of it (indirectly through
mapStartEndPositionsIntoFragmentCoordinates) now uses the pre-computed
text metrics - which should account for the same thing automatically.
Also fix up the TODO in endPositionOfCharacterCallback by doing what it
says.
==========

to

==========
Make the findMetricsForCharacter SVGTextQuery-helper return an iterator

Convert the helper to return an iterator into the metrics vector. Then
use that new property in the calculateGlyphRange helper to avoid O(n^2)
runtime.

Also remove modifyStartEndPositionsRespectingLigatures, because all
users of it (indirectly through
mapStartEndPositionsIntoFragmentCoordinates) now uses the pre-computed
text metrics - which should account for the same thing automatically.
Also fix up the TODO in endPositionOfCharacterCallback by doing what it
says.

Committed: https://crrev.com/bd5d36e41e8ed061b646157465c91b4530b117d1
Cr-Commit-Position: refs/heads/master@{#382536}
==========

[commit-bot: I haz the power, 2016-03-22 09:13:36]

Patchset 1 (id:??) landed as
https://crrev.com/bd5d36e41e8ed061b646157465c91b4530b117d1
Cr-Commit-Position: refs/heads/master@{#382536}