GitLab

This CL cleans up std::string usage in translate_language_list.cc and
translate_browser_metrics.cc, expanding to callers in a few places. This
CL also includes some miscellaneous cleanup to these files, such as
removing the test-only method
translate::TranslateBrowserMetrics::GetMetricsName() and instead inlines
the string literals in the tests themselves.

This CL changes no functionality, and is clean up only.

Bug: 1109032
Change-Id: Icc18b29b7f79ec9e57b491548d1f05e032923ee8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2332363Reviewed-by: Megan Jablonski <megjablon@chromium.org>
Commit-Queue: Scott Little <sclittle@chromium.org>
Cr-Commit-Position: refs/heads/master@{#795603}