BackgroundSyncManager tracks client registrations

content/browser/background_sync/background_sync_service_impl.h

 // Copyright 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.
 
 #ifndef CONTENT_BROWSER_BACKGROUND_SYNC_BACKGROUND_SYNC_SERVICE_IMPL_H_
 #define CONTENT_BROWSER_BACKGROUND_SYNC_BACKGROUND_SYNC_SERVICE_IMPL_H_
 
-#include <vector>
-
+#include "base/id_map.h"
 #include "base/memory/ref_counted.h"
+#include "base/memory/scoped_vector.h"
 #include "content/browser/background_sync/background_sync_manager.h"
 #include "content/common/background_sync_service.mojom.h"
 #include "third_party/mojo/src/mojo/public/cpp/bindings/binding.h"
 
 namespace content {
 
 class BackgroundSyncContextImpl;
 
 class CONTENT_EXPORT BackgroundSyncServiceImpl
     : public NON_EXPORTED_BASE(BackgroundSyncService) {
  public:
   BackgroundSyncServiceImpl(
       BackgroundSyncContextImpl* background_sync_context,
       mojo::InterfaceRequest<BackgroundSyncService> request);
 
   ~BackgroundSyncServiceImpl() override;
 
  private:
   friend class BackgroundSyncServiceImplTest;
 
-
   // BackgroundSyncService methods:
   void Register(content::SyncRegistrationPtr options,
                 int64_t sw_registration_id,
                 const RegisterCallback& callback) override;
   void Unregister(BackgroundSyncPeriodicity periodicity,
                   int64_t id,

[michaeln, 2015/08/28 02:53:11]

int handle_id?

[jkarlin, 2015/09/02 23:51:41]

Done.

-                  const mojo::String& tag,
                   int64_t sw_registration_id,

[michaeln, 2015/08/28 02:53:11]

is sw_registration_id needed, is handle_id not sufficient?

[jkarlin, 2015/09/02 23:51:41]

Acknowledged.

                   const UnregisterCallback& callback) override;
   void GetRegistration(BackgroundSyncPeriodicity periodicity,
                        const mojo::String& tag,
                        int64_t sw_registration_id,
                        const GetRegistrationCallback& callback) override;
   void GetRegistrations(BackgroundSyncPeriodicity periodicity,
                         int64_t sw_registration_id,
                         const GetRegistrationsCallback& callback) override;
   void GetPermissionStatus(
       BackgroundSyncPeriodicity periodicity,
       int64_t sw_registration_id,
       const GetPermissionStatusCallback& callback) override;
+  void DuplicateRegistrationHandle(
+      int64_t handle_id,

[michaeln, 2015/08/28 02:53:11]

int to match BackgroundSyncRegistrationHandle::HandleId

[jkarlin, 2015/09/02 23:51:40]

Done.

+      int64_t service_worker_registration_id,

[michaeln, 2015/08/28 02:53:11]

is service_worker_registration_id needed?

[jkarlin, 2015/09/02 23:51:41]

Done.

+      const DuplicateRegistrationHandleCallback& callback) override;
+  void ReleaseRegistration(int64_t handle_id) override;
 
   void OnRegisterResult(const RegisterCallback& callback,
                         BackgroundSyncStatus status,
-                        const BackgroundSyncRegistration& result);
+                        scoped_ptr<BackgroundSyncRegistrationHandle> result);
   void OnUnregisterResult(const UnregisterCallback& callback,
                           BackgroundSyncStatus status);
   void OnGetRegistrationsResult(
       const GetRegistrationsCallback& callback,
       BackgroundSyncStatus status,
-      const std::vector<BackgroundSyncRegistration>& result);
+      scoped_ptr<ScopedVector<BackgroundSyncRegistrationHandle>> result);
 
   // Called when an error is detected on binding_.
   void OnConnectionError();
 
   // background_sync_context_ owns this.
   BackgroundSyncContextImpl* background_sync_context_;
 
   mojo::Binding<BackgroundSyncService> binding_;
 
+  // The registrations that the client might reference.
+  IDMap<BackgroundSyncRegistrationHandle, IDMapOwnPointer> active_handles_;
+
   base::WeakPtrFactory<BackgroundSyncServiceImpl> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(BackgroundSyncServiceImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_BACKGROUND_SYNC_BACKGROUND_SYNC_SERVICE_IMPL_H_