Audit the last usage of Geolocation and Notification permissions.

content/public/browser/content_browser_client.h

Index: content/public/browser/content_browser_client.h
diff --git a/content/public/browser/content_browser_client.h b/content/public/browser/content_browser_client.h
index 356c724e59da6db092aa0055e5fa68f3d94432e0..665e4fad427b138eb5e7c95bc78b2a086c4edd6b 100644
--- a/content/public/browser/content_browser_client.h
+++ b/content/public/browser/content_browser_client.h
@@ -474,6 +474,11 @@ class CONTENT_EXPORT ContentBrowserClient {
       base::Callback<void(bool)> result_callback,
       base::Closure* cancel_callback);
 
+  // Invoked when the Geolocation API uses its permission.
+  virtual void DidUseGeolocationPermission(WebContents* web_contents,

[jam, 2014/07/14 18:20:53]

nit: put this at line 456 so it's right after the other geolocation related
method

[Daniel Nishi, 2014/07/14 22:25:05]

Done.

+                                           const GURL& primary_url,

[jam, 2014/07/14 18:20:53]

nit: primary and secondary urls is ambiguous. use frame_url and main_frame_url
to be clear

[Daniel Nishi, 2014/07/14 22:25:05]

Done.

+                                           const GURL& secondary_url);
+
   // Returns true if the given page is allowed to open a window of the given
   // type. If true is returned, |no_javascript_access| will indicate whether
   // the window that is created should be scriptable/in the same process.