BackgroundSyncManager tracks client registrations

content/browser/background_sync/background_sync_service_impl_unittest.cc

Index: content/browser/background_sync/background_sync_service_impl_unittest.cc
diff --git a/content/browser/background_sync/background_sync_service_impl_unittest.cc b/content/browser/background_sync/background_sync_service_impl_unittest.cc
index 5e177503d4ee938f9d0b59355319a939fb5eb2d1..b03dc78b531a55e35481d0a572d9ce41e0d70295 100644
--- a/content/browser/background_sync/background_sync_service_impl_unittest.cc
+++ b/content/browser/background_sync/background_sync_service_impl_unittest.cc
@@ -171,9 +171,11 @@ class BackgroundSyncServiceImplTest : public testing::Test {
     mojo::InterfaceRequest<BackgroundSyncService> service_request =
         mojo::GetProxy(&service_ptr_);
     // Create a new BackgroundSyncServiceImpl bound to the dummy channel
-    service_impl_.reset(new BackgroundSyncServiceImpl(
-        background_sync_context_.get(), service_request.Pass()));
+    background_sync_context_->CreateService(service_request.Pass());
     base::RunLoop().RunUntilIdle();
+
+    service_impl_ = *background_sync_context_->services_.begin();
+    ASSERT_TRUE(service_impl_);
   }
 
   // Helpers for testing BackgroundSyncServiceImpl methods
@@ -189,7 +191,7 @@ class BackgroundSyncServiceImplTest : public testing::Test {
       const BackgroundSyncService::UnregisterCallback& callback) {
     service_impl_->Unregister(
         BackgroundSyncPeriodicity::BACKGROUND_SYNC_PERIODICITY_ONE_SHOT,
-        sync->id, sync->tag, sw_registration_id_, callback);
+        sync->id, sw_registration_id_, callback);
     base::RunLoop().RunUntilIdle();
   }
 
@@ -219,7 +221,8 @@ class BackgroundSyncServiceImplTest : public testing::Test {
   int64 sw_registration_id_;
   scoped_refptr<ServiceWorkerRegistration> sw_registration_;
   BackgroundSyncServicePtr service_ptr_;
-  scoped_ptr<BackgroundSyncServiceImpl> service_impl_;
+  BackgroundSyncServiceImpl*
+      service_impl_;  // Owned by background_sync_context_
   SyncRegistrationPtr default_sync_registration_;
 };
 
@@ -257,7 +260,7 @@ TEST_F(BackgroundSyncServiceImplTest, Unregister) {
       default_sync_registration_.Clone(),
       base::Bind(&ErrorCallback, &unregister_called, &unregister_error));
   EXPECT_TRUE(unregister_called);
-  EXPECT_EQ(BackgroundSyncError::BACKGROUND_SYNC_ERROR_NOT_FOUND,
+  EXPECT_EQ(BackgroundSyncError::BACKGROUND_SYNC_ERROR_NOT_ALLOWED,
             unregister_error);
 }