[Sync] Rework Invalidator-related unit tests

sync/internal_api/public/base/invalidation_state.h

Index: sync/internal_api/public/base/invalidation_state.h
diff --git a/sync/internal_api/public/base/invalidation_state.h b/sync/internal_api/public/base/invalidation_state.h
index 8b790197a88d68f449adcd6bffa8ad3bc8ffe20f..695a9f8fd604954a54b28f9dcd3b92b50a6ac78c 100644
--- a/sync/internal_api/public/base/invalidation_state.h
+++ b/sync/internal_api/public/base/invalidation_state.h
@@ -7,17 +7,36 @@
 
 #include <string>
 
+namespace base {
+class DictionaryValue;
+class Value;
+}  // namespace
+
 namespace syncer {
 
 // Opaque class that represents an ack handle.
 // TODO(dcheng): This is just a refactoring change, so the class is empty for
 // the moment. It will be filled once we start implementing 'reminders'.
 class AckHandle {
+ public:
+  bool Equals(const AckHandle& other) const;
+
+  // Caller owns the returned Value.
+  base::Value* ToValue() const;

[Nicolas Zea, 2012/08/29 23:41:05]

have these return ScopedPtr's?

[akalin, 2012/08/30 22:20:38]

Done.

+
+  bool ResetFromValue(const base::Value& value);
 };
 
 struct InvalidationState {
   std::string payload;
   AckHandle ack_handle;
+
+  bool Equals(const InvalidationState& other) const;
+
+  // Caller owns the returned DictionaryValue.
+  base::DictionaryValue* ToValue() const;
+
+  bool ResetFromValue(const base::DictionaryValue& value);
 };
 
 }  // namespace syncer