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-by: Stephen Chenney <>
Commit-Queue: Fredrik Söderquist <>
Cr-Commit-Position: refs/heads/master@{#621236}
1 file changed