Clean up SVGSMILElement::FindInstanceTime

Use lambdas and iterators, handling the case where an instance time
equal to the request time is not a valid result in the predicate passed
to std::lower_bound().

There's theoretically a change in behavior where (for 'begin' times), we
could previously have returned the magic 'indefinite' value after
looking for the next larger instance time.

Change-Id: I5e51c85298530548c6527116086355b00ea7ad3d
Reviewed-on: https://chromium-review.googlesource.com/c/1400822
Reviewed-by: Stephen Chenney <schenney@chromium.org>
Commit-Queue: Fredrik Söderquist <fs@opera.com>
Cr-Commit-Position: refs/heads/master@{#621236}
1 file changed