Make GCMProfileService own GCMDriver, instead of deriving from it

chrome/browser/services/gcm/gcm_profile_service.cc

 // Copyright (c) 2013 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 "chrome/browser/services/gcm/gcm_profile_service.h"
 
 #include "base/files/file_path.h"
 #include "base/logging.h"
 #include "base/prefs/pref_service.h"
 #include "base/values.h"
 #include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/services/gcm/gcm_driver.h"
 #include "chrome/browser/signin/profile_identity_provider.h"
 #include "chrome/browser/signin/profile_oauth2_token_service_factory.h"
 #include "chrome/browser/signin/signin_manager_factory.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_version_info.h"
 #include "chrome/common/pref_names.h"
+#include "components/gcm_driver/gcm_client_factory.h"
 #include "components/pref_registry/pref_registry_syncable.h"
 #include "components/signin/core/browser/signin_manager.h"
 #include "google_apis/gaia/identity_provider.h"
 #include "net/url_request/url_request_context_getter.h"
 
 #if !defined(OS_ANDROID)
 #include "chrome/browser/ui/webui/signin/login_ui_service_factory.h"
 #endif
 
 namespace gcm {
@@ skipping 38 matching lines @@
       channel == chrome::VersionInfo::CHANNEL_CANARY ||
       channel == chrome::VersionInfo::CHANNEL_DEV) {
     on_by_default = true;
   }
   registry->RegisterBooleanPref(
       prefs::kGCMChannelEnabled,
       on_by_default,
       user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
 }
 
-GCMProfileService::GCMProfileService(Profile* profile)
-    : GCMDriver(scoped_ptr<IdentityProvider>(new ProfileIdentityProvider(
-          SigninManagerFactory::GetForProfile(profile),
-          ProfileOAuth2TokenServiceFactory::GetForProfile(profile),
+GCMProfileService::GCMProfileService(
+    Profile* profile,
+    scoped_ptr<GCMClientFactory> gcm_client_factory)
+    : profile_(profile) {
+  DCHECK(!profile->IsOffTheRecord());
+
 #if defined(OS_ANDROID)
-          NULL))),
+  LoginUIService* login_ui_service = NULL;
 #else
-          LoginUIServiceFactory::GetForProfile(profile)))),
+  LoginUIService* login_ui_service =
+      LoginUIServiceFactory::GetForProfile(profile_);
 #endif
-      profile_(profile) {
-  DCHECK(!profile->IsOffTheRecord());
+  driver_.reset(new GCMDriver(
+      gcm_client_factory.Pass(),
+      scoped_ptr<IdentityProvider>(new ProfileIdentityProvider(
+          SigninManagerFactory::GetForProfile(profile_),
+          ProfileOAuth2TokenServiceFactory::GetForProfile(profile_),
+          login_ui_service)),
+      profile_->GetPath().Append(chrome::kGCMStoreDirname),
+      profile_->GetRequestContext()));
+}
+
+GCMProfileService::GCMProfileService() : profile_(NULL) {
 }
 
 GCMProfileService::~GCMProfileService() {
 }
 
-void GCMProfileService::Shutdown() {
-  ShutdownService();
+void GCMProfileService::AddAppHandler(const std::string& app_id,
+                                      GCMAppHandler* handler) {
+  if (driver_)
+    driver_->AddAppHandler(app_id, handler);
 }
 
-std::string GCMProfileService::SignedInUserName() const {
-  if (IsStarted())
-    return identity_provider_->GetActiveUsername();
-  return std::string();
+void GCMProfileService::RemoveAppHandler(const std::string& app_id) {
+  if (driver_)
+    driver_->RemoveAppHandler(app_id);
 }
 
-bool GCMProfileService::ShouldStartAutomatically() const {
-  return GetGCMEnabledState(profile_) == ALWAYS_ENABLED;
+void GCMProfileService::Register(const std::string& app_id,
+                                 const std::vector<std::string>& sender_ids,
+                                 const GCMDriver::RegisterCallback& callback) {
+  if (driver_)
+    driver_->Register(app_id, sender_ids, callback);
 }
 
-base::FilePath GCMProfileService::GetStorePath() const {
-  return profile_->GetPath().Append(chrome::kGCMStoreDirname);
+void GCMProfileService::Shutdown() {
+  if (driver_) {
+    driver_->Shutdown();
+    driver_.reset();
+  }
 }
 
-scoped_refptr<net::URLRequestContextGetter>
-GCMProfileService::GetURLRequestContextGetter() const {
-  return profile_->GetRequestContext();
+void GCMProfileService::SetDriverForTesting(GCMDriver* driver) {
+  driver_.reset(driver);
 }
 
 }  // namespace gcm