base: CommandLine::RemoveSwitch should drop multiple same switches from argv

When there are multiple switches having the same key within the argv
array, CommandLine constructor only puts one of them within the
parsed-out switches map.

On the other hand, CommandLine::RemoveSwitch removes a switch from the
switches map.  It should also drop all the switches having the same key
from the argv array.

BUG=957138
TEST=base_unittests --gtest_filter="CommandLineTest.*"

Change-Id: Ia4cf08bc497d4d53e0213879df71f85d945a4fec
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1724079
Auto-Submit: Andrei Polushin <anpol@yandex-team.ru>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#682443}
2 files changed