Final batch adding real histogram values for extension events.

chrome/browser/extensions/api/location/location_manager.cc

 // Copyright (c) 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/location/location_manager.h"
 
 #include <math.h>
 #include <vector>
 
 #include "base/bind.h"
@@ skipping 295 matching lines @@
     coordinates->speed.reset(new double(position.speed));
 }
 
 void LocationManager::SendLocationUpdate(
     const std::string& extension_id,
     const std::string& request_name,
     const content::Geoposition& position) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 
   scoped_ptr<base::ListValue> args(new base::ListValue());
+  events::HistogramValue histogram_value = events::UNKNOWN;
   std::string event_name;
 
   if (position.Validate() &&
       position.error_code == content::Geoposition::ERROR_CODE_NONE) {
     // Set data for onLocationUpdate event.
     location::Location location;
     location.name = request_name;
     GeopositionToApiCoordinates(position, &location.coords);
     location.timestamp = position.timestamp.ToJsTime();
 
     args->Append(location.ToValue().release());
+    histogram_value = events::LOCATION_ON_LOCATION_UPDATE;
     event_name = location::OnLocationUpdate::kEventName;
   } else {
     // Set data for onLocationError event.
     // TODO(vadimt): Set name.
     args->AppendString(position.error_message);
+    histogram_value = events::LOCATION_ON_LOCATION_ERROR;
     event_name = location::OnLocationError::kEventName;
   }
 
-  scoped_ptr<Event> event(new Event(events::UNKNOWN, event_name, args.Pass()));
+  scoped_ptr<Event> event(new Event(histogram_value, event_name, args.Pass()));
 
   EventRouter::Get(browser_context_)
       ->DispatchEventToExtension(extension_id, event.Pass());
 }
 
 void LocationManager::OnExtensionLoaded(
     content::BrowserContext* browser_context,
     const Extension* extension) {
   // Grants permission to use geolocation once an extension with "location"
   // permission is loaded.
@@ skipping 20 matching lines @@
 LocationManager::GetFactoryInstance() {
   return g_factory.Pointer();
 }
 
 // static
 LocationManager* LocationManager::Get(content::BrowserContext* context) {
   return BrowserContextKeyedAPIFactory<LocationManager>::Get(context);
 }
 
 }  // namespace extensions