Improve rejection test for character-based SVG text queries

Improve rejection test for character-based SVG text queries

Perform an intersection of the involved ranges to reject fragments early,
while still avoiding calls to mapStartEndPositionsIntoFragmentCoordinates.
This means the "cache" added in https://codereview.chromium.org/349223003
can be dropped, since we'll now never even reach it. This gives a moderate
speed-up - a few percent - on the original TC compared to using the cache.

BUG=267504
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=177148

[fs, 2014-06-27 15:44:03]

Previously, on issue 267504: The protagonist adds a cache for ligature
adjustment.
In this episode: The protagonist realizes that may just have been overkill and
replaces it with a simple range intersection test. It's starts raining money.

[pdr., 2014-06-27 22:38:32]

On 2014/06/27 15:44:03, fs wrote:
> Previously, on issue 267504: The protagonist adds a cache for ligature
> adjustment.
> In this episode: The protagonist realizes that may just have been overkill and
> replaces it with a simple range intersection test. It's starts raining money.

I will play the oblivious sidekick who drinks too much and let the perpetrator
slip by unnoticed while the audience laughs.

Nice find. LGTM.

[pdr., 2014-06-27 22:38:37]

The CQ bit was checked by pdr@chromium.org

[commit-bot: I haz the power, 2014-06-27 22:38:58]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/fs@opera.com/359783002/1

[commit-bot: I haz the power, 2014-06-27 22:41:59]

Change committed as 177148