Starting a refactor to allow adding metrics on upload.

chrome/browser/metrics/android_metrics_provider.cc

Index: chrome/browser/metrics/android_metrics_provider.cc
diff --git a/chrome/browser/metrics/android_metrics_provider.cc b/chrome/browser/metrics/android_metrics_provider.cc
index ddb41589c715bbea1a3da00abf8d4cc864306d4e..5904d88fef3864bc190f0f26d5b734ab1d0dcb2d 100644
--- a/chrome/browser/metrics/android_metrics_provider.cc
+++ b/chrome/browser/metrics/android_metrics_provider.cc
@@ -9,6 +9,7 @@
 #include "base/prefs/pref_service.h"
 #include "base/prefs/scoped_user_pref_update.h"
 #include "base/values.h"
+#include "chrome/browser/android/feature_utilities.h"
 #include "chrome/common/pref_names.h"
 
 namespace {
@@ -37,6 +38,12 @@ AndroidMetricsProvider::AndroidMetricsProvider(PrefService* local_state)
 AndroidMetricsProvider::~AndroidMetricsProvider() {
 }
 
+void AndroidMetricsProvider::OnCollectFinalMetrics() {
+  UMA_HISTOGRAM_ENUMERATION(
+      "DocumentMode.Enabled",

[Alexei Svitkine (slow), 2014/10/17 18:30:51]

Please don't introduce a new top-level prefix. Maybe use the same prefix as the
other histograms logged by this file - Chrome.Android.

Also, since this histogram is recorded by the open source code, can you put its
description in histograms.xml in the Chromium code - in the same CL?

[Maria, 2014/10/17 21:44:50]

I am reusing a top level prefix that we use in internal histograms now. Is that
ok? Also added the histograms.xml entry.

[Alexei Svitkine (slow), 2014/10/20 15:47:00]

Ah fair enough, OK,

+      chrome::android::GetDocumentModeValue(),
+      chrome::android::HISTOGRAM_MAX);
+}
 
 void AndroidMetricsProvider::ProvideStabilityMetrics(
     metrics::SystemProfileProto* system_profile_proto) {