Geolocation: extract ContentBrowserClient methods specific to Geolocation into a class

chromecast/browser/cast_content_browser_client.cc

 // Copyright 2014 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 "chromecast/browser/cast_content_browser_client.h"
 
 #include <stddef.h>
 
 #include <string>
 #include <utility>
@@ skipping 65 matching lines @@
       new media::CastMojoMediaClient(
           base::Bind(&CastContentBrowserClient::CreateMediaPipelineBackend,
                      base::Unretained(browser_client)),
           base::Bind(&CastContentBrowserClient::CreateCdmFactory,
                      base::Unretained(browser_client))));
   return std::unique_ptr<::shell::ShellClient>(
       new ::media::MojoMediaApplication(std::move(mojo_media_client),
                                         quit_closure));
 }
 #endif  // defined(ENABLE_MOJO_MEDIA_IN_BROWSER_PROCESS)
+
+// A provider of services for Geolocation.
+class CastGeolocationServiceOverrides
+    : public content::GeolocationProvider::ServiceOverrides {
+ public:
+  explicit CastGeolocationServiceOverrides(CastBrowserContext* context)
+      : context_(context) {}
+  AccessTokenStore* CreateAccessTokenStore() override {
+    return new CastAccessTokenStore(context_);
+  }
+
+ private:
+  CastBrowserContext* context_;
+};
+
 }  // namespace
 
 CastContentBrowserClient::CastContentBrowserClient()
     : cast_browser_main_parts_(nullptr),
       url_request_context_factory_(new URLRequestContextFactory()) {}
 
 CastContentBrowserClient::~CastContentBrowserClient() {
   content::BrowserThread::DeleteSoon(
       content::BrowserThread::IO,
       FROM_HERE,
@@ skipping 157 matching lines @@
   // GLES2 contexts can be lost to a power-save mode, which breaks GPU canvas
   // apps.
   if (process_type == switches::kGpuProcess) {
     command_line->AppendSwitch(switches::kGpuNoContextLost);
   }
 #endif
 
   AppendExtraCommandLineSwitches(command_line);
 }
 
-content::AccessTokenStore* CastContentBrowserClient::CreateAccessTokenStore() {
-  return new CastAccessTokenStore(
+GeolocationProvider::ServiceOverrides*
+CastContentBrowserClient::GetGeolocationServiceOverrides() {
+  return new CastGeolocationServiceOverrides(
       CastBrowserProcess::GetInstance()->browser_context());
 }
 
 void CastContentBrowserClient::OverrideWebkitPrefs(
     content::RenderViewHost* render_view_host,
     content::WebPreferences* prefs) {
   prefs->allow_scripts_to_close_windows = true;
   // TODO(lcwu): http://crbug.com/391089. This pref is set to true by default
   // because some content providers such as YouTube use plain http requests
   // to retrieve media data chunks while running in a https page. This pref
@@ skipping 242 matching lines @@
         process_type, dumps_path, false /* upload */);
   // StartUploaderThread() even though upload is diferred.
   // Breakpad-related memory is freed in the uploader thread.
   crash_handler->StartUploaderThread();
   return crash_handler;
 }
 #endif  // !defined(OS_ANDROID)
 
 }  // namespace shell
 }  // namespace chromecast