Audit the last usage of Geolocation and Notification permissions.

content/public/browser/content_browser_client.h

 // Copyright (c) 2012 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_
 #define CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_
 
 #include <map>
 #include <string>
 #include <utility>
@@ skipping 456 matching lines @@
 
   // Request permission to access protected media identifier. |result_callback
   // will tell whether it's permitted. If |cancel_callback| is non-null, it's
   // set to a callback which can be used to cancel the permission request.
   virtual void RequestProtectedMediaIdentifierPermission(
       WebContents* web_contents,
       const GURL& origin,
       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.
   // This is called on the IO thread.
   virtual bool CanCreateWindow(const GURL& opener_url,
                                const GURL& opener_top_level_frame_url,
                                const GURL& source_origin,
                                WindowContainerType container_type,
                                const GURL& target_url,
                                const Referrer& referrer,
@@ skipping 161 matching lines @@
   // Allows an embedder to provide its own ExternalVideoSurfaceContainer
   // implementation.  Return NULL to disable external surface video.
   virtual ExternalVideoSurfaceContainer*
   OverrideCreateExternalVideoSurfaceContainer(WebContents* web_contents);
 #endif
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_