Some new unittests for ReplaceStringPlaceholders, inspired by
some of the tricky cases for ReplaceSubstringsAfterOffsets.
These tests uncovered a correctness bug involving a sorted vector,
which is also fixed.
Bug: None
Change-Id: Ic96d709e70732705e7bf3be9655e5427bdfe396f
Reviewed-on: https://chromium-review.googlesource.com/648215
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Nick Carter <nick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504824}
CrOS-Libchrome-Original-Commit: e2c7d65de60f1db8d105e97373e30d71b7767148
2 files changed