[GCM] Add more UMA to GCM

chrome/browser/extensions/api/gcm/gcm_api.cc

 // Copyright 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/extensions/api/gcm/gcm_api.h"
 
 #include <algorithm>
 #include <map>
 #include <vector>
 
+#include "base/metrics/histogram.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/services/gcm/gcm_profile_service.h"
 #include "chrome/browser/services/gcm/gcm_profile_service_factory.h"
 #include "chrome/common/extensions/api/gcm.h"
 #include "extensions/common/extension.h"
 
 namespace {
@@ skipping 103 matching lines @@
   SetResult(new base::StringValue(registration_id));
   SetError(GcmResultToError(result));
   SendResponse(gcm::GCMClient::SUCCESS == result);
 }
 
 GcmUnregisterFunction::GcmUnregisterFunction() {}
 
 GcmUnregisterFunction::~GcmUnregisterFunction() {}
 
 bool GcmUnregisterFunction::DoWork() {
+  UMA_HISTOGRAM_COUNTS("GCM.APICallUnregister", 1);

[Alexei Svitkine (slow), 2014/05/08 21:21:57]

Use an UMA_HISTOGRAM_BOOLEAN() instead. The resulting histogram will be the
same, but it will just allocate 2 buckets instead of 50, so a memory saving.

[jianli, 2014/05/08 21:26:07]

Done.

+
   GCMProfileService()->Unregister(
       GetExtension()->id(),
       base::Bind(&GcmUnregisterFunction::CompleteFunctionWithResult, this));
 
   return true;
 }
 
 void GcmUnregisterFunction::CompleteFunctionWithResult(
     gcm::GCMClient::Result result) {
   SetError(GcmResultToError(result));
@@ skipping 107 matching lines @@
 
 void GcmJsEventRouter::OnListenerAdded(const EventListenerInfo& details) {
   if (gcm::GCMProfileService::GetGCMEnabledState(profile_) ==
       gcm::GCMProfileService::ALWAYS_DISABLED) {
     return;
   }
   gcm::GCMProfileServiceFactory::GetForProfile(profile_)->Start();
 }
 
 }  // namespace extensions