country_names.cc: change SetLocaleString to take const ref instead of value

components/autofill/core/browser/country_names.cc

Index: components/autofill/core/browser/country_names.cc
diff --git a/components/autofill/core/browser/country_names.cc b/components/autofill/core/browser/country_names.cc
index f5b4cdff3aa87b02b3eec2379bdbe69a5d563dec..7a47d1f367a69ac21892cc76ecc86878dcbcf158 100644
--- a/components/autofill/core/browser/country_names.cc
+++ b/components/autofill/core/browser/country_names.cc
@@ -5,7 +5,6 @@
 #include "components/autofill/core/browser/country_names.h"
 
 #include <stdint.h>
-#include <utility>
 
 #include "base/lazy_instance.h"
 #include "base/logging.h"
@@ -142,13 +141,13 @@ CountryNames* CountryNames::GetInstance() {
 }
 
 // static
-void CountryNames::SetLocaleString(std::string locale) {
+void CountryNames::SetLocaleString(const std::string& locale) {
   DCHECK(!locale.empty());
   // Application locale should never be empty. The empty value of
   // |g_application_locale| means that it has not been initialized yet.
   std::string* storage = g_application_locale.Pointer();
   if (storage->empty()) {
-    *storage = std::move(locale);
+    *storage = locale;
   }
   // TODO(crbug.com/579971) CountryNames currently cannot adapt to changed
   // locale without Chrome's restart.