[BackgroundSync] Remove BackgroundSyncRegistrationHandle

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 bcfe375d875390ed2618f4ec6647253652238dfe..be388ee0e900ebac445db939f708575c161856b5 100644
--- a/content/browser/background_sync/background_sync_registration.h
+++ b/content/browser/background_sync/background_sync_registration.h
@@ -28,6 +28,10 @@ class CONTENT_EXPORT BackgroundSyncRegistration {
   static const RegistrationId kInitialId;
 
   BackgroundSyncRegistration();
+  BackgroundSyncRegistration(const BackgroundSyncRegistration& other) = default;

[iclelland, 2016/03/08 16:48:14]

Style question -- these are defined default inline here, while the constructor
and destructor are defined in the .cc file -- is there any reason behind that?
(I ask because I had to go look at the constructor to make sure it was trivial)

[jkarlin, 2016/03/08 19:37:36]

Thanks. They're supposed to be in the header.

+  BackgroundSyncRegistration& operator=(
+      const BackgroundSyncRegistration& other) = default;
+
   ~BackgroundSyncRegistration();
 
   bool Equals(const BackgroundSyncRegistration& other) const;
@@ -57,9 +61,6 @@ class CONTENT_EXPORT BackgroundSyncRegistration {
   BackgroundSyncState sync_state_ = BackgroundSyncState::PENDING;
   int num_attempts_ = 0;
   base::Time delay_until_;
-
-
-  DISALLOW_COPY_AND_ASSIGN(BackgroundSyncRegistration);
 };
 
 }  // namespace content