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

components/autofill/core/browser/country_names.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/autofill/core/browser/country_names.h"
 
 #include <stdint.h>
-#include <utility>
 
 #include "base/lazy_instance.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/singleton.h"
 #include "base/stl_util.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "components/autofill/core/browser/country_data.h"
 #include "components/autofill/core/common/autofill_l10n_util.h"
@@ skipping 116 matching lines @@
 }
 
 }  // namespace
 
 // static
 CountryNames* CountryNames::GetInstance() {
   return base::Singleton<CountryNames>::get();
 }
 
 // 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.
 }
 
 CountryNames::CountryNames(const std::string& locale_name)
     : locale_(locale_name.c_str()),
       collator_(CreateCollator(locale_)),
       default_collator_(CreateDefaultCollator(locale_)),
       common_names_(GetCommonNames()),
@@ skipping 43 matching lines @@
 
   auto result = localized_names.find(sort_key);
 
   if (result != localized_names.end())
     return result->second;
 
   return std::string();
 }
 
 }  // namespace autofill