Audit the last usage of Geolocation and Notification permissions.

content/browser/geolocation/geolocation_dispatcher_host.cc

Index: content/browser/geolocation/geolocation_dispatcher_host.cc
diff --git a/content/browser/geolocation/geolocation_dispatcher_host.cc b/content/browser/geolocation/geolocation_dispatcher_host.cc
index ad1f2fcc4441f138726f67d6f0e8e049302e5962..b89368f31d2e13db6f6a2fc642b7ce1a9a57e623 100644
--- a/content/browser/geolocation/geolocation_dispatcher_host.cc
+++ b/content/browser/geolocation/geolocation_dispatcher_host.cc
@@ -21,6 +21,8 @@
 namespace content {
 namespace {
 
+const char kGeolocationPermission[] = "geolocation";
+
 // Geoposition error codes for reporting in UMA.
 enum GeopositionErrorCode {
   // NOTE: Do not renumber these as that would confuse interpretation of
@@ -186,6 +188,12 @@ void GeolocationDispatcherHost::OnStartUpdating(
       "Geolocation.GeolocationDispatcherHostImpl.EnableHighAccuracy",
       enable_high_accuracy);
 
+  GetContentClient()->browser()->UseContentSettingPermission(

[Michael van Ouwerkerk, 2014/07/01 11:02:23]

If this is meant to track actual successful usage of the Geolocation API then
I'd suggest doing that in OnLocationUpdate. That's where location data is sent
to the renderers. In fact, you'd want to track usage for each RenderFrame that
we send the data to, otherwise the tracking is incomplete.

[Daniel Nishi, 2014/07/01 18:37:48]

I've moved the call into OnLocationUpdate in the for loop to grab all
RenderFrameHosts.

Upon further inspection, I noticed that the |requesting_frame| coming into
OnStartUpdating actually is just a GURL() without anything in it, anyway.

+      web_contents(),
+      requesting_frame,
+      web_contents()->GetLastCommittedURL().GetOrigin(),
+      kGeolocationPermission);
+
   updating_frames_[render_frame_host] = enable_high_accuracy;
   RefreshGeolocationOptions();
 }