Geolocation cleanup: make GeolocationDelegate::OverrideSystemLocationProvider return unique_ptr

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 2dc6047d12055564a8434c7efaa86ba61a413042..1b1b37a3ed26714742c36c9474ce49cd6995c1bd 100644
--- a/content/public/browser/geolocation_delegate.h
+++ b/content/public/browser/geolocation_delegate.h
@@ -20,17 +20,17 @@ class CONTENT_EXPORT GeolocationDelegate {
   // Returns true if the location API should use network-based location
   // approximation in addition to the system provider, if any.
   virtual bool UseNetworkLocationProviders();
-   // Creates a new AccessTokenStore for geolocation. May return nullptr.
+
+  // 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();
-   // Allows an embedder to return its own LocationProvider implementation.
-  // Return nullptr to use the default one for the platform to be created.
-  // Caller takes ownership of the returned LocationProvider. FYI: Used by an
-  // external project; please don't remove. Contact Viatcheslav Ostapenko at
-  // sl.ostapenko@samsung.com for more information.
-  // TODO(mcasas): return std::unique_ptr<> instead, https://crbug.com/623132.
-  virtual LocationProvider* OverrideSystemLocationProvider();
+
+  // Allows an embedder to return its own LocationProvider implementation.
+  // Return nullptr to use the default one for the platform to be created. FYI:

[Wez, 2016/07/07 21:47:10]

nit: Suggest starting a new line for the FYI: so it stands out to the reader.

[mcasas, 2016/07/07 23:58:42]

Done.

+  // Used by an external project; please don't remove. Contact Viatcheslav
+  // Ostapenko at sl.ostapenko@samsung.com for more information.
+  virtual std::unique_ptr<LocationProvider> OverrideSystemLocationProvider();
 };
 
 }  // namespace content