Extract guts of ChromeStabilityMetricsProvider into helper class

chrome/browser/android/metrics/uma_session_stats.cc

 // Copyright 2015 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/android/metrics/uma_session_stats.h"
 
 #include "base/android/jni_string.h"
 #include "base/command_line.h"
 #include "base/metrics/histogram.h"
 #include "base/prefs/pref_service.h"
@@ skipping 101 matching lines @@
   g_browser_process->GetMetricsServicesManager()->UpdatePermissions(
       may_record, may_upload);
 }
 
 // Renderer process crashed in the foreground.
 static void LogRendererCrash(JNIEnv*, const JavaParamRef<jclass>&) {
   DCHECK(g_browser_process);
   // Increment the renderer crash count in stability metrics.
   PrefService* pref = g_browser_process->local_state();
   DCHECK(pref);
-  int value = pref->GetInteger(prefs::kStabilityRendererCrashCount);
-  pref->SetInteger(prefs::kStabilityRendererCrashCount, value + 1);
+  int value = pref->GetInteger(metrics::prefs::kStabilityRendererCrashCount);
+  pref->SetInteger(metrics::prefs::kStabilityRendererCrashCount, value + 1);
 }
 
 static void RegisterExternalExperiment(JNIEnv* env,
                                        const JavaParamRef<jclass>& clazz,
                                        jint study_id,
                                        jint experiment_id) {
   const std::string group_name_utf8 = base::IntToString(experiment_id);
 
   variations::ActiveGroupId active_group;
   active_group.name = static_cast<uint32>(study_id);
@@ skipping 57 matching lines @@
   // We should have only one UmaSessionStats instance.
   DCHECK(!g_uma_session_stats);
   g_uma_session_stats = new UmaSessionStats();
   return reinterpret_cast<intptr_t>(g_uma_session_stats);
 }
 
 // Register native methods
 bool RegisterUmaSessionStats(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }