BackgroundSyncManager tracks client registrations

content/browser/background_sync/background_sync_registration_handle.h

Index: content/browser/background_sync/background_sync_registration_handle.h
diff --git a/content/browser/background_sync/background_sync_registration_handle.h b/content/browser/background_sync/background_sync_registration_handle.h
new file mode 100644
index 0000000000000000000000000000000000000000..1e247771f62aa611986f5ca5b34de96319f88d9d
--- /dev/null
+++ b/content/browser/background_sync/background_sync_registration_handle.h
@@ -0,0 +1,78 @@
+// 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_REGISTRATION_HANDLE_H_
+#define CONTENT_BROWSER_BACKGROUND_SYNC_BACKGROUND_SYNC_REGISTRATION_HANDLE_H_
+
+#include "base/callback.h"
+#include "base/memory/weak_ptr.h"
+#include "content/browser/background_sync/background_sync_registration.h"
+#include "content/browser/background_sync/background_sync_status.h"
+#include "content/common/content_export.h"
+
+namespace content {
+
+class BackgroundSyncManager;
+
+// Handle to BackgroundSyncRegistration that is exposed to clients. Each
+// BackgroundSyncRegistrationHandle is given a unique handle id (by the
+// BackgroundSyncManager) which is released at destruction.
+// BackgroundSyncRegistrationHandle objects must not be used (but may be
+// destroyed) after the BackgroundSyncManager has been deleted.
+class CONTENT_EXPORT BackgroundSyncRegistrationHandle {
+ public:
+  using HandleId = int;
+  using StatusCallback = base::Callback<void(BackgroundSyncStatus)>;
+
+  BackgroundSyncRegistrationHandle(

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

can this constructor be put in the private section? does anything other than the
mgr (a friend) have any reason to construct one of these?

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

Done.

+      base::WeakPtr<BackgroundSyncManager> background_sync_manager,
+      HandleId handle_id);
+  ~BackgroundSyncRegistrationHandle();
+
+  const BackgroundSyncRegistrationOptions* options() const {
+    DCHECK(background_sync_manager_);

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

#include "base/logging.h" for dcheck

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

Done.

+    return registration_->options();
+  }
+
+  SyncState sync_state() const {
+    DCHECK(background_sync_manager_);
+    return registration_->sync_state();
+  }
+
+  // Unregisters the background sync registration.  Calls |callback|
+  // with BACKGROUND_SYNC_STATUS_OK if it succeeds.
+  void Unregister(int64_t service_worker_id, const StatusCallback& callback);
+
+  // Returns true if the handle is backed by a BackgroundSyncRegistration in the
+  // BackgroundSyncManager.
+  bool IsValid() const;
+
+  HandleId handle_id() const { return handle_id_; }
+
+ private:
+  friend class BackgroundSyncManager;
+
+  BackgroundSyncRegistration* registration() {
+    DCHECK(background_sync_manager_);
+    return registration_;
+  }
+
+  // The BackgroundSyncManager is expected to remain alive for all operations
+  // except for possibly at destruction.
+  base::WeakPtr<BackgroundSyncManager> background_sync_manager_;
+
+  // Each BackgroundSyncRegistrationHandle is assigned a unique handle id.
+  // The BackgroundSyncManager maps the id to an internal pointer.
+  HandleId handle_id_;
+
+  // This is owned by background_sync_manager_ and is valid until handle_id_ is
+  // released in the destructor or background_sync_manager_ has been destroyed.
+  BackgroundSyncRegistration* registration_;
+
+  DISALLOW_COPY_AND_ASSIGN(BackgroundSyncRegistrationHandle);
+};
+
+}  // namespace
+
+#endif  // CONTENT_BROWSER_BACKGROUND_SYNC_BACKGROUND_SYNC_REGISTRATION_HANDLE_H_