Refactor to make BlimpLocationProvider accessible to content layer.

blimp/engine/app/blimp_content_browser_client.cc

 // Copyright (c) 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "blimp/engine/app/blimp_content_browser_client.h"
 #include "blimp/engine/app/blimp_browser_main_parts.h"
 #include "blimp/engine/app/settings_manager.h"
 #include "blimp/engine/mojo/blob_channel_service.h"
 #include "content/public/common/service_registry.h"
 
@@ skipping 21 matching lines @@
     return;
 
   blimp_browser_main_parts_->GetSettingsManager()->UpdateWebkitPreferences(
       prefs);
 }
 
 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>(
       base::Bind(&BlobChannelService::Create));
 }
 
 }  // namespace engine
 }  // namespace blimp