Move LanguageUsageMetrics and TranslateBrowserMetrics to components

components/language_usage_metrics/language_usage_metrics.h

 // Copyright (c) 2011 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.
 
-#ifndef CHROME_BROWSER_LANGUAGE_USAGE_METRICS_H_
-#define CHROME_BROWSER_LANGUAGE_USAGE_METRICS_H_
+#ifndef COMPONENTS_LANGUAGE_USAGE_METRICS_LANGUAGE_USAGE_METRICS_H_
+#define COMPONENTS_LANGUAGE_USAGE_METRICS_LANGUAGE_USAGE_METRICS_H_
 
 #include <set>
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/gtest_prod_util.h"
 
 // Methods to record language usage as UMA histograms.

[blundell, 2014/01/09 09:53:08]

This should be in the language_usage_metrics namespace (I know you're holding
off on the translate one until that componentization is complete; no reason to
hold off on this one).

 class LanguageUsageMetrics {

[Jói, 2014/01/09 10:44:14]

Code in a component located in //components/xyz should be in namespace xyz, so
this should be in namespace language_usage_metrics.

I don't mind if you defer that change to a follow-up, but I would recommend
doing it right after.

[droger, 2014/01/09 11:52:28]

Done.

  public:
   // Records accept languages as a UMA histogram. |accept_languages| is a
   // case-insensitive comma-separated list of languages/locales of either xx,
   // xx-YY, or xx_YY format where xx is iso-639 language code and YY is iso-3166
   // country code. Country code is ignored. That is, xx and XX-YY are considered
   // identical and recorded once.
   static void RecordAcceptLanguages(const std::string& accept_languages);
 
   // Records the application language as a UMA histogram. |application_locale|
   // is a case-insensitive locale string of either xx, xx-YY, or xx_YY format.
@@ skipping 13 matching lines @@
   DISALLOW_IMPLICIT_CONSTRUCTORS(LanguageUsageMetrics);
 
   // Parses |accept_languages| and returns a set of language codes in
   // |languages|.
   static void ParseAcceptLanguages(const std::string& accept_languages,
                                    std::set<int>* languages);
 
   FRIEND_TEST_ALL_PREFIXES(LanguageUsageMetricsTest, ParseAcceptLanguages);
 };
 
-#endif  // CHROME_BROWSER_LANGUAGE_USAGE_METRICS_H_
+#endif  // COMPONENTS_LANGUAGE_USAGE_METRICS_LANGUAGE_USAGE_METRICS_H_