Refactor to make BlimpLocationProvider accessible to content layer.

blimp/engine/app/blimp_content_browser_client.cc

Index: blimp/engine/app/blimp_content_browser_client.cc
diff --git a/blimp/engine/app/blimp_content_browser_client.cc b/blimp/engine/app/blimp_content_browser_client.cc
index 8e88f5dfb6a19a1d6d5362c72bb9d5d37e969921..b99a881727f37eb9572000d1be5505dfce055c77 100644
--- a/blimp/engine/app/blimp_content_browser_client.cc
+++ b/blimp/engine/app/blimp_content_browser_client.cc
@@ -39,6 +39,18 @@ BlimpBrowserContext* BlimpContentBrowserClient::GetBrowserContext() {
   return blimp_browser_main_parts_->GetBrowserContext();
 }
 
+content::LocationProvider*
+BlimpContentBrowserClient::OverrideSystemLocationProvider() {
+  if (location_provider_ == nullptr) {

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

nit: if (!location_provider_)

(Testing a pointer against nullptr is a special-case in which we test-as-bool)

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

Thanks, I was wondering what was the actual way we did this!

Done.

+    location_provider_ = base::WrapUnique(new BlimpLocationProvider());
+  }
+  return location_provider_.get();
+}
+
+bool BlimpContentBrowserClient::UseDefaultLocationProviders() {
+  return false;
+}
+
 void BlimpContentBrowserClient::RegisterRenderProcessMojoServices(
     content::ServiceRegistry* registry) {
   registry->AddService<mojom::BlobChannel>(