Refactor to make BlimpLocationProvider accessible to content layer.

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 7a61399c7d788336dfba16b3a9796183d0ef96da..e5910bf852c244037d70e4bf7a7ae8bb2ab6606d 100644
--- a/content/public/browser/content_browser_client.h
+++ b/content/public/browser/content_browser_client.h
@@ -621,6 +621,10 @@ class CONTENT_EXPORT ContentBrowserClient {
   // information.

[Wez, 2016/06/01 21:45:19]

nit: Now that we're actually using this in Chromium we could consider removing
this comment?

[CJ, 2016/06/01 22:37:10]

Maybe keep it in case a refactor removes our use case? 

[Wez, 2016/06/10 23:40:34]

Acknowledged.

   virtual LocationProvider* OverrideSystemLocationProvider();
 
+  // Indicates that LocationProviders normally used by a process should be
+  // used.

[Wez, 2016/06/01 21:45:19]

Not sure what "used by a process" means here.

I'd suggest something like "Returns true if the location API should fall-back to
using network-based location approximation." or similar.

[CJ, 2016/06/01 22:37:10]

It doesn't really fall back on it though? If true, we use it in addition to the
SystemLocationProvider. I did update the comment though. Let me know if you
think this suffices. 

[Wez, 2016/06/10 23:40:34]

Looks good - you might say "... in addition to the system provider, if any."

[CJ, 2016/06/13 23:45:20]

Done.

+  virtual bool UseDefaultLocationProviders();
+
   // Creates a new DevToolsManagerDelegate. The caller owns the returned value.
   // It's valid to return nullptr.
   virtual DevToolsManagerDelegate* GetDevToolsManagerDelegate();