tree 482a61ef335461b81a891d23936bd25388ea2a54
parent efa1e2e0a1249edaef4399bf45b535f43ea27099
author Sylvain Defresne <sdefresne@chromium.org> 1620053133 +0200
committer Commit Bot <commit-bot@chromium.org> 1620296971 +0000

Pass std::string_view by value

As std::string_view is a relatively small structure it can be passed
by value. The code was using a mix of passing it either by value or
by const reference. Settle on passing by value.

Remove some unnecessary templated functions that was only overloaded
for `const std::string&` and `const std::string_view` as the latter
was enough.

Fix IWYU by adding `#include <string_view>` in all header files that
uses std::string_view.

Bug: none
Change-Id: If42ab28342e8d73f6f0543e77319fdf6380c41b5
Reviewed-on: https://gn-review.googlesource.com/c/gn/+/11400
Reviewed-by: Brett Wilson <brettw@chromium.org>
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>