Refactor to make BlimpLocationProvider accessible to content layer.

content/browser/geolocation/location_arbitrator.h

Index: content/browser/geolocation/location_arbitrator.h
diff --git a/content/browser/geolocation/location_arbitrator.h b/content/browser/geolocation/location_arbitrator.h
index 3f9c6326e223884ec895429cfb2ce1eb59c2e7c2..7f550ae4df3d060d307c67d88126f842c7a9f594 100644
--- a/content/browser/geolocation/location_arbitrator.h
+++ b/content/browser/geolocation/location_arbitrator.h
@@ -17,7 +17,7 @@ public:
   virtual ~LocationArbitrator() {};
 
   // See more details in geolocation_provider.
-  virtual void StartProviders(bool enable_high_accuracy) = 0;
+  virtual void StartProviders(bool enable_high_accuracy, bool use_network) = 0;

[Michael van Ouwerkerk, 2016/06/06 17:17:28]

I'm not yet convinced that adding a bool here is the right way to go, it's a bit
confusing and not a very scalable approach. Maybe you can avoid adding this
argument by inserting a fake ContentBrowserClient instead? See
SetBrowserClientForTesting and TestContentBrowserClient. That way, you could
fake its methods like OverrideSystemLocationProvider and
UseNetworkLocationProviders.

[Kevin M, 2016/06/08 20:47:25]

+1

   virtual void StopProviders() = 0;
 
   // Called everytime permission is granted to a page for using geolocation.