Implement Invalidator::Acknowledge

sync/internal_api/public/base/invalidation.h

 // Copyright (c) 2012 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 SYNC_INTERNAL_API_PUBLIC_BASE_INVALIDATION_H_
 #define SYNC_INTERNAL_API_PUBLIC_BASE_INVALIDATION_H_
 
 #include <string>
 
+#include "base/basictypes.h"
 #include "base/memory/scoped_ptr.h"
 
 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'.
+// Opaque class that represents an ack handle. We don't reuse the Tango ack

[akalin, 2012/10/19 13:27:16]

Tango -> invalidation::

[dcheng, 2012/10/19 19:38:11]

Done.

+// handles to avoid unnecessary dependencies.
 class AckHandle {
  public:
+  static AckHandle CreateUnique();
+  static AckHandle InvalidAckHandle();
+
   bool Equals(const AckHandle& other) const;
 
   scoped_ptr<base::Value> ToValue() const;
+  bool ResetFromValue(const base::Value& value);

[akalin, 2012/10/19 13:27:16]

comment that copy constructor and assignment operator welcome

(make default constructor private also?)

[dcheng, 2012/10/19 19:38:11]

Done. Unless I'm missing something, no default ctor will be emitted (and we
don't define one--that's why I had to add a default ctor for
syncer::Invalidation).

 
-  bool ResetFromValue(const base::Value& value);
+ private:
+  AckHandle(const std::string& state);
+
+  std::string state_;
 };
 
 // Represents a local invalidation, and is roughly analogous to
 // invalidation::Invalidation. It contains a payload (which may be empty) and an
 // associated ack handle that an InvalidationHandler implementation can use to
 // acknowledge receipt of the invalidation. It does not embed the object ID,
 // since it is typically associated with it through ObjectIdInvalidationMap.
 struct Invalidation {
-  std::string payload;
-  AckHandle ack_handle;
+  Invalidation();
 
   bool Equals(const Invalidation& other) const;
 
-  // Caller owns the returned DictionaryValue.
   scoped_ptr<base::DictionaryValue> ToValue() const;
+  bool ResetFromValue(const base::DictionaryValue& value);
 
-  bool ResetFromValue(const base::DictionaryValue& value);
+  std::string payload;
+  AckHandle ack_handle;
 };
 
 }  // namespace syncer
 
 #endif  // SYNC_INTERNAL_API_PUBLIC_BASE_INVALIDATION_H_