Cleanup histogram classes mixing SetFlags into FactoryGet arguments...

chrome/browser/sync/profile_sync_service.cc

 // Copyright (c) 2006-2008 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/sync/profile_sync_service.h"
 
 #include <stack>
 #include <vector>
 
 #include "app/l10n_util.h"
@@ skipping 449 matching lines @@
 
 void ProfileSyncService::AddObserver(Observer* observer) {
   observers_.AddObserver(observer);
 }
 
 void ProfileSyncService::RemoveObserver(Observer* observer) {
   observers_.RemoveObserver(observer);
 }
 
 void ProfileSyncService::SyncEvent(SyncEventCodes code) {
-  static scoped_refptr<Histogram> histogram =
-      LinearHistogram::LinearHistogramFactoryGet("Sync.EventCodes",
-          MIN_SYNC_EVENT_CODE + 1, MAX_SYNC_EVENT_CODE - 1,
-          MAX_SYNC_EVENT_CODE);
-  histogram->SetFlags(kUmaTargetedHistogramFlag);
-  histogram->Add(code);
+  UMA_HISTOGRAM_ENUMERATION("Sync.EventCodes", code, MAX_SYNC_EVENT_CODE);
 }
 
 bool ProfileSyncService::IsSyncEnabled() {
   // We have switches::kEnableSync just in case we need to change back to
   // sync-disabled-by-default on a platform.
   return !CommandLine::ForCurrentProcess()->HasSwitch(switches::kDisableSync);
 }
 
 bool ProfileSyncService::ShouldPushChanges() {
   // True only after all bootstrapping has succeeded: the bookmark model is
   // loaded, the sync backend is initialized, the two domains are
   // consistent with one another, and no unrecoverable error has transpired.
   for (std::set<ChangeProcessor*>::const_iterator it =
        change_processors_.begin(); it != change_processors_.end(); ++it) {
     if (!(*it)->IsRunning()) return false;
   }
   return true;
 }