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..b9a3d5672ae07b4a32b061f199cd15bbf3b6aca0 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
@@ -130,6 +132,11 @@ void GeolocationDispatcherHost::OnLocationUpdate(
 
   for (std::map<RenderFrameHost*, bool>::iterator i = updating_frames_.begin();
        i != updating_frames_.end(); ++i) {
+    GetContentClient()->browser()->UseContentSettingPermission(
+        web_contents(),
+        i->first->GetLastCommittedURL().GetOrigin(),
+        i->first->GetLastCommittedURL().GetOrigin(),

[Michael van Ouwerkerk, 2014/07/02 18:51:15]

If I understand correctly, this should be the url of the top level frame, in
case this is an iframe. If that is true, I think you'll need to traverse the
parent frames until you find the top level frame.

[Daniel Nishi, 2014/07/07 17:46:40]

Done. We traverse the parents now.

+        kGeolocationPermission);
     i->first->Send(new GeolocationMsg_PositionUpdated(
         i->first->GetRoutingID(), geoposition));
   }