Refactor to make BlimpLocationProvider accessible to content layer.

blimp/engine/app/blimp_content_browser_client.h

Index: blimp/engine/app/blimp_content_browser_client.h
diff --git a/blimp/engine/app/blimp_content_browser_client.h b/blimp/engine/app/blimp_content_browser_client.h
index ab5e89578e5d1e634a3e13afc166cfe3a05ab1d5..5554609dc71ab932389800cf60bc8ffbeda1aeaf 100644
--- a/blimp/engine/app/blimp_content_browser_client.h
+++ b/blimp/engine/app/blimp_content_browser_client.h
@@ -6,6 +6,7 @@
 #define BLIMP_ENGINE_APP_BLIMP_CONTENT_BROWSER_CLIENT_H_
 
 #include "base/macros.h"
+#include "blimp/engine/feature/geolocation/blimp_location_provider.h"
 #include "content/public/browser/content_browser_client.h"
 
 namespace blimp {
@@ -33,10 +34,14 @@ class BlimpContentBrowserClient : public content::ContentBrowserClient {
 
   BlimpBrowserContext* GetBrowserContext();
 
+  content::LocationProvider* GetSoleLocationProvider() override;

[Kevin M, 2016/05/31 22:42:55]

The implementation is the same as OverrideSystemLocationProvider, but with
subtly different semantics that must be obeyed by the caller. What if we just
expressed those semantics specifically via a getter interface? Like e.g. have a
virtual bool "UseDefaultLocationProviders()" that would return true by default
in the base class, but would be overridden to return false for the
BlimpContentBrowserClient.

We would use the same OverrideSystemLocationProvider() method in that case.

+
  private:
   // Owned by BrowserMainLoop
   BlimpBrowserMainParts* blimp_browser_main_parts_ = nullptr;
 
+  BlimpLocationProvider location_provider_;
+
   DISALLOW_COPY_AND_ASSIGN(BlimpContentBrowserClient);
 };