Report data usage UMA for Chrome services

components/data_use_measurement/core/data_use_user_data.cc

Index: components/data_use_measurement/core/data_use_user_data.cc
diff --git a/components/data_use_measurement/core/data_use_user_data.cc b/components/data_use_measurement/core/data_use_user_data.cc
index c6ad360a9afb488bc6e0a723de7cec8ab9678ff1..8fbe619ba62078954b6aff29aaa7d10312927444 100644
--- a/components/data_use_measurement/core/data_use_user_data.cc
+++ b/components/data_use_measurement/core/data_use_user_data.cc
@@ -31,9 +31,38 @@ const char* DataUseUserData::GetServiceName(
       return "Suggestions";
     case NOT_TAGGED:
       return "NotTagged";
-    default:
-      NOTREACHED() << "Unknown service type in DataUseUserData";
+    case TRANSLATE:
+      return "Translate";
+    case SYNC:
+      return "Sync";
+    case OMNIBOX:
+      return "Omnibox";
+    case INVALIDATION:
+      return "Invalidation";
+    case RAPPOR:
+      return "Rappor";
+    case POLICY:
+      return "Policy";
+    case UMA:
+      return "UMA";
+    case GOOGLE_APIS:
+      return "GoogleAPIs";
+    case DOMAIN_RELIABILITY:
+      return "DomainReliability";
+    case PROFILE_DOWNLOADER:
+      return "ProfileDownloader";
+    case GOOGLE_URL_TRACKER:
+      return "GoogleURLTracker";
+    case AUTOFILL:
+      return "Autofill";
+    case VARIATIONS:
+      return "Variations";
+    case SPELL_CHECKER:
+      return "SpellChecker";
+    case GEO_LOCATION:
+      return "GeoLocation";
   }
+  NOTREACHED() << "Unknown service type in DataUseUserData";

[Ilya Sherman, 2015/09/08 21:45:25]

nit: Please omit at least the string after the NOTREACHED -- it adds weight to
the binary, without providing any additional information beyond what a comment
could do.  Better yet, IMO, would be to remove the NOTREACHED() stmt entirely,
since the clang compiler will actually enforce that this code is not reachable.

[amohammadkhan, 2015/09/08 23:28:31]

Done.

   return "INVALID";
 }