BackgroundSyncManager tracks client registrations

content/browser/background_sync/background_sync_registration.h

Index: content/browser/background_sync/background_sync_registration.h
diff --git a/content/browser/background_sync/background_sync_registration.h b/content/browser/background_sync/background_sync_registration.h
index 94d764811eb6fc2136fa96e1e71f78894af5067d..a7dca8abc0b86ffe2e1cb605048985268b6c57c7 100644
--- a/content/browser/background_sync/background_sync_registration.h
+++ b/content/browser/background_sync/background_sync_registration.h
@@ -5,6 +5,7 @@
 #ifndef CONTENT_BROWSER_BACKGROUND_SYNC_BACKGROUND_SYNC_REGISTRATION_H_
 #define CONTENT_BROWSER_BACKGROUND_SYNC_BACKGROUND_SYNC_REGISTRATION_H_
 
+#include "base/macros.h"
 #include "base/memory/scoped_ptr.h"
 #include "content/browser/background_sync/background_sync.pb.h"
 #include "content/browser/background_sync/background_sync_registration_options.h"
@@ -20,6 +21,8 @@ class CONTENT_EXPORT BackgroundSyncRegistration {
   static const RegistrationId kInitialId;
 
   BackgroundSyncRegistration();
+  virtual ~BackgroundSyncRegistration();

[michaeln, 2015/08/21 02:39:24]

This class doesn't have any derivatives anymore, so nix the virtual dtor?

[jkarlin, 2015/08/25 17:32:58]

Done.

+
   bool Equals(const BackgroundSyncRegistration& other) const;
   bool IsValid() const;
 
@@ -38,6 +41,8 @@ class CONTENT_EXPORT BackgroundSyncRegistration {
   BackgroundSyncRegistrationOptions options_;
   RegistrationId id_ = kInvalidRegistrationId;
   SyncState sync_state_ = SYNC_STATE_PENDING;
+
+  DISALLOW_COPY_AND_ASSIGN(BackgroundSyncRegistration);
 };
 
 }  // namespace content