Improve the code added in r194423 by using the Version class.

chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.cc

 // Copyright (c) 2013 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 "chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.h"
 
 #include <string>
-#include <vector>
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/cpu.h"
 #include "base/metrics/histogram.h"
 #include "base/threading/sequenced_worker_pool.h"
 #include "base/time.h"
 #include "chrome/browser/about_flags.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chrome_browser_main.h"
 #include "chrome/browser/shell_integration.h"
 #include "content/public/browser/browser_thread.h"
 #include "ui/base/touch/touch_device.h"
 #include "ui/base/ui_base_switches.h"
 
 #if defined(OS_LINUX) && !defined(OS_CHROMEOS)
 #include <gnu/libc-version.h>
 
-#include "base/strings/string_number_conversions.h"
-#include "base/strings/string_split.h"
+#include "base/version.h"
 #endif
 
 namespace {
 
 enum UMALinuxGlibcVersion {
   UMA_LINUX_GLIBC_NOT_PARSEABLE,
   UMA_LINUX_GLIBC_UNKNOWN,
   UMA_LINUX_GLIBC_2_11,
   UMA_LINUX_GLIBC_2_19 = UMA_LINUX_GLIBC_2_11 + 8,
   // NOTE: Add new version above this line and update the enum list in
@@ skipping 23 matching lines @@
 void RecordDefaultBrowserUMAStat() {
   // Record whether Chrome is the default browser or not.
   ShellIntegration::DefaultWebClientState default_state =
       ShellIntegration::GetDefaultBrowser();
   UMA_HISTOGRAM_ENUMERATION("DefaultBrowser.State", default_state,
                             ShellIntegration::NUM_DEFAULT_STATES);
 }
 
 void RecordLinuxGlibcVersion() {
 #if defined(OS_LINUX) && !defined(OS_CHROMEOS)
-  std::string glibc_version_string(gnu_get_libc_version());
-  std::vector<std::string> split_glibc_version;
-  base::SplitString(glibc_version_string, '.', &split_glibc_version);
+  Version version(gnu_get_libc_version());
 
   UMALinuxGlibcVersion glibc_version_result = UMA_LINUX_GLIBC_NOT_PARSEABLE;
-  unsigned glibc_major_version = 0;
-  unsigned glibc_minor_version = 0;
-  if (split_glibc_version.size() == 2 &&
-      base::StringToUint(split_glibc_version[0], &glibc_major_version) &&
-      base::StringToUint(split_glibc_version[1], &glibc_minor_version)) {
+  if (version.IsValid() && version.components().size() == 2) {
     glibc_version_result = UMA_LINUX_GLIBC_UNKNOWN;
+    int glibc_major_version = version.components()[0];
+    int glibc_minor_version = version.components()[1];
     if (glibc_major_version == 2) {
       // A constant to translate glibc 2.x minor versions to their
       // equivalent UMALinuxGlibcVersion values.
-      const unsigned kGlibcMinorVersionTranslationOffset =
-          11 - UMA_LINUX_GLIBC_2_11;
-      unsigned translated_glibc_minor_version =
+      const int kGlibcMinorVersionTranslationOffset = 11 - UMA_LINUX_GLIBC_2_11;
+      int translated_glibc_minor_version =
           glibc_minor_version - kGlibcMinorVersionTranslationOffset;
       if (translated_glibc_minor_version >= UMA_LINUX_GLIBC_2_11 &&
           translated_glibc_minor_version <= UMA_LINUX_GLIBC_2_19) {
         glibc_version_result =
             static_cast<UMALinuxGlibcVersion>(translated_glibc_minor_version);
       }
     }
   }
   UMA_HISTOGRAM_ENUMERATION("Linux.GlibcVersion", glibc_version_result,
                             UMA_LINUX_GLIBC_VERSION_COUNT);
@@ skipping 52 matching lines @@
   RecordTouchEventState();
 }
 
 namespace chrome {
 
 void AddMetricsExtraParts(ChromeBrowserMainParts* main_parts) {
   main_parts->AddParts(new ChromeBrowserMainExtraPartsMetrics());
 }
 
 }  // namespace chrome