[Background Sync] Converting Blink code to the MVP API

Source/modules/background_sync/SyncRegistration.h

Index: Source/modules/background_sync/SyncRegistration.h
diff --git a/Source/modules/background_sync/SyncRegistration.h b/Source/modules/background_sync/SyncRegistration.h
index 7c549cc12063fca5d50fb0864578c246b7e8c351..66b38c6c7189e037715c5a3335216f9ed2bc0f08 100644
--- a/Source/modules/background_sync/SyncRegistration.h
+++ b/Source/modules/background_sync/SyncRegistration.h
@@ -5,6 +5,7 @@
 #ifndef SyncRegistration_h
 #define SyncRegistration_h
 
+#include "bindings/core/v8/Nullable.h"
 #include "bindings/core/v8/ScriptPromise.h"
 #include "bindings/core/v8/ScriptWrappable.h"
 #include "modules/background_sync/SyncRegistrationOptions.h"
@@ -26,48 +27,19 @@ public:
 
     virtual ~SyncRegistration();
 
-    bool hasAllowOnBattery() const { return !m_allowOnBattery.isNull(); }
-    bool allowOnBattery() const { return m_allowOnBattery.get(); }
-    void setAllowOnBattery(bool value) { m_allowOnBattery = value; }
-
-    bool hasId() const { return !m_id.isNull(); }
-    String id() const { return m_id; }
-    void setId(String value) { m_id = value; }
-
-    bool hasIdleRequired() const { return !m_idleRequired.isNull(); }
-    bool idleRequired() const { return m_idleRequired.get(); }
-    void setIdleRequired(bool value) { m_idleRequired = value; }
-
-    bool hasMaxDelay() const { return !m_maxDelay.isNull(); }
-    unsigned long maxDelay() const { return m_maxDelay.get(); }
-    void setMaxDelay(unsigned long value) { m_maxDelay = value; }
-
-    bool hasMinDelay() const { return !m_minDelay.isNull(); }
-    unsigned long minDelay() const { return m_minDelay.get(); }
-    void setMinDelay(unsigned long value) { m_minDelay = value; }
-
-    bool hasMinPeriod() const { return !m_minPeriod.isNull(); }
-    unsigned long minPeriod() const { return m_minPeriod.get(); }
-    void setMinPeriod(unsigned long value) { m_minPeriod = value; }
-
-    bool hasMinRequiredNetwork() const { return !m_minRequiredNetwork.isNull(); }
-    String minRequiredNetwork() const { return m_minRequiredNetwork; }
-    void setMinRequiredNetwork(String value) { m_minRequiredNetwork = value; }
+    bool hasTag() const { return !m_tag.isNull(); }
+    String tag() const { return m_tag; }
+    void setTag(String value) { m_tag = value; }
 
     ScriptPromise unregister(ScriptState*);
 
     DECLARE_TRACE();
 
 private:
-    SyncRegistration(const SyncRegistrationOptions&, ServiceWorkerRegistration*);
+    SyncRegistration(int64_t id, const SyncRegistrationOptions&, ServiceWorkerRegistration*);
 
-    Nullable<bool> m_allowOnBattery;
-    String m_id;
-    Nullable<bool> m_idleRequired;
-    Nullable<unsigned long> m_maxDelay;
-    Nullable<unsigned long> m_minDelay;
-    Nullable<unsigned long> m_minPeriod;
-    String m_minRequiredNetwork;
+    Nullable<int64_t> m_id;

[jkarlin, 2015/04/17 19:04:49]

Doesn't need to be nullable.

[iclelland, 2015/04/22 14:05:11]

Done. Fixed (also fixed in PeriodicSyncRegistration.cpp)

+    String m_tag;
 
     Member<ServiceWorkerRegistration> m_serviceWorkerRegistration;
 };