Geolocation cleanup: corrects uses of content::AccessTokenStore* and net::URLRequestContextGetter*

content/public/browser/geolocation_delegate.h

Index: content/public/browser/geolocation_delegate.h
diff --git a/content/public/browser/geolocation_delegate.h b/content/public/browser/geolocation_delegate.h
index 0e94c01ae78803938e1f45b5b57f4b86e889e559..36ff280a815387a560553228832e4d0bf6069632 100644
--- a/content/public/browser/geolocation_delegate.h
+++ b/content/public/browser/geolocation_delegate.h
@@ -7,6 +7,7 @@
 
 #include <memory>
 
+#include "base/memory/ref_counted.h"
 #include "content/common/content_export.h"
 
 namespace content {
@@ -24,9 +25,7 @@ class CONTENT_EXPORT GeolocationDelegate {
   virtual bool UseNetworkLocationProviders();
 
   // Creates a new AccessTokenStore for geolocation. May return nullptr.
-  // TODO(mcasas): consider changing it return type to std::unique_ptr<> to
-  // clarify ownership, https://crbug.com/623114.
-  virtual AccessTokenStore* CreateAccessTokenStore();
+  virtual scoped_refptr<AccessTokenStore> CreateAccessTokenStore();
 
   // Allows an embedder to return its own LocationProvider implementation.
   // Return nullptr to use the default one for the platform to be created.